Last modified: 2014-01-21 22:23:33 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T58611, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 56611 - Email and Email confirmation options in Preferences should not use red boxes .
Email and Email confirmation options in Preferences should not use red boxes .
Status: PATCH_TO_REVIEW
Product: MediaWiki
Classification: Unclassified
Interface (Other open bugs)
1.23.0
All All
: Low minor with 1 vote (vote)
: ---
Assigned To: roopak
: easy
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-11-05 09:20 UTC by roopak
Modified: 2014-01-21 22:23 UTC (History)
9 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments
Screenshot of the email options in MWpreferences. (279.71 KB, image/png)
2013-11-05 09:20 UTC, roopak
Details
Changed the red boxed email and email confirmation options (850 bytes, patch)
2013-11-06 17:49 UTC, roopak
Details
Changed the red boxed email and email confirmation options (908 bytes, patch)
2013-11-06 18:11 UTC, roopak
Details
Changed the red boxed email and email confirmation options (829 bytes, patch)
2013-11-07 03:57 UTC, roopak
Details
Screenshot of proposed change (104.79 KB, image/png)
2014-01-18 12:59 UTC, roopak
Details

Description roopak 2013-11-05 09:20:35 UTC
Created attachment 13705 [details]
Screenshot of the email options in  MWpreferences.

Email and Email confirmation options(which are optionals) in MW preferences should not be given in red boxes which generally indicates error message.
Comment 1 Tony Thomas 2013-11-05 09:39:04 UTC
The Email conformation box is ok to be in red, but the email input box need not be.
Comment 2 Bartosz Dziewoński 2013-11-05 13:47:47 UTC
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.
Comment 3 roopak 2013-11-06 03:14:27 UTC
I plan to work on this bug. Kindly do assign me this bug.
Comment 4 roopak 2013-11-06 17:49:24 UTC
Created attachment 13714 [details]
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.
Comment 5 roopak 2013-11-06 18:11:55 UTC
Created attachment 13715 [details]
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.
Comment 6 Gerrit Notification Bot 2013-11-06 18:16:30 UTC
Change 93981 had a related patch set uploaded by Gerrit Patch Uploader:
Changed the red boxed email and email confirmation options

https://gerrit.wikimedia.org/r/93981
Comment 7 roopak 2013-11-07 03:57:26 UTC
Created attachment 13718 [details]
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.
Comment 8 Andre Klapper 2013-11-07 08:35:02 UTC
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.
Comment 9 Gerrit Notification Bot 2013-11-07 09:27:09 UTC
Change 94117 had a related patch set uploaded by Gerrit Patch Uploader:
Changed the red boxed email and email confirmation options

https://gerrit.wikimedia.org/r/94117
Comment 10 p858snake 2013-11-07 10:07:26 UTC
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.
Comment 11 Gerrit Notification Bot 2013-11-07 14:11:39 UTC
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.

https://gerrit.wikimedia.org/r/93981
Comment 12 Dereckson 2014-01-06 15:03:51 UTC
See also : https://gerrit.wikimedia.org/r/#/c/94117/
Comment 13 roopak 2014-01-16 13:52:42 UTC
p858snake: I have considered your suggestion and submitted my new patch.
Comment 14 Matthew Flaschen 2014-01-16 19:02:10 UTC
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.
Comment 15 Matthew Flaschen 2014-01-16 19:03:18 UTC
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.
Comment 16 Jared Zimmerman (WMF) 2014-01-16 20:22:25 UTC
Can someone upload a screenshot of the proposed change, rather than just the code changed.
Comment 17 roopak 2014-01-18 12:59:41 UTC
Created attachment 14338 [details]
Screenshot of proposed change

Screenshot of the proposed change that shows that only email not authenticated option should be highlighted red as an error.
Comment 18 Jared Zimmerman (WMF) 2014-01-21 22:23:06 UTC
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."
Comment 19 Jared Zimmerman (WMF) 2014-01-21 22:23:33 UTC
Small correction

"Your email address, $emailaddress is not yet confirmed. You will not receive
email notifications until you confirm this address."

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links