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.

Sign up for GitHub

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

Jump to bottom

Keybinding to Arm Modulation #6549

Closed
baconpaul opened this issue Aug 20, 2022 · 3 comments · Fixed by #6550
Closed

Keybinding to Arm Modulation #6549

baconpaul opened this issue Aug 20, 2022 · 3 comments · Fixed by #6550
Labels
Feature Request New feature request UX Issues related to user experience (UX) - mouse, touch, keyboard, MIDI inputs, etc.
Milestone
Surge XT 1.1.1

Comments

@baconpaul
Copy link
Collaborator

Used to be tab
We basically took that out when we added proper focus traversal
and added a hundred other keybindings
but not an arm one

I propose COMMAND-M which i sunused

@mkruselj unless you object I'll bang this into 1.1.1
@baconpaul baconpaul added the Feature Request New feature request label Aug 20, 2022
@baconpaul baconpaul added this to the Surge XT 1.1.1 milestone Aug 20, 2022
@mkruselj mkruselj added the UX Issues related to user experience (UX) - mouse, touch, keyboard, MIDI inputs, etc. label Aug 20, 2022
@mkruselj
Copy link
Collaborator

mkruselj commented Aug 20, 2022 •
edited
Loading

I wonder for how many DAWs Ctrl+M is an already bound shortcut tho...

(I agree with adding the action for keyboard shortcuts, though, of course.)

@mkruselj
Copy link
Collaborator

Maybe Alt/Opt+A (for "Arm")?

@baconpaul
Copy link
Collaborator Author

Oh sure!

baconpaul added a commit to baconpaul/surge that referenced this issue Aug 20, 2022
@baconpaul
Modulation: Keybind to Arm and Restore Middle Mouse
6911909 
1. Keybind CMD-A toggles modulation
2. The middle mouse button forward-to-main-frame wasn't in
   ModulationSourceButton so that command selected and armed
   which wasn't desired.

Closes surge-synthesizer#6549
@baconpaul baconpaul mentioned this issue Aug 20, 2022
Merged
mkruselj pushed a commit that referenced this issue Aug 20, 2022
@baconpaul
Modulation: Keybind to Arm and Restore Middle Mouse (#6550)
6e7708e 
* Modulation: Keybind to Arm and Restore Middle Mouse

1. Keybind Alt/Opt+A toggles modulation
2. The middle mouse button forward-to-main-frame wasn't in
   ModulationSourceButton so that command selected and armed
   which wasn't desired.

Closes #6549
@bobobo199733 bobobo199733 mentioned this issue Mar 5, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Feature Request New feature request UX Issues related to user experience (UX) - mouse, touch, keyboard, MIDI inputs, etc.
Projects
None yet
Milestone
Surge XT 1.1.1
Development

Successfully merging a pull request may close this issue.

Modulation: Keybind to Arm and Restore Middle Mouse baconpaul/surge
2 participants
@mkruselj @baconpaul