-
Notifications
You must be signed in to change notification settings - Fork 999
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
[23.0] Remove "required" indicator from tool interface Radio select elements #15617
[23.0] Remove "required" indicator from tool interface Radio select elements #15617
Conversation
- Remove the * required indicator from radio `FormSelect`s
I don't think that was the conclusion from #15368 ? I think it makes sense to be consistent here and label these as required. Yes, in Galaxy radio buttons are only valid for required, non-multiple parameters (why that is I don't know, seems like an arbitrary restriction), but that isn't true for other web forms on other platforms that users may be familiar with. At the minimum the indicator helps answer the question "do I really need to choose something here or is there a way to unselect something here". Also the |
@mvdbeek |
It's not a galaxy thing. Radio buttons are defined this way, ie that they allow to select exactly one option (hence they can't be optional or multiple): And let me just link this one #11797 |
do you mean the select parameter of a conditional? They can't be optional by the linter galaxy/lib/galaxy/tool_util/linters/inputs.py Line 507 in 3841b31
|
That doesn't mean much, and there's IMHO no reason for that. You're going to take action in the |
Would be cool if this would already work. It was actually one of the points here: #13931 (comment). But the question of conditionals seems a bit offtopic. The main question is if radio selects can be optional. |
I believe most UX design guides recommend radio buttons should not be optional, e.g. https://uxplanet.org/radio-buttons-checkboxes-toggle-switches-and-dropdown-lists-design-tips-for-using-selection-d120a1e323c5 and https://www.nngroup.com/articles/radio-buttons-default-selection/ |
We all seem to be in agreement, so why don't we close this ? |
I have removed the * required indicator from radio
FormSelect
s, like the one below. Fixes #15368There is ambiguity on whether the same should be done for some drop-down fields that have a "blank" default value, as discussed here: #15368 (comment)
How to test the changes?
(Select all options that apply)
License