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.

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

feat: POC - virtual tree configurable #1610

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Jinnie
Copy link
Contributor

@Jinnie Jinnie commented Nov 4, 2024

Tree with configurable virtual scrolling POC.
Done to test performance of tree where virtual scrolling can be defined by the user only on specific nodes
This POC helps prove that there is significant improvement in performance. There are serious complications, though, in terms of visualization patterns, viewport configuration, accessibility.

PR Checklist

Please check if your PR fulfills the following requirements:

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)
  • If applicable, have a visual design approval

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • Other... Please describe:

What is the current behavior?

Issue Number: CDE-2320

What is the new behavior?

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

Copy link
Contributor

github-actions bot commented Nov 4, 2024

👋 @Jinnie,

  • 😭 The build for this PR has failed
  • 🗒 Please check the build log
  • 🖐 You can always follow up here. If you're a VMware employee, you can also reach us on our internal Clarity Support space

Thank you,

🤖 Clarity Release Bot

@Jinnie Jinnie self-assigned this Nov 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants