Last modified: 2013-09-23 01:37:55 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 T48333, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 46333 - Make new user login/account creation the only version after testing and MW namespace cleanup
Make new user login/account creation the only version after testing and MW na...
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
User login and signup (Other open bugs)
1.21.x
All All
: High normal (vote)
: ---
Assigned To: spage
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-03-19 19:03 UTC by Matthew Flaschen
Modified: 2013-09-23 01:37 UTC (History)
6 users (show)

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


Attachments
Screenshot of the old login page on Commons (73.92 KB, image/png)
2013-05-24 23:08 UTC, Tomasz W. Kozlowski
Details
Screenshot of the new login page on Commons (55.12 KB, image/png)
2013-05-24 23:08 UTC, Tomasz W. Kozlowski
Details

Description Matthew Flaschen 2013-03-19 19:03:41 UTC
The account creation (ACUX) and user login changes in E3Experiments have been removed to core.  However, they only apply when a useAgora flag is in the URL.  This allows testing and MW message cleanup where necessary.

When this is done, useAgora should be removed and Agora made the only version.
Comment 1 Andre Klapper 2013-04-23 10:25:36 UTC
Matthew: Is there any timeframe (or "Is this stable enough" / "Should any bug reports get fixed first and hence block this one which is about 'deployment'")?
Comment 2 Matthew Flaschen 2013-04-23 19:37:13 UTC
There may be some small blockers.  Currently, only the login version of this is in core.  We'll get create in (on its way at https://gerrit.wikimedia.org/r/#/c/57823/ ).  Then, after it's stable enough and wikis are happy with their customizations/help pages/messages, we can resolve this bug.
Comment 3 Matthew Flaschen 2013-04-23 19:37:51 UTC
Also, we renamed useAgora to useNew.
Comment 4 Steven Walling 2013-04-23 19:44:16 UTC
(In reply to comment #2)
> There may be some small blockers.  Currently, only the login version of this
> is
> in core.  We'll get create in (on its way at
> https://gerrit.wikimedia.org/r/#/c/57823/ ).  Then, after it's stable enough
> and wikis are happy with their customizations/help pages/messages, we can
> resolve this bug.

Yes, this is looking forward a bit, as a bug. We are going to give one week of community testing time after both account creation and login are soft-launched across the cluster, respond to feedback, and work on the necessary cleanup before launching as default.
Comment 5 Gerrit Notification Bot 2013-05-24 22:12:57 UTC
Related URL: https://gerrit.wikimedia.org/r/65340 (Gerrit Change I2f4cb4032871737f96f3d28a8fd974f1d8ba4ccc)
Comment 6 Tomasz W. Kozlowski 2013-05-24 22:25:02 UTC
Why did you decide/are you planning to remove it from core? I'd be perfectly happy with using &useNew=0 forever, as the new version looks absolutely crappy on my browser.
Comment 7 Steven Walling 2013-05-24 22:30:38 UTC
(In reply to comment #6)
> Why did you decide/are you planning to remove it from core? I'd be perfectly
> happy with using &useNew=0 forever, as the new version looks absolutely
> crappy
> on my browser.

What browser are you using? Please file any and all bugs you want for browser-specific flaws, and we'll do our best to address them. You'll need to be more specific and constructive than "crappy". 

Maintaing multiple sets of i18n messages and styles forever is not something we're going to do. When the use of the version gated behind a URL parameter was submitted, we specifically committed to not leaving two forms in one codebase forever. Reviewers rightfully pointed out it is a messy, hackish approach to a redesign, and we only did it so that critical bugs and i18n issues could be tested in production pre-launch as the default.
Comment 8 Tomasz W. Kozlowski 2013-05-24 23:08:14 UTC
Created attachment 12391 [details]
Screenshot of the old login page on Commons
Comment 9 Tomasz W. Kozlowski 2013-05-24 23:08:33 UTC
Created attachment 12392 [details]
Screenshot of the new login page on Commons
Comment 10 Tomasz W. Kozlowski 2013-05-24 23:14:17 UTC
(In reply to comment #7)

> What browser are you using? Please file any and all bugs you want for
> browser-specific flaws, and we'll do our best to address them. You'll need to
> be more specific and constructive than "crappy". 

I think I already described this under your blog post on the WMF blog, Steven, so I'll resort to those two screenshots, which clearly show that the new login screen is a regression, not an improvement.

The 'old' login screen is much more usable for me as a regular user: the test for various languages is bigger, it uses a black font instead of a gray one, and does not leave a lot of empty space for whatever's supposed to be on the right side of the new login screen.

A definite plus of the new login page is that it provides a link to a help page, which the old version does not have—though I'm sure it's a matter of editing some system message, and it shouldn't be too hard to fix.
Comment 11 Steven Walling 2013-05-24 23:17:32 UTC
(In reply to comment #10)
> (In reply to comment #7)
> 
> > What browser are you using? Please file any and all bugs you want for
> > browser-specific flaws, and we'll do our best to address them. You'll need to
> > be more specific and constructive than "crappy". 
> 
> I think I already described this under your blog post on the WMF blog,
> Steven,
> so I'll resort to those two screenshots, which clearly show that the new
> login
> screen is a regression, not an improvement.

I think it's hardly clear. I think you disagree with the design direction taken, but that's not the same thing as a regression. From looking at your screenshot, I can say that it's as-designed, and that there are no browser-specific bugs as far as can be told from the screenshot. 

If you have a beef with the design choices made, we can go through those one by one, but this bug probably isn't the place. 

> 
> The 'old' login screen is much more usable for me as a regular user: the test
> for various languages is bigger, it uses a black font instead of a gray one,
> and does not leave a lot of empty space for whatever's supposed to be on the
> right side of the new login screen.
> 
> A definite plus of the new login page is that it provides a link to a help
> page, which the old version does not have—though I'm sure it's a matter of
> editing some system message, and it shouldn't be too hard to fix.
Comment 12 Matthew Flaschen 2013-05-24 23:37:45 UTC
We do have an open bug for possibly making the text color darker (bug 47777).

Making the languages bigger is possible.  But I wonder if UniversalLanguageSelector could provide a better approach than a huge list (currently it doesn't seem to work logged out).
Comment 13 Steven Walling 2013-05-24 23:39:30 UTC
(In reply to comment #12)
> We do have an open bug for possibly making the text color darker (bug 47777).
> 
> Making the languages bigger is possible.  But I wonder if
> UniversalLanguageSelector could provide a better approach than a huge list
> (currently it doesn't seem to work logged out).

The current plan is for ULS to be enabled much more widely: http://blog.wikimedia.org/2013/05/22/getting-ready-for-uls-everywhere-2/
Comment 14 Gerrit Notification Bot 2013-05-28 21:39:39 UTC
Related URL: https://gerrit.wikimedia.org/r/65851 (Gerrit Change I47e1a93d69b2dd739071738a094ce82c34b46344)
Comment 15 Matthew Flaschen 2013-05-29 19:34:46 UTC
As part of this, we should consolidate the styles for these forms into resources/mediawiki.special.  Right now the new styles are there (if they're specific to login/create account), but there are old ones in skins/ (for example, the Monobook background color mentioned at https://bugzilla.wikimedia.org/show_bug.cgi?id=47698#c14).

We can still use skinStyles where appropriate.
Comment 16 Gerrit Notification Bot 2013-06-05 23:08:21 UTC
Related URL: https://gerrit.wikimedia.org/r/67182 (Gerrit Change I0c8115245ed93f15394ec97847fc97981c13306b)
Comment 17 Gerrit Notification Bot 2013-06-06 05:55:43 UTC
Related URL: https://gerrit.wikimedia.org/r/67233 (Gerrit Change I50f25583c3aa9a4a263fb40f50256f92c18ddb4d)
Comment 18 Gerrit Notification Bot 2013-08-27 01:53:59 UTC
Change 81174 had a related patch set uploaded by Spage:
Remove unused UseVForm settings variables.

https://gerrit.wikimedia.org/r/81174
Comment 19 Gerrit Notification Bot 2013-09-19 21:46:47 UTC
Change 81174 merged by jenkins-bot:
Remove unused UseVForm settings variables.

https://gerrit.wikimedia.org/r/81174
Comment 20 MZMcBride 2013-09-22 17:58:19 UTC
Is this bug now resolved/fixed?
Comment 21 Matthew Flaschen 2013-09-23 01:37:55 UTC
Yes, thanks.  it was fixed when https://gerrit.wikimedia.org/r/#/c/67233/ was merged.  The config patch removes dead code.

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


Navigation
Links