Skip to content
New issue

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

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

Already on GitHub? # to your account

Don't expand Timegaph trees by default #801

Open
bhufmann opened this issue Aug 3, 2022 · 0 comments
Open

Don't expand Timegaph trees by default #801

bhufmann opened this issue Aug 3, 2022 · 0 comments
Labels
Theia UI frontend trace server protocol Needs changes to the trace-server-protocol itself Trace Server Involve changes in the trace server itself (in the incubator source code) UX User experience improvement

Comments

@bhufmann
Copy link
Collaborator

bhufmann commented Aug 3, 2022

When opening a time graph view the tree is fully expanded by default. This might be ok for some of such views, but not for all. For example the Resources Status view of the Trace Compass server show the CPU states and it's children by default. However, it should only show the first level and on user demand the children of should be shown.

Since this behaviour is not the same across the different time graph views, expansion level of the tree needs to be provided by the corresponding data provider back-end.

The solution could be similar to Eclipse Trace Compass which provides an API so that adopters can define an expansion level for the different time graph views. However, the solution here can be different.

This is how the Resources View looks like with code on master branch of today after opening it:
image

For this view all nodes should be collapsed instead:
image

@bhufmann bhufmann added Trace Server Involve changes in the trace server itself (in the incubator source code) trace server protocol Needs changes to the trace-server-protocol itself UX User experience improvement Theia UI frontend labels Aug 3, 2022
@bhufmann bhufmann changed the title Timegaph trees are expanded by default Don't expand Timegaph trees by default Aug 3, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Theia UI frontend trace server protocol Needs changes to the trace-server-protocol itself Trace Server Involve changes in the trace server itself (in the incubator source code) UX User experience improvement
Projects
None yet
Development

No branches or pull requests

1 participant