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

LV2 help window - Issue on closing with the 'x' button #6753

Closed
zonkmachine opened this issue Jun 28, 2023 · 4 comments · Fixed by #6957
Closed

LV2 help window - Issue on closing with the 'x' button #6753

zonkmachine opened this issue Jun 28, 2023 · 4 comments · Fixed by #6957
Assignees

Comments

@zonkmachine
Copy link
Member

zonkmachine commented Jun 28, 2023

Bug Summary

LV2 plugins with help button/window is disrupted when the help window is closed with the builtin [x] button.

Steps to reproduce

  • Open LV2 plugin. Either an instrument or an effect. Press [Help] (Open right corner, if the plugin has one which is not always the case). Help window opens.
  • Close the help window by once more pressing the [Help] button. Help window closes. This works.
  • Now open the help window once more and close it by pressing the top right [x] on the help window. Help window closes.
  • Repeat opening the help window and this time it's:
    A - Broken, no text in there.
    B - You need to press one extra time to get the focus on the help button.
    The window stays broken for this instance of the effect until lmms is restarted.

Affected LMMS versions

On master build since LV2 was introduced. At least since LV2-Options.

Screenshots

Help window opened.
help2

Help window opened a second time after closing it with the [x] instead of the [Help] button.
help3

@zonkmachine
Copy link
Member Author

I've looked at other software.

Ardour: Toggle only
help4
help5

Audacity: Can't find anyway to read the string. Doesn't look implemented.
Zrythm: Doesn't seem to show any info/help text. Not implemented.

Just hide the gui x for now?

@zonkmachine
Copy link
Member Author

zonkmachine commented Jun 28, 2023

Also. If the help window is maximized (with arrows left of the x/close button) it's not possible to close it again from within the open window. You need to open up the song window, reopen the lv2 plugin and then toggle the window with the help button.

@zonkmachine zonkmachine changed the title LV2 help window - issue when cloing with the 'x' button LV2 help window - Issue on closing with the 'x' button Aug 3, 2023
@JohannesLorenz JohannesLorenz self-assigned this Aug 19, 2023
@zonkmachine
Copy link
Member Author

zonkmachine commented Oct 29, 2023

Another issue found.

  • Start a default project
  • Add an LV2 plugin with a help window and open the help window.
  • Open another project or a new one altogether

The LV2 plugin help window is still there and can be closed manually. It should have been destroyed when the new project were loaded.

@JohannesLorenz
Copy link
Contributor

For now, solved all mentioned issues in a temporary branch (including the one from yesterday).

The last thing I am still unhappy about is the size of this window. I cannot find out how to let the window automatically have such a size that the contained QLabel fits. I might push the PR soon and move the question into that PR.

JohannesLorenz added a commit to JohannesLorenz/lmms that referenced this issue Nov 2, 2023
This fixes at least two issues:

* Help window is not synched with window manager's `X`-button
* Help window is not being closed on track destruction
JohannesLorenz added a commit that referenced this issue Nov 12, 2023
This fixes at least three issues:

* Help window is not synched with window manager's `X`-button
* Help window is not being closed on track destruction or on closing the plugin window
* Trims help window strings and force-adds a newline, because `QLabel::sizeHint` sometimes computes too small values. Now, together with #6956, all help windows fit their strings. Some help windows are too large by one line, but this seems better than forcing the user to resize them if they are too small by one line.
@github-project-automation github-project-automation bot moved this from To do to Done in Add LV2 support Nov 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants