carrierdown / mutateful

Add-on for Ableton Live that enables live coding functionality fully integrated into Live's session view.

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

"Relength" Relen and RL

twobob opened this issue · comments

commented

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.

commented

{ "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?

commented

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.

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.

commented

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 :)

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

commented

Handled in Pull #35