-
Notifications
You must be signed in to change notification settings - Fork 72
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
[BUG] Tree view expanded view #1118
Labels
bug
Something isn't working
Comments
ill take this issue |
This was referenced Oct 29, 2023
Closed
Closed
keskami
added a commit
to keskami/oui
that referenced
this issue
Oct 31, 2023
Signed-off-by: keskami <[email protected]>
keskami
added a commit
to keskami/oui
that referenced
this issue
Oct 31, 2023
Signed-off-by: keskami <[email protected]>
keskami
added a commit
to keskami/oui
that referenced
this issue
Nov 3, 2023
Signed-off-by: keskami <[email protected]>
ashwin-pc
pushed a commit
that referenced
this issue
Dec 12, 2023
* Treeview #1118 Signed-off-by: keskami <[email protected]> * revert Signed-off-by: keskami <[email protected]> * [BUG] Tree view expanded view #1118 Signed-off-by: keskami <[email protected]> * CHANGELOG updated Signed-off-by: keskami <[email protected]> --------- Signed-off-by: keskami <[email protected]> Co-authored-by: Josh Romero <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the bug
The current implementation of tree view expanded node has a limit in height of
100vh
. This is an issue when the children of the a tree node take more then100vh
of space. The next tree nodes overlap content on UI. In our implementation we useoverflow
property to make the tree view scrollable and removed the max height limit.To Reproduce
Steps to reproduce the behavior:
Probable solutions
For our use-case, removing max-height limit worked:
Screenshots
If applicable, add screenshots to help explain your problem.
Host/Environment (please complete the following information):
Additional context
Add any other context about the problem here.
The text was updated successfully, but these errors were encountered: