-
Notifications
You must be signed in to change notification settings - Fork 27.9k
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
Provide a global context menu to auto expand lazy variables #179742
Comments
I'm not opposed to it but don't want to put every option in the context menu necessarily. Have you heard the same feedback @connor4312? |
I think a context menu action like "Reveal All Children" (wording TBD) that would apply to a variable and all of its children would make sense. |
But not one to set the default via the setting? |
Yea, I'm not a fan of a context menu to change the setting. Context menu actions should be _context_ual. |
I think it is contextual, it would immediately expand all the visible variables and also set the default. It's similar to some of the toggles we have for notebook UI. A "Reveal All Children" command actually doesn't feel that helpful to me, I think it usually wouldn't save you that many clicks |
Yes, the menu should act as a toggle between different data views. When toggled, it should change the display style of all visible nodes instantly. I also expect the operation to be saved. This type of menu can also be applied to other cases in the future, such as showing values as Hex or Dec. We have added it in Java extension, and it has a high usage rate as well . |
Hey @roblourens, this issue might need further attention. @testforstephen, you can help us out by closing this issue if the problem no longer exists, or adding more information. |
Looking at this again, I actually don't like having a context menu option that controls a setting, because it should probably work at the workspace scope but it shouldn't go write to my workspace settings file. I think the setting as it exists is fine. |
as https://code.visualstudio.com/updates/v1_67#_debugging said:
|
We use lazy variables in the Java Debugger extension to display the toString() value of objects on demand. However, some users have complained that this makes the user experience worse and that it requires lots of clicks to see the details of the variables (See microsoft/vscode-java-debug#1297 (comment) and microsoft/java-debug#401 (comment)). VS Code has a setting to automatically expand lazy variables (Defaults to false), but most users are not aware of it.
In Java Debugger, we have exposed a context menu "Auto Expand Lazy Variables" to promote discovery and it has received more clicks than other context menu items. And we think this could be a useful feature for VS Code to provide globally.
The text was updated successfully, but these errors were encountered: