-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
UI: New behavior for the Plugin Browser #3773
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
Conversation
…der instead. This needs some work still; it crops the description text if the window size is too small.
This looks great. I do still think that having the description at the top might not be ideal. Normally I'd assume users' eyes "seek" to where their mouse is, and with this design they need to dart back and forth between wherever they are the list and the top to see information about the plugin. I think an alternative would be to display a tooltip. This gives the user a single focal point (plugin list + information + mouse in one area), but doesn't rely on the current expanding description design.. |
@irrenhaus3 @Spekular |
@Sawuare No, I just put together an arbitrary highlighting gradient, because I didn't know there was a standard for it. I'll change it to use the standard instead. 😄 |
Or not, lol. |
With this done, I suppose the header for Instrument Plugins could be shortened to the same height as the other tabs? I also really hate to nitpick when you've already fixed the things I complained about, but does the tooltip stay inside the window properly? If my LMMS window is so short that a plugin in the list ends up right at the bottom edge, the tooltip should hopefully shift up so it doesn't go offscreen. Otherwise it could be wonky or impossible to read it. If it does go offscreen, it should hopefully let the users scroll down to read the whole text. This is what the current implementation does, so while less convenient, it wouldn't be a regression for the tooltip to act this way. Thanks so much for working on this! |
@Spekular If the location of the tool-tip is properly within the desktop bounds, it displays underneath the cursor (if necessary, partly outside the LMMS window borders). If it's at the bottom edge of the desktop, though, it shifts upwards and displays above the cursor, so it's still fully contained in the visible desktop area. Same goes for behavior at the left/right desktop edges. Also, there's no need to feel bad about bringing stuff like this up. I'm not one to take code reviews personally. 😉 |
Cleans up code, makes the UI more consistent, approved by UI coders. Another benefit is the average user may wait for the animation to finish each time, so this has a slight productivity gain. |
Remove widget resizing altogether, display plugin info in tooltip instead
This PR makes the following changes to the GUI:
See the following demonstration.

Note This PR is currently not ready to merge. There's still a major issue regarding the height of the description text label, causing said text to be cropped in some circumstances. The main reason I'm opening it anyway is for people to look over it, review it and suggest improvements and solutions to existing problems.Edit2: Since we're now using tool-tips to display plugin details, the header height problem is obsolete. The PR is ready for merging.
Edit: This PR sort-of implements #3507 in that it removes the resizing behaviour altogether.