Skip to content
This repository was archived by the owner on Feb 22, 2018. It is now read-only.

hover tooltip for top-level identifiers with source maps should show value #72

Closed
jmesserly opened this issue Feb 27, 2015 · 3 comments
Closed

Comments

@jmesserly
Copy link
Contributor

we generate top level fields as exports.name, which the JS debugger won't show the value for. babeljs generates a pattern that keeps a local around. We could do that too, does it seem worth it?

export let foo = 1;
foo++

desugars to

var foo = exports.foo = 1;
foo = exports.foo += 1;

the local "foo" means that "foo" hover will work

@jmesserly
Copy link
Contributor Author

(but the tradeoff is uglier code for the JS reader)

@jmesserly jmesserly mentioned this issue Feb 27, 2015
@jmesserly
Copy link
Contributor Author

and here's a live version of that babeljs example

@jmesserly
Copy link
Contributor Author

@jacob314 I believe this sort of thing is fixed by the debugger extension. so closing.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

No branches or pull requests

1 participant