-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Rename "Song Editor" to "Sequencer" #121
Comments
👍 |
I like this idea. |
I think Song Editor better describes what it does. |
Please elaborate. I've wrote it before and I'll do it again, even better: Not every track is a song, however, every sequence can be a song, or not. I think that "Sequence" is much more universal than "Song". It's a common It's not just "Song Editor" it's much more! Maybe "Song Editor" could be named "Arrangement" and "B+B Editor" could be "Sequencer" + "Pattern Editor" (or even shorter: "Patterns" - as in 2014/1/24 Oskar Wallgren [email protected]
Tobiasz unfa -----BEGIN GEEK CODE BLOCK----- |
Maybe your right, but equencer I associate with hardware sequencers and something that controls your synths and sounds. Control messages. When you add an actual audiotrack on top of the sequences as you can in LMMS, then sequencer isn't as fitting. 'Project Editor' maybe? |
I guess we can look at some other establihsed software and find out what 2014/1/24 Oskar Wallgren [email protected]
Tobiasz unfa -----BEGIN GEEK CODE BLOCK----- |
LMMS is (by deffiniton) a software sequencer, right? I can't think if a more appropriate word in this case, although I don't have the hardware "sequencing" history/experience to influence my understanding. |
Project Editor (song), Pattern Editor (bb), Automation Editor, Piano Roll Editor. Possibly drop the "Editor" for the window titles. All of the above are technically "sequencers" in my opion. The big concept to illustrate is at what depth in the project hierarchy do they operate. And the song editor is the root. |
Is this still a thing? If so, could someone point out how this change would affect LMMS holistically? I guess this would not just be a change to the window label but a fundamental terminology change. |
Yeah, it's still under (dormant) discussion. Most of the devs agreed for better names and this topic still remains relevant. :) |
Oh, just noticed that this discussion is elaborated at #120. I'll check there. |
Yeah, the rename issues should probably be organized and consolidated since nomanclature in general tends to affect a lot of the code, and influence one another. |
I'd like to resurrect this thread. Most were in favor of changing it back in January 2014 when it was originally proposed. This is a large refactoring task. |
@tresf Can this be as simple as a translation file edit? (I believe we can have an english translation file) |
Well, it could... We could make the class called |
@tresf Point taken - translation files might not be the best way to tackle this - indeed, I can think of multiple reasons why they aren't currently suited for the job. We should come at it from an angle where it's easy to rename it again in the future though, right? I mean, it's pretty much impossible to make all of the sources trivially renamable, but from the UI perspective, it should amount to a single string constant being changed - right? |
I think this is one of those all-or-nothing scenarios where we just need to pick a new permanent name (or alternatively decide to keep what we have). I don't think we'll ever have easy class renaming. :) From a UI perspective, you're right, the front end could be changed without changing the underlying class names, but I just think it would be harder for developers to discuss and would make the codebase confusing if they didn't match... at least in english. |
@tresf hmm. I haven't dug into the code too far yet, but if we were to reduce this to one string constant per editor, I would suspect it allows for more code reuse. For example, we could add a virtual method, |
As part of a pruning effort, this |
The titles says it all.
Let's have shorter and more universal names,
The text was updated successfully, but these errors were encountered: