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

Enhancements: Allow scroll secondary menus #43

Closed
papampi opened this issue Jan 28, 2024 · 3 comments
Closed

Enhancements: Allow scroll secondary menus #43

papampi opened this issue Jan 28, 2024 · 3 comments
Assignees
Labels
bug Something isn't working fixed

Comments

@papampi
Copy link

papampi commented Jan 28, 2024

When there are too many values in secondary value screen some top and bottom values are not visible, anyway to make them visible by scroll or other ways?
Now when I try to get the mouse to that section it goes hidden.

Screenshot from 2024-01-28 13-46-40

@papampi papampi added the bug Something isn't working label Jan 28, 2024
@ljuzig ljuzig self-assigned this Jan 28, 2024
@ljuzig
Copy link
Collaborator

ljuzig commented Jan 28, 2024

I can't make it scrollable cuz currently it doesn't interact with the mouse. And I don't think I want to change this behaviour because it would add too much complexity to the menus. I think I will try to detect whether the size is too big and if so, separate the data into several columns.

@ljuzig ljuzig closed this as completed in 24b9033 Jan 28, 2024
@ljuzig
Copy link
Collaborator

ljuzig commented Jan 28, 2024

@papampi Try this build:
[email protected]

Now the list should be split when it's too long:
image

ljuzig added a commit that referenced this issue Jan 28, 2024
@ljuzig ljuzig added the fixed label Jan 28, 2024
@papampi
Copy link
Author

papampi commented Jan 28, 2024

This is much better than my suggestion to scroll.
Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working fixed
Projects
None yet
Development

No branches or pull requests

2 participants