Skip to content

Conversation

@bhufmann
Copy link
Contributor

Note: The classic Eclipse UI has been doing this in the UI code without that setting (see ResourcesView).

What it does

linux: set auto-expand-level in ResourcesStatusDataProvider

Setting the auto-expand-level to 1 of the returned TmfTreeModel when calling fetchTree() will indicate to the clients that only the top-level should be expanded and their children supposed to be collapsed.

With this trace server clients can apply this new setting and it fixes FE issue eclipse-cdt-cloud/theia-trace-extension#801.

How to test

Run server with this change and run theia-trace-extension change eclipse-cdt-cloud/theia-trace-extension#1202.

Follow-ups

N/A

Review checklist

  • As an author, I have thoroughly tested my changes and carefully followed the instructions in this template

Setting the auto-expand-level to 1 of the returned TmfTreeModel when
calling fetchTree() will indicate to the clients that only the top-level
should be expanded and their children supposed to be collapsed.

With this trace server clients can apply thhis and it fixes FE issue
eclipse-cdt-cloud/theia-trace-extension#801

Note: The classic Eclipse UI has been doing this in the UI code without
that setting (see ResourcesView).

Signed-off-by: Bernd Hufmann <[email protected]>
Copy link
Contributor

@marcdumais-work marcdumais-work left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(placebo approval, as a non-committer) LGTM!

image

@bhufmann bhufmann merged commit 539acbb into eclipse-tracecompass:master Jun 17, 2025
4 checks passed
@bhufmann bhufmann deleted the fix-expand-level_rs branch June 17, 2025 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants