Problem/Motivation
In User interface its recommended in Drupal not to use "Sorry" in the error message
Reference:
https://www.drupal.org/ui-standards -> Interface text ->Style -> bullet point number 5. (Attached the screen short)
Also the description says that "These guidelines are being applied in Drupal core." but its not.
Proposed resolution
Can we consider rephrasing the text something like. "Username or password Unrecognized. Have you forgotten your password?" For better user Interface.
User interface changes
Rephrase the text for better User interface and implement recommendation in core.
Beta Evaluation
Markup, which most of this is, is "unfrozen". This small amount of user-interface text is highly-visible in some cases, but the risk of patching this now is minimal.
| Comment | File | Size | Author |
|---|---|---|---|
| #33 | d7-sorry_in_user_facing-2576037-23-33.patch | 9.49 KB | sdstyles |
| #23 | interdiff-17-23.txt | 1.64 KB | sdstyles |
| #23 | sorry_in_user_facing-2576037-23.patch | 12.42 KB | sdstyles |
| #17 | interdiff-2576037-14-17.txt | 1.49 KB | unqunq |
| #17 | sorry_in_user_facing-2576037-17.patch | 12.4 KB | unqunq |
Comments
Comment #2
devaraj johnson commentedComment #3
cilefen commented"Sorry" is used elsewhere in user-facing error messages, so I changed the component to "base system".
Drupal 8 is the same thing, so I have moved this to the Drupal 8 queue according to the backport policy.
Comment #4
er.manojsharma commentedComment #5
er.manojsharma commentedPlease review my patch. Do we have to remove
Sorrykeyword from all files likeSorry, too many failed login attempts from your IP address.. etc.Comment #6
er.manojsharma commentedattached the patch file
Comment #10
er.manojsharma commentedPlease review updated patch
Comment #12
cilefen commentedComment #13
unqunqI found a few more and removed the text. I left unmodified a few more because I am not sure if I should change them or not.
Comment #14
martins.kajins commentedUpdated #13 patch with two more changes
Comment #15
Bojhan commentedCool! At the time we avoided the "front-facing" parts but I am happy to fix this now.
Comment #16
cilefen commentedThere are two more in the quickedit module's JavaScript that should have this change. In those cases "Sorry!" should perhaps be "Error!".
Comment #17
unqunqAdded the last two from quickedit module.
Comment #18
cilefen commentedComment #19
wim leersThis should then become something like .
Comment #20
Bojhan commentedOk, lets do that.
Comment #23
sdstyles commentedChanged according #19
Comment #24
cilefen commented#23 removes all user-facing "sorry"s. I read over the patch and found no problems with it. I manually tested the bad password error and it looks fine. Could someone else give the patch a thorough read and mark this RTBC?
Comment #25
Bojhan commentedComment #28
webchickAs a Canadian, I am only committing this under extreme duress.
:P
Committed and pushed to 8.0.x. Sniff. ;)
Back to 7.x.
Comment #30
Bojhan commentedHah! duress, fascinating :) Learning new words every day.
Comment #31
cilefen commented@Bojhan "Duress" is fun, but "sorry" is the hardest word.
Comment #32
nod_Comment #33
sdstyles commentedPorted patch to Drupal 7.
Comment #34
David_Rothstein commentedI'm not sure how much of this we can backport given https://www.drupal.org/node/1527558 - at least we'd probably want to leave out end-user-facing strings and error messages.
Comment #35
cilefen commentedThat is pretty much all this issue is, so is this a "won't fix"?
Comment #36
Bojhan commentedYup, pretty much.
Comment #37
David_Rothstein commentedYeah, guess so. The IP address "has been banned" message isn't even translated so technically that could certainly be changed, but not sure anyone really cares much about that one.