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

Add searching to vestige instrument view #7369

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jiffygist
Copy link

@jiffygist jiffygist commented Jul 6, 2024

Solves #308, listed in #5433.

Makes finding knobs easier when working with VST plugins with lots of knobs (like Serum).

I'm very much noob in lmms and qt, but this was requested by my brother, so I decided to try implementing it))

@jiffygist
Copy link
Author

jiffygist commented Jul 7, 2024

I added analogous changes for VST Effects (some effect plugins also have lots of knobs). I also reduced max length of the search bar to 35 since this is also the length of paramStr.

@Rossmaxx
Copy link
Contributor

screenshots pls

@Rossmaxx
Copy link
Contributor

Not really a concern but just so you know, commiting to master is a bad idea, you should consider branching off for every feature you make. Thanks for the contribution ;-)

@jiffygist
Copy link
Author

It looks like this
https://github.com/user-attachments/assets/23c063a8-777f-4226-a2ad-4609786ff404

Of course it would be better to realign them on grid, but it is not obvious for me how to do that.

@Rossmaxx
Copy link
Contributor

#6456 might be related

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.

None yet

2 participants