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

New LADSPA control dialog too tall on low-res screens for plugins with many controls #6982

Closed
kamkamkamkamkamkamkamkam opened this issue Nov 13, 2023 · 4 comments · Fixed by #7019
Labels

Comments

@kamkamkamkamkamkamkamkam

image
It'd be really nice to be able to shrink these these for those of us using resolutions under 1080p (screenshot is 1366x768)

@musikBear
Copy link

could the theme be a thing?
Is it classic?

@kamkamkamkamkamkamkamkam
Copy link
Author

could the theme be a thing? Is it classic?

It isn't much better at all with the default theme
image
Regardless of the theme being used, being able to vertically resize these plugins would be extremely useful. Having tall plugins like that can be a pain.

michaelgregorius added a commit to michaelgregorius/lmms that referenced this issue Dec 8, 2023
Remove the code which computes a minimum height for the LADSPA dialogs. It was intended to make sure that no scrollbar is shown in most cases. However, doing so came at the cost that the computed height was the minimum height as well. Therefore the dialogs took a lot of space on low-res displays and could not be made smaller.

After the removal the behavior is still sane. Small dialogs are shown in full and dialogs which are larger, e.g. "Calf Equalizer 12 Band LADSPA", seem to be sized around half the height of the workspace and show scrollbars.
@michaelgregorius
Copy link
Contributor

Hi @kamkamkamkamkamkamkamkam, this problem should be fixed with pull request #7019.

@michaelgregorius michaelgregorius linked a pull request Dec 8, 2023 that will close this issue
michaelgregorius added a commit that referenced this issue Dec 31, 2023
Remove the code which computes a minimum height for the LADSPA dialogs. It was intended to make sure that no scrollbar is shown in most cases. However, doing so came at the cost that the computed height was the minimum height as well. Therefore the dialogs took a lot of space on low-res displays and could not be made smaller.

After the removal the behavior is still sane. Small dialogs are shown in full and dialogs which are larger, e.g. "Calf Equalizer 12 Band LADSPA", seem to be sized around half the height of the workspace and show scrollbars.
@michaelgregorius
Copy link
Contributor

Hi @kamkamkamkamkamkamkamkam! I have just merged a fix with pull request #7019. If you still encounter problems please reopen this issue or create a new one.

sakertooth pushed a commit to sakertooth/lmms that referenced this issue Jan 5, 2024
Remove the code which computes a minimum height for the LADSPA dialogs. It was intended to make sure that no scrollbar is shown in most cases. However, doing so came at the cost that the computed height was the minimum height as well. Therefore the dialogs took a lot of space on low-res displays and could not be made smaller.

After the removal the behavior is still sane. Small dialogs are shown in full and dialogs which are larger, e.g. "Calf Equalizer 12 Band LADSPA", seem to be sized around half the height of the workspace and show scrollbars.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants