When signing up for an account, user register form, the time zone selection reads:
"Your time zone setting will be automatically detected if possible. Confirm the selection and click save."
This could cause a possible confusion as there is no "Save" button, only the "Create new account". I would suggest changing the description to:
"Your time zone setting will be automatically detected if possible. Please confirm the selection."
Beta phase evaluation
| Issue category | Bug |
|---|---|
| Unfrozen changes | Unfrozen because it only changes help text in the UI |
| Prioritized changes | The main goal of this issue is usability |
| Comment | File | Size | Author |
|---|---|---|---|
| #20 | timezone-1055150-with.png | 21.07 KB | rootwork |
| #20 | timezone-1055150-without.png | 29.56 KB | rootwork |
| #18 | 1055150-18-user-time-zone.patch | 810 bytes | devin carlson |
| #12 | 1055150-user-time-zone.patch | 767 bytes | yoroy |
| #10 | 1055150-timezone-info-account.png | 75.93 KB | yoroy |
Comments
Comment #1
devin carlson commentedSounds like a small but useful change to me.
Moving to 8.x-dev. I've attached a patch against Drupal 8 that makes the same change.
Comment #2
brianV commented+1 for this text change as well. Rerolling for HEAD
Comment #3
brianV commentedAnd actually attaching the patch this time...
Comment #4
rootworkSurprised this never went anywhere; I just ran into this and it seems needlessly confusing (also given all the accessibility/mobile work, we all know that not everyone is "clicking" things anyway). Personally this is a UX/documentation bug to me.
Rerolled against 8.x HEAD. Hope we can backport this to 7, too.
Comment #5
rootworkHere's the interdiff just for kicks. Looks like it's just changes in line numbers.
Comment #6
jhedstrom+1 for this change. Patch needs a reroll though.
Comment #7
devin carlson commentedA reroll of #4.
Comment #8
jhedstromThis change makes the help text more clear than it currently stands. This is allowed under beta phase since string changes are unfrozen, and it improves usability.
Comment #9
rootworkSo glad this is finally going to get in! Such a small, useful change.
Comment #10
yoroy commentedNice catch, but: we try not to use “Please”, see #604342: Interface text.
Additionally, I wonder wether we need that string at all, why even explain that the system made a best guess. Seems to me it’s adds more UI noise than useful information. Here’s what it looks like without, in Bartik:
Comment #11
rootworkI like the idea of just dropping it entirely, actually. It reads just fine.
What do others think?
Comment #12
yoroy commentedIs simply removing that line enough?
Comment #13
devin carlson commentedAfter reviewing the screenshot from #10, I agree that simply removing the description makes the most sense.
Comment #14
jhedstrom+1 for completely removing!
Comment #15
alexpottCommitted acd8af4 and pushed to 8.0.x. Thanks!
Thanks for adding the beta evaluation for to the issue summary.
Comment #17
David_Rothstein commentedThis would make sense to backport, right?
Comment #18
devin carlson commentedBackport of #12.
Comment #19
dcam commentedThis is a simple change. It eliminates the offending description. RTBC.
Comment #20
rootworkConfirmed the patch from #18 still applies to 7.x HEAD and makes the changes (once regional settings are set to allow user-set time zones).
Without patch:

With patch:

Comment #21
David_Rothstein commentedCommitted to 7.x - thanks!