-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
revert subtitution of "---" for "None" #3549
Conversation
my motivation to have a clearly distinguishable state. Shapes are much quicker to recognise than text. I'd agree if you revert the change for Pref > Effects an EQ but please just leave it as it in the skin widgets. Same for the Sound API: Btw the english None is shorter than most translations that come to mind. |
IMO it is more important that the meaning is clear than it looks visually distinct. Why does it need to look so distinct? Why is it important that an empty effect slot stands out from other others? To me that is a little bit distracting. The problem with the Sound Hardware preferences is that there are so many comboboxes shown at once. The solution to that problem is a redesign |
so that the effect selectors fulfill their purpose: display the current status of loaded effects. Edit that was also the reason for me to put a blank effect label on top of the effect selectors in Tango. |
yes, we're touching that topic over and over again without someone starting a real design discussion (in the right moment). As I said, you feel free to change it back to None in the Effects and EQ preferences if it really bothers you. |
Okay, thank you for explaining.
Will do. |
How about using empty string instead of |
You can't really good select something empty in a combo box. If you have the dashes, your eyes have something to hook on.
An empty string is even worse in this context. |
An empty string looks odd in a combobox. It seems like it could be an error instead of a valid value. |
I consider |
Angle brackets look like XML. |
Anyway, can we all agree to go back to "None" for the EQ preferences, merge this, and move on? |
yup. |
//: Shown when no EQ is loaded | ||
const QString kNoEffectLoadedLabel = QObject::tr("None"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
recently I used Disambiguation string to add a hint for translators. Then I discovered //:
is similiar.
What's the difference for transifex translators?
Yes. I think this inconsistency does not make sense. IIRC it's possible to use rich-text in these comboboxes. Can we maybe make "None" in italics or something, and then use that everywhere? And maybe use braces, e.g. (None) ? |
The EQ preferences are in a different context from the skin widgets. I don't think there's a problem not having them consistent. |
The EQ preferences should at least be consistent with all other preferences comboboxes, e.g. the sound preferences. |
okay, so you'd like to have all preferecnes comboboxes consistent? or is it about skin effect widgets vs. Effect/EQ comboboxes? As explained above I find --- much better in the Sound preferences where one needs to grasp the overall config at a glance. @Be-ing Would it help to have all comboboxes consistent by adding the empty @Be-ing before you explain that in capitals, I'm aware such a refactoring was not your intention here ;) I could take over if you like.. |
I don't care enough about this to continue discussing it. |
Reverting the functional changes of #3485 and keeping the refactoring