Implement autoExpandLevel for EntryModel trees#1202
Conversation
5891711 to
151a424
Compare
|
Fixes #801 |
bhufmann
left a comment
There was a problem hiding this comment.
tsp-typescript-client 0.7.0 has been deployed. Please update the package.json file for traceviewer-base and traceviewer-react-components to this version.
I tested this change with trace server that with applied this setting. It works well:
eclipse-tracecompass/org.eclipse.tracecompass#289
|
I updated Seems to work fine: |
151a424 to
92c004b
Compare
|
Add support for the autoExpandLevel flag to automatically collapse tree nodes beyond the specified depth level. When autoExpandLevel is set, nodes at depths greater than the specified level are automatically collapsed in the tree view. Related to eclipse-cdt-cloud/tsp-typescript-client#130 Signed-off-by: Will Yang <william.yang@ericsson.com>
92c004b to
182a865
Compare
|
Fixed the linting errors (hopefully) |
|
IP Ticket to open:
ATM, when I run |
|
A review is required for npm/npmjs/-/allure-commandline/2.34.0. Note: |
|
@williamsyang-work Update: seems you already are on the latest baseline - not sure why |
|
All good for the license check. |
marcdumais-work
left a comment
There was a problem hiding this comment.
LGTM, thanks @williamsyang-work !
I quickly tried the new feature in the vscode extension, using yarn link, and it worked well too:
bhufmann
left a comment
There was a problem hiding this comment.
Looks good to me. Thanks for adding this.


What it does
Add support for the autoExpandLevel flag to automatically collapse tree nodes beyond the specified depth level. When autoExpandLevel is set, nodes at depths greater than the specified level are automatically collapsed in the tree view.
Related to eclipse-cdt-cloud/tsp-typescript-client#130
How to test
What I did was comment out
getCollapsedNodesFromAutoExpandLeveland add a replacement test call using different values forautoExpandLevelto ensure that it was working correctly for each value.Follow-ups
No follow ups required.
Review checklist