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 <Insert> keyboard shortcut to create a new FX channel #1980

Merged
merged 2 commits into from
Apr 20, 2015

Conversation

Wallacoloo
Copy link
Member

This implements a keyboard shortcut from #1488 for quickly creating a new mixer channel.

The discussion wasn't clear as to where the channel should be inserted (e.g. to the right of the current channel or to the right of the rightmost channel). So I decided upon having the Insert shortcut mimic the behavior of the "+" button for consistency & ease of implementation.

I tested this change by making some channels, connecting them to a source and swapping them around; seemed to work as expected.

@tresf
Copy link
Member

tresf commented Apr 19, 2015

@Sti2nd we need to assign a valid shortcut to this. I don't know if insert is good or not.

@Wallacoloo
Copy link
Member Author

This really should have been settled in #1488, but I guess that shows the downside to megathreads in that some things get missed.

I think either ctrl+insert or shift+insert would make sense, but they are both awkward to press. Another option is something using the + key. Unfortunately, both ctrl+plus and ctrl+shift+plus are both commonly associated with zooming in.

One of the suggestions in the megathread was that insert+<number> should insert <number> new FX channels. While I personally don't see much use in adding 9 FX channels simultaneously, ctrl+1 seems like an intuitive way to add 1 new channel (and it could be extended to ctrl+2, ctrl+3, ... if desired).

Another possibility is ctrl+i. This again requires both hands, but is less awkward than ctrl+insert.

Currently, my vote goes towards implementing ctrl+1 (and only ctrl+1 - no higher numbers). But I don't know if this might have potential future conflicts.

@tresf
Copy link
Member

tresf commented Apr 20, 2015

Right now, we use CTRL + Insert (or possibly CTRL + Shift + insert) to add new bars to the Song Editor. Perhaps that should be carried over?

@Wallacoloo
Copy link
Member Author

@tresf (edited because I initially misunderstood what shift+insert did) Shift+Insert in the song editor appears to insert an empty bar to the right of the playhead. I suppose inserting a new FX channel is a similar action, so Shift+insert might indeed be the right shortcut to use here.

@tresf
Copy link
Member

tresf commented Apr 20, 2015

@Wallacoloo I guess I'm having a hard time with this one... CTRL + N is the closest I can think of, but that's already used for New Project. :\

That's why I'm hoping for some feedback from others. :)

@Sti2nd @Umcaruje @mikobuntu @unfa opinions? :)

@Wallacoloo
Copy link
Member Author

@tresf in case you read my comment through email, I immediately edited my previous comment pretty heavily because I was wrong about what shift+insert did in the song editor. I think shift+insert may be the best option mentioned so far.

@tresf
Copy link
Member

tresf commented Apr 20, 2015

@Wallacoloo let's march forward with that then. We may decide to change it down the road, but that way we can have something, and it can be fairly consistent. 👍

@Wallacoloo
Copy link
Member Author

@tresf switched to shift+insert. Tested & it works as expected.

@tresf
Copy link
Member

tresf commented Apr 20, 2015

👍

tresf added a commit that referenced this pull request Apr 20, 2015
Add <Insert> keyboard shortcut to create a new FX channel
@tresf tresf merged commit fa2cd22 into LMMS:master Apr 20, 2015
@tresf
Copy link
Member

tresf commented Apr 20, 2015

@Sti2nd please add to the chart. :)

@Sti2nd
Copy link
Contributor

Sti2nd commented Apr 22, 2015

Yup!

@Sti2nd Sti2nd mentioned this pull request Apr 22, 2015
6 tasks
@Wallacoloo Wallacoloo deleted the mixer-insert branch April 25, 2015 03:25
@miketurn
Copy link
Contributor

Sorry to ask this question but does this post mean that the "Shift + Insert" key has been implented into LMMS to insert the track or does it mean that the hotkey has been reserved for a future adding of this feature?
I am using version 1.1.3 (32Bit) and this hotkey does not work for me when testing it out in the mixer.
Thank You

@tresf
Copy link
Member

tresf commented Dec 16, 2016

This is the code commit, adding the feature as described (1.2). The other conversation is in regards to documentation.

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.

4 participants