Skip to content

Conversation

irrenhaus3
Copy link
Contributor

@irrenhaus3 irrenhaus3 commented Aug 23, 2017

This PR makes the following changes to the GUI:

  • Plugin browser widgets no longer resize on mouse-over; the list stays at a fixed size at all times.
  • The detailed information for every plugin is no longer displayed within the plugin widgets. Instead, the title, icon and text of the header widget above the list is updated on the fly.
  • The default theme now highlights plugin widgets on mouse-over.

See the following demonstration.
Demo

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.

@Spekular
Copy link
Member

Spekular commented Aug 23, 2017

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..

@husamalhomsi
Copy link
Member

@irrenhaus3
Looks great!
Is the color of the highlighting here the same as the color of the highlighting of other GUI elements like the buttons in the editors?

@Spekular
I agree. This way, users who are familiar with the program and the instrument plug-ins (these users usually have tooltips disabled) can disable tooltips so they don't have to see these descriptions.

@irrenhaus3
Copy link
Contributor Author

@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. 😄
I'll also change the plugin description to display in a tooltip instead of the header. This comes along with a lot of reverts though, so it may take a while. But I definitely agree that it's a better option than having to deal with text size issues in a constrained space.

@irrenhaus3
Copy link
Contributor Author

so it may take a while.

Or not, lol.

@Spekular
Copy link
Member

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.

image

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!

@irrenhaus3
Copy link
Contributor Author

@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.
I do believe this is default QT behavior for tool-tips, so we don't need to take care of that in client code.

Also, there's no need to feel bad about bringing stuff like this up. I'm not one to take code reviews personally. 😉

@Umcaruje Umcaruje added this to the 1.3.0 milestone Oct 2, 2017
@tresf
Copy link
Member

tresf commented Nov 15, 2017

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.
Tested, works, merging.

@tresf tresf merged commit 918f449 into LMMS:master Nov 15, 2017
sdasda7777 pushed a commit to sdasda7777/lmms that referenced this pull request Jun 28, 2022
Remove widget resizing altogether, display plugin info in tooltip instead
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants