| • bzimport | |
| Nov 5 2013, 9:20 AM |
| F3069597: Screen Shot 2015-12-11 at 17.12.50.png | |
| Dec 11 2015, 4:23 PM |
| F12735: rpk.png | |
| Oct 27 2015, 9:39 PM |
| F12737: Screenshot_from_2014-01-18_17:56:00.png | |
| Nov 22 2014, 2:40 AM |
| F12736: patchnewcss.diff | |
| Nov 22 2014, 2:40 AM |
Author: mailstorpk
Even when it's unconfirmed, the user's email address in preferences (which is optional) should not be given in a red box, which generally indicates an error message. The information about your e-mail address not being confirmed should probably be changed to display as a warning rather than error
Screenshot of the email options in preferences:
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Special:Preferences Remove red box in email and changed confirm error to warning | mediawiki/core | master | +13 -14 |
The classes which set those colors are .mw-email-not-authenticated and
.mw-email-none.
This will probably require some simple, but careful adjustments in their definitions in mediawiki.special.preferences.css and/or where they're added to the page in Preferences.php.
mailstorpk wrote:
Changed the red boxed email and email confirmation options
In MW preferences, email and email confirmation options were given in red boxes indicating error message.In this attachment the red color is removed and the outline color is also changed from red.
attachment patchcss.diff ignored as obsolete
mailstorpk wrote:
Changed the red boxed email and email confirmation options
In MW preferences, email and email confirmation options were given in red boxes
indicating error message.In this attachment the red color is removed and the
outline color is also changed from red.
//The last patch was incomplete.
attachment patchnew.diff ignored as obsolete
Change 93981 had a related patch set uploaded by Gerrit Patch Uploader:
Changed the red boxed email and email confirmation options
mailstorpk wrote:
Changed the red boxed email and email confirmation options
In MW preferences, email and email confirmation options were given in red boxes
indicating error message.In this attachment the red color is removed and the
outline color is also changed from red.
Attached:
Please upload updated patches to Gerrit if possible. Please see http://www.mediawiki.org/wiki/Gerrit/Tutorial for how to do so.
We do not use Bugzilla to review patches at all.
Change 94117 had a related patch set uploaded by Gerrit Patch Uploader:
Changed the red boxed email and email confirmation options
The second box (Email conformation section) is display when its half done (email entered but not confirmed) so it should probably still be countered as a error and displayed as such.
Change 93981 abandoned by Bartosz Dziewoński:
Changed the red boxed email and email confirmation options
Reason:
Superseded by https://gerrit.wikimedia.org/r/94117 . You should submit new patchsets to existing changesets instead of new ones.
mailstorpk wrote:
p858snake: I have considered your suggestion and submitted my new patch.
I'm not really convinced red (actually pinkish) is bad here.
This is only for non-authenticated emails. We *want* to catch their attention. They lose important functionality, including password reset, if they don't confirm their email.
CCing Jared for UX feedback.
Actually, if I'm reading the code right, you don't need to confirm your email for password reset.
However, if it's not confirmed, you might have mistyped it, in which case you still effectively won't be able to reset.
Can someone upload a screenshot of the proposed change, rather than just the code changed.
mailstorpk wrote:
Screenshot of proposed change
Screenshot of the proposed change that shows that only email not authenticated option should be highlighted red as an error.
Attached:
I'm fine with this change, but the one thing we lose is the correlation between the "error" and the email associated with it. Can we change the text to
"Your email address, $emailaddress is not yet confirmed. You will not receive email notification until you confirm this address."
Small correction
"Your email address, $emailaddress is not yet confirmed. You will not receive
email notifications until you confirm this address."
@matmarex: Looks like a student could could pick up https://gerrit.wikimedia.org/r/#/c/94117/ and improve it?
@matmarex: Reading through the comments of this task, this task should probably first receive an updated description explaining what exactly is expected from a contributor, before making this available in GCI: Whether both boxes should not be red (and what else instead), whether text changes are also expected (see T58611#643768).
I think it'll be easier to start from scratch. That patch makes a few questionable changes with no explanation (which is why it wasn't accepted). It can still be useful for reference, though.
All that's required is to make the unconfirmed email address not appear in a red box. The other should be changed to display as a warning rather than error, but it's fine either way. Jared's suggestion from T58611#643768 was incorrect (this controls more than just notifications), I don't think any localisation changes are needed.
Is it a good idea to change email confirmation field to warning rather than error using yellow color like on the screenshot? Should I use some specific colors (in hex) for that?
Yes, it seems appropriate, there is a serious consequence involved here: the lack of possibility of password reminder, and so the risk to lose account, so yes, that could be a warning.
It would probably be good to use the same colors as for .warningbox: https://phabricator.wikimedia.org/diffusion/MW/browse/master/resources/src/mediawiki.legacy/shared.css;d54591216efd17238fa8df7f6d85f4195b89d683$442 (or to just use that class).
Change 258917 had a related patch set uploaded (by Neonowy):
Special:Preferences Remove red box in email and changed confirm error to warning
Change 258917 merged by jenkins-bot:
Special:Preferences Remove red box in email and changed confirm error to warning