We can do this either by center-aligning "Select All" checkbox or by left-aligning the ones below. Either works.
Also remove the "Make this setting global" from where it is right now (it is very easy to miss) and change it to -
• Niharika | |
Feb 27 2018, 1:39 AM |
F14065820: Checkbox issues.m4v | |
Feb 27 2018, 9:56 PM |
F14047624: Screenshot-2018-2-27 Global preferences - Dev Wiki wiki1(2).png | |
Feb 27 2018, 3:08 AM |
F14047500: Screenshot-2018-2-27 Global preferences - Dev Wiki wiki1.png | |
Feb 27 2018, 3:08 AM |
F14046345: Screen Shot 2018-02-26 at 5.30.48 PM.png | |
Feb 27 2018, 1:39 AM |
F14046366: Screen Shot 2018-02-26 at 5.37.40 PM.png | |
Feb 27 2018, 1:39 AM |
We can do this either by center-aligning "Select All" checkbox or by left-aligning the ones below. Either works.
Also remove the "Make this setting global" from where it is right now (it is very easy to miss) and change it to -
Subject | Repo | Branch | Lines +/- | |
---|---|---|---|---|
Left align checkboxes and improve select-all message(s) | mediawiki/extensions/GlobalPreferences | master | +29 -27 |
Status | Subtype | Assigned | Task | ||
---|---|---|---|---|---|
Resolved | • Niharika | T139145 global disabling of compact language list | |||
Resolved | None | T16950 [Epic] Support global preferences on Wikimedia wikis | |||
Resolved | Samwilson | T188358 Design: Make the "Select All" checkbox align with the ones that come below it and fix the labelling |
The other thing to remember for the column heading there is that the select-all box is not visible to non-JS users. So for them, if there's no "Make this setting global" label at the top, they'll just get a column of checkboxes with nothing other than hover-text to tell them what they're for.
Perhaps we could keep the "Make this setting global" for non-JS users, and replace it with the checkbox with JS. Sound okay?
Change 414892 had a related patch set uploaded (by Samwilson; owner: Samwilson):
[mediawiki/extensions/GlobalPreferences@master] Left align checkboxes and improve select-all message(s)
I'm minorly concerned that moving the column of 'make this global' checkboxes further away from the actual preferences, but am willing to go either route.
The "Make this setting global" label doesn't just apply to 'select all' but the entire column. Combining it with the 'select all' weakens the label, in my opinion. Here's a maybe-good maybe-terrible idea: Could we make the "Select all" checkbox a link instead, that performs the same check all/none action? And it could only appear for JS users?
@TBolliger Hmm, I'm not sure about the link idea. The idea for "select all checkbox checking everything below" is a commonly seen web design pattern. But for links, the intuition is that the link will open up something new/go to a new page.
The reason for combining the "Make this setting global" and "select all" is that the former was really easy to miss. But making them bold, combined and properly aligned should fix that. There are a couple of ideas in T188424: Improve the UI of Global Preferences (aka "fix two vertical rows of checkboxes") too.
Since Sam has already made the changes, I think we should proceed with your initial suggestion (as long as we make it bold!)
We can think of ways to address the columns in the other ticket.
Hmm, I pulled up the patch on commtech wiki and I see the flash of non-JS version before the page fully loads (see recording below). I think that's problematic.
Yeah, that's not great. We should fix this (or come up with a different solution) before release.
Does the non-JS label have to be on two lines? If it were just one one, and positioned similarly to the line that replaces it, the FOUC would at least not result in vertical shifting of the whole form below.
Keeping in mind other skins as well of course, who might style things quite differently.
Or could the two messages be more similar, so the change from
Make this setting global
to
[ ] Select all options below to be global
is less pronounced?
Let's do away with "Make this setting global" and instead have "Select options below to be global" for both JS and non-JS versions. It'll appear with the checkbox for the JS version. I think it's fairly obvious that checking that checkbox will select all and it's something people can click and get the hang of in a minute.
Good idea. Done. Also minimised the FOUC with margins etc. (and found T188968).
https://gerrit.wikimedia.org/r/#/c/414892/ is ready I think.
Change 414892 merged by jenkins-bot:
[mediawiki/extensions/GlobalPreferences@master] Left align checkboxes and improve select-all message(s)