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

Allow mousewheel on preset menu #14

Open
mkruselj opened this issue Feb 18, 2020 · 2 comments · Fixed by #25
Open

Allow mousewheel on preset menu #14

mkruselj opened this issue Feb 18, 2020 · 2 comments · Fixed by #25

Comments

@mkruselj
Copy link
Collaborator

Of course, it should behave logically - mousewheel up would select previous preset, and mousewheel down would select next preset, by number :)

baconpaul added a commit to baconpaul/dexed that referenced this issue Feb 18, 2020
Mosue wheel can change patch. Closes surge-synthesizer#14
baconpaul added a commit that referenced this issue Feb 18, 2020
Mosue wheel can change patch. Closes #14
@mkruselj
Copy link
Collaborator Author

mkruselj commented Feb 19, 2020

Testing this now, on Windows and with my MX Master S2, it jumps 2 patches at a time. Sometimes when going down right after we went up it goes by 1 patch, but then continues going down in 2 patch jumps.

@mkruselj mkruselj reopened this Feb 19, 2020
@mkruselj
Copy link
Collaborator Author

Also current implementation has inverted mousewheel behavior. I would expect when I do mousewheel down, that the patch lower in the list is loaded, and vice versa. Currently it's the other way around.

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 a pull request may close this issue.

1 participant