Skip to content

ComponentViewer: added "expand all" button next to "collapse all"#866

Closed
thorstendb-ARM wants to merge 6 commits intomainfrom
addExpandAllButton
Closed

ComponentViewer: added "expand all" button next to "collapse all"#866
thorstendb-ARM wants to merge 6 commits intomainfrom
addExpandAllButton

Conversation

@thorstendb-ARM
Copy link
Copy Markdown
Collaborator

Fixes

Adds an "expand all" button nect to the "collapse all"

Screenshots

image

Checklist

  • 🤖 This change is covered by unit tests (if applicable).
  • 🤹 Manual testing has been performed (if necessary).
  • 🛡️ Security impacts have been considered (if relevant).
  • 📖 Documentation updates are complete (if required).
  • 🧠 Third-party dependencies and TPIP updated (if required).

@qltysh
Copy link
Copy Markdown

qltysh bot commented Feb 27, 2026

Qlty

Coverage Impact

⬇️ Merging this pull request will decrease total coverage on main by 0.08%.

Modified Files with Diff Coverage (3)

RatingFile% DiffUncovered Line #s
Coverage rating: A Coverage rating: A
src/views/component-viewer/component-viewer-tree-view.ts100.0%
Coverage rating: A Coverage rating: A
src/views/component-viewer/scvd-gui-tree.ts0.0%132
Coverage rating: A Coverage rating: A
src/views/component-viewer/component-viewer-main.ts36.4%80-124
Total50.0%
🤖 Increase coverage with AI coding...

In the `addExpandAllButton` branch, add test coverage for this new code:

- `src/views/component-viewer/component-viewer-main.ts` -- Line 80-124
- `src/views/component-viewer/scvd-gui-tree.ts` -- Line 132

🚦 See full report on Qlty Cloud »

🛟 Help
  • Diff Coverage: Coverage for added or modified lines of code (excludes deleted files). Learn more.

  • Total Coverage: Coverage for the whole repository, calculated as the sum of all File Coverage. Learn more.

  • File Coverage: Covered Lines divided by Covered Lines plus Missed Lines. (Excludes non-executable lines including blank lines and comments.)

    • Indirect Changes: Changes to File Coverage for files that were not modified in this PR. Learn more.

Copy link
Copy Markdown
Collaborator

@jreineckearm jreineckearm left a comment

Choose a reason for hiding this comment

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

Changes LGTM

Copy link
Copy Markdown
Collaborator

@jreineckearm jreineckearm left a comment

Choose a reason for hiding this comment

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

Hang on, build is broken. Had locally fixed those two:

(provider.getRoots as jest.Mock) = jest.fn(() => [root]);

instead of

provider.getRoots = jest.fn(() => [root]);

@thorstendb-ARM thorstendb-ARM marked this pull request as draft March 2, 2026 15:40
@thorstendb-ARM
Copy link
Copy Markdown
Collaborator Author

closing, new one is here: #918

@thorstendb-ARM thorstendb-ARM deleted the addExpandAllButton branch March 19, 2026 12:42
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.

2 participants