carrierdown / mutateful

Add-on for Ableton Live that enables live coding functionality fully integrated into Live's session view.
GNU General Public License v3.0
406 stars 13 forks source link

"Relength" Relen and RL #23

Closed twobob closed 4 years ago

twobob commented 5 years ago

In practical use commands that don't have a TLA (Two/three letter acronym in this case) are cumbersome. Going forward if all command could have a TLA that would be ideal. (With exception to things that possibly have no good shortname like "scan")

For those cases that are currently missing a TLA obviously considerations such as 'clarity of short-name'; 'avoiding accidental cognates' and of course the general lack of namespace are key.

Recognising that would it be better to just provide a push with my ideas or do you want to hash it out in comments on this issue first?

Many thanks.

twobob commented 5 years ago

{ "relength", Relength }, { "relen", Relength }, { "rl", Relength },

Cognates: No obvious, Clarity: Seems Legit, Namespace: R is currently pretty empty.

for example would be the thought process. Reasonable?

twobob commented 5 years ago

I also note that the order of "nicknames" is not 'fixed' in the code perhaps it could simply be ordered by length. { "il", Interleave }, { "interleave", Interleave }, { "ilev", InterleaveEvent }, { "interleaveevent", InterleaveEvent }, Currently break that convention for example, could make that a "thing" and tidy it up during my push.

carrierdown commented 4 years ago

Thanks, feel free to submit a PR for this :) I think 4-letter commands could make do without further shortening, but 3-4 letters is a good fit I think.

twobob commented 4 years ago

Willdo. are one letter abbreviations okay for special cases like MidiVelocity where "v" really makes the most sense and you type it all the time. I'll consider the justification anyway on this side and see what makes the most sense in use for now. Obviously it has a chance of misunderstanding the users part and also perhaps being a typo; but typing v.9 was very freeing :)

carrierdown commented 4 years ago

Feel free to experiment a bit here and we'll see what sticks. Using v is probably ok for the velocity/velo command :)

twobob commented 4 years ago

Handled in Pull #35