Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Evaluate based hover #3245

Closed
3 tasks done
isidorn opened this issue Feb 22, 2016 · 1 comment
Closed
3 tasks done

Evaluate based hover #3245

isidorn opened this issue Feb 22, 2016 · 1 comment
Labels
debug Debug viewlet, configurations, breakpoints, adapter issues testplan-item
Milestone

Comments

@isidorn
Copy link
Contributor

isidorn commented Feb 22, 2016

For node debugging when a user hover over a word we evaluate that word instead of traversing the variables scopes as we did previously. Due to this, user hovering over a function call can potentially mutate the program state and might be dangerous. We have not yet decided if we will ship this and are looking for feedback. Verify:

  • hovering works as before and you can inspect both complex and primitive values
  • be creative and by hovering try to trigger a function call which could mutate the program state
@isidorn isidorn added debug Debug viewlet, configurations, breakpoints, adapter issues testplan-item labels Feb 22, 2016
@isidorn isidorn added this to the Feb 2016 milestone Feb 22, 2016
@egamma egamma mentioned this issue Feb 22, 2016
97 tasks
@alexdima
Copy link
Member

I would not ship with this experiment.

@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 18, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
debug Debug viewlet, configurations, breakpoints, adapter issues testplan-item
Projects
None yet
Development

No branches or pull requests

2 participants