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

revert subtitution of "---" for "None" #3549

Closed
wants to merge 1 commit into from

Conversation

Be-ing
Copy link
Contributor

@Be-ing Be-ing commented Jan 11, 2021

Reverting the functional changes of #3485 and keeping the refactoring

@Be-ing Be-ing added this to the 2.3.0 milestone Jan 11, 2021
@ronso0
Copy link
Member

ronso0 commented Jan 11, 2021

my motivation to have a clearly distinguishable state. Shapes are much quicker to recognise than text.
None may be at the end or top of the affected lists which may make it easy to find but for closed comboboxes this is not relevant.
--- is a very simple shape and you don't have to read None to know an effect slot is empty.
In the collapsed effect units it's so much clearer now.
A compromise would be to have None in the expanded list, and --- when they're closed. But the effort of refactoring the effect selector or any Pref combobox wasn't worth it.

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:
--- vs None in a list that also has similiar short items like Alsa
Same for the item in the sound in/output configuration below
--- vs None so that it's much easier to spot if all devices are set, respectively that unwanted in/outputs are not accidentally set.

Btw the english None is shorter than most translations that come to mind.

@Be-ing
Copy link
Contributor Author

Be-ing commented Jan 11, 2021

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

@ronso0
Copy link
Member

ronso0 commented Jan 11, 2021

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?

so that the effect selectors fulfill their purpose: display the current status of loaded effects.
the reason for me using --- was that I engaged effects without noticing at a glance they were empty (due to unnoticed Shift+Meta knob turn earlier, another UX issue IMO, disabled now).
With --- that's unlikely to happen again.

Edit that was also the reason for me to put a blank effect label on top of the effect selectors in Tango.

@ronso0
Copy link
Member

ronso0 commented Jan 11, 2021

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

yes, we're touching that topic over and over again without someone starting a real design discussion (in the right moment).
this was a small step to improve UX.

As I said, you feel free to change it back to None in the Effects and EQ preferences if it really bothers you.

@Be-ing
Copy link
Contributor Author

Be-ing commented Jan 11, 2021

the reason for me using --- was that I engaged effects without noticing at a glance they were empty (due to unnoticed Shift+Meta knob turn earlier, another UX issue IMO, disabled now).

Okay, thank you for explaining.

As I said, you feel free to change it back to None in the Effects and EQ preferences if it really bothers you.

Will do.

@Holzhaus
Copy link
Member

How about using empty string instead of ---? It's both visually distinct and the meaning is clear.

@daschuer
Copy link
Member

You can't really good select something empty in a combo box. If you have the dashes, your eyes have something to hook on.
I think the dashes are some kind of industrial standard for "no value". The value is "dashed out".
At least in the embedded display context, dashed lines are used for indicating that the display is OK, but that there is just no value.

I don't like this change. This feels like a regression. What was ever the problem with "None"? What was the motivation for changing this? Have you done any usability tests comparing this with "None"? I cannot recall any problems ever caused by the string "None". Who would know what --- means in this context? This looks bizarre IMO.

An empty string is even worse in this context.

@Be-ing
Copy link
Contributor Author

Be-ing commented Jan 11, 2021

An empty string looks odd in a combobox. It seems like it could be an error instead of a valid value.

@uklotzde
Copy link
Contributor

uklotzde commented Jan 11, 2021

I consider --- a recognizable placeholder. An alternative could be <None>. Angle brackets are often used for such placeholders and special values, e.g. <Keep>, <Clear>, or <Remove> when batch editing fields with different values from multiple entities.

@Be-ing
Copy link
Contributor Author

Be-ing commented Jan 11, 2021

Angle brackets look like XML.

@Be-ing
Copy link
Contributor Author

Be-ing commented Jan 11, 2021

Anyway, can we all agree to go back to "None" for the EQ preferences, merge this, and move on?

@ronso0
Copy link
Member

ronso0 commented Jan 11, 2021

Anyway, can we all agree to go back to "None" for the EQ preferences, merge this, and move on?

yup.
any objections?

Comment on lines +26 to +27
//: Shown when no EQ is loaded
const QString kNoEffectLoadedLabel = QObject::tr("None");
Copy link
Member

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?

@Holzhaus
Copy link
Member

Anyway, can we all agree to go back to "None" for the EQ preferences, merge this, and move on?

yup.
any objections?

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

@Be-ing
Copy link
Contributor Author

Be-ing commented Jan 11, 2021

The EQ preferences are in a different context from the skin widgets. I don't think there's a problem not having them consistent.

@Holzhaus
Copy link
Member

The EQ preferences should at least be consistent with all other preferences comboboxes, e.g. the sound preferences.

@ronso0
Copy link
Member

ronso0 commented Jan 11, 2021

Yes. I think this inconsistency does not make sense.

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.
(None) wouldn't help much IMO. For my previous PR I really couldn't come up with any other graphical placeholder than ---

@Be-ing Would it help to have all comboboxes consistent by adding the empty --- item at the top? Currently the position differs.
Sound API > top
Sound devices > top
EQ effects > bottom
effect selector widget > bottom (list may be long and bottom items may be hidden)

@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..

@Be-ing
Copy link
Contributor Author

Be-ing commented Feb 24, 2021

I don't care enough about this to continue discussing it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants