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.

Sign up for GitHub

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

Jump to bottom

Expand local variables by default #14738

Closed
mvtec-richter opened this issue Jan 17, 2025 · 2 comments · Fixed by #15017
Closed

Expand local variables by default #14738

mvtec-richter opened this issue Jan 17, 2025 · 2 comments · Fixed by #15017

Comments

@mvtec-richter
Copy link
Contributor

Feature Description:

When using the debugger in Theia, the first thing I need to do is to expand the local variable scope to see the values of the local variables of the current scope. This is annoying, as this has to be done every time I start debugging again.
The should expand the local variables by default.
VSCode does this too.
@mvtec-richter
Copy link
Contributor Author

I already looked into the implementation and saw, that initial state of expanding is set here:
https://github.com/eclipse-theia/theia/blob/master/packages/core/src/browser/source-tree/source-tree.ts#L67
Unfortuately, it's always initialized with false.
I wonder, if it would be ok to add a boolean property expandByDefault to CompositeTreeElement and use this in the toNode method?

@JonasHelming
Copy link
Contributor

@rschnekenbu

mvtec-richter added a commit to mvtecsoftware/theia that referenced this issue Feb 24, 2025
@mvtec-richter
Expand local variables by default
cbd9be8 
Looking at the local variables is a very comon thing. In Theia,
the local variables are not visible when starting to debug. This
commit will expand the local variables, so they become visible
when starting to debug.

Fixes eclipse-theia#14738
mvtec-richter added a commit to mvtecsoftware/theia that referenced this issue Feb 24, 2025
@mvtec-richter
Expand local variables by default
5fa20d6 
Looking at the local variables is a very comon thing. In Theia,
the local variables are not visible when starting to debug. This
commit will expand the local variables, so they become visible
when starting to debug.

Fixes eclipse-theia#14738

Signed-off-by: Florian Richter <[email protected]>
@mvtec-richter mvtec-richter mentioned this issue Feb 24, 2025
Merged
2 tasks
martin-fleck-at pushed a commit that referenced this issue Mar 20, 2025
@mvtec-richter
Expand local variables by default (#15017)
455f967 
Looking at the local variables is a very comon thing. In Theia,
the local variables are not visible when starting to debug. This
commit will expand the local variables, so they become visible
when starting to debug.

Fixes #14738

Signed-off-by: Florian Richter <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Expand local variables by default mvtecsoftware/theia
2 participants
@JonasHelming @mvtec-richter