-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Separator on plots plugin acts more like a button than a resize area #9747
Comments
If that is the road we are taking then yes, (it would be a tool button with a expanding vertical policy)
I guess, but the UX of having the button right there feels nicer? Besides ease of implementation any particular preferences @jnsebgosselin ? The other option is that the dragging actually reduces gradually the size of the thumbnails (from a minimum value, to a maximum value) |
Is it something that would be worth implementing? I think this wouldn't be too complicated to do and could be done as part of PR #9720. |
I do not have a particular favorite to be honest, it is the UX/UI that bothers me. Either way is fine by me as long as the UX/UI is consistent.
If either seems like too much for nothing then I guess the button on the toolbar is fine 🤷♂ Thoughts @ccordoba12 ? |
I also prefer this option instead of the current behavior. Thankfully is not too complicated to implement (according to @jnsebgosselin), so it'd be nice to have it as part of PR #9720. |
Ok good, I'll work on it 👍 |
Thanks @jnsebgosselin ! |
Question @jnsebgosselin then in this case we would still need the toggle button in the toolbar? so that users can completely hide it? The resize should probably have a minimum value (otherwise it would be "lost"? |
We can set a minimum value on the resize and if the user continue to drag after hitting the minimum, the scrollbar would collapse. That being said, I think that also adding a collapse button in the toolbar is a good idea. |
The separator has the appearance of allowing the user to change the width gradually, but currently the behavior is more of something being toggled (a button instead of a draggable area)
The text was updated successfully, but these errors were encountered: