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

Scrolling does not work for the LFO waveform selector #1648

Closed
VincyZed opened this issue Mar 21, 2020 · 3 comments · Fixed by #1973
Closed

Scrolling does not work for the LFO waveform selector #1648

VincyZed opened this issue Mar 21, 2020 · 3 comments · Fixed by #1973
Labels
UX Issues related to user experience (UX) - mouse, touch, keyboard, MIDI inputs, etc.
Milestone

Comments

@VincyZed
Copy link
Collaborator

Describe the bug
Scrolling does not work for the LFO waveform selector.

Expected behavior
Scrolling on that area should switch LFO waveforms, like it does on other button arrays like this one.

Screenshots
image

Additional context
This has been there for as long as I remember, it's not a recently introduced bug or anything. Just a minor inconvenience :)

@VincyZed VincyZed added the UX Issues related to user experience (UX) - mouse, touch, keyboard, MIDI inputs, etc. label Mar 21, 2020
@baconpaul
Copy link
Collaborator

Yeah there's a particular reason this is annoying to add, but I can overcome it.

Also chatting with dave, I'll probably make the wheel support on switches a global UI option

@baconpaul
Copy link
Collaborator

Oh ha I added a wheel for the step sequencer and that wheel command can do this also. Just a note to myself.

baconpaul added a commit to baconpaul/surge that referenced this issue May 28, 2020
@baconpaul
Copy link
Collaborator

Push coming to fix this this morning.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
UX Issues related to user experience (UX) - mouse, touch, keyboard, MIDI inputs, etc.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants