Last modified: 2013-07-20 22:39:29 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 T53595, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 51595 - returnTo link layout wrong if no JavaScript
returnTo link layout wrong if no JavaScript
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
GettingStarted (Other open bugs)
master
All All
: Low minor (vote)
: ---
Assigned To: spage
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-07-18 04:29 UTC by spage
Modified: 2013-07-20 22:39 UTC (History)
4 users (show)

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


Attachments
detail of Getting Started page displayed after account creation on en-beta (8.74 KB, image/png)
2013-07-18 04:29 UTC, spage
Details

Description spage 2013-07-18 04:29:49 UTC
Created attachment 12875 [details]
detail of Getting Started page displayed after account creation on en-beta

If you disable JavaScript and create an account on a wiki that has Extension:GettingStarted, the returnto link that appears on the bottom of the "Welcome, $Username!" getting started page appears strangely (see attachment). Instead of "Return to _Page title_." the link appears out of sequence below and to the left of "Return to.".

This happens because Special:GS loads CSS (resources/ext.gettingstarted.css) that styles '#mw-returnto a' as "display: block, float: left". JavaScript (resources/ext.gettingstarted.test.accountCreation.js) removes the "Return to " and "." text surrounding the link, avoiding the bug.  I think '#mw-returnto a' CSS is from when the returnto was an Agora-style button, and is obsolete.

The introduction of CentralAuth/SUL2's silent redirect has disabled Special:GS on account creation, and the way GettingStarted works is likely to change, hence minor severity and priority.
Comment 1 Gerrit Notification Bot 2013-07-18 18:35:02 UTC
Change 74414 had a related patch set uploaded by Spage:
Remove styling from returnto link

https://gerrit.wikimedia.org/r/74414
Comment 2 Gerrit Notification Bot 2013-07-18 19:09:23 UTC
Change 74414 merged by jenkins-bot:
Remove styling from returnto link

https://gerrit.wikimedia.org/r/74414
Comment 3 Steven Walling 2013-07-18 19:18:10 UTC
Testing patch 74414 on labs looks okay with JS and removes redundant code, but this _doesn't_ fix the unnecessary line break for users without JS.
Comment 4 spage 2013-07-20 08:05:43 UTC
(In reply to comment #3)
> this _doesn't_ fix the unnecessary line break for users without JS.

Can you show me or take a screen shot?  In Firefox on en.wikipedia.beta.wmflabs.org I see a clean "Return to Main Page" on one line.
Comment 5 Steven Walling 2013-07-20 22:39:29 UTC
(In reply to comment #4)
> (In reply to comment #3)
> > this _doesn't_ fix the unnecessary line break for users without JS.
> 
> Can you show me or take a screen shot?  In Firefox on
> en.wikipedia.beta.wmflabs.org I see a clean "Return to Main Page" on one
> line.

I was testing on w2 where the SUL2 stuff was as well. I guess it didn't have the patch when I tested, because Beta Labs looks okay in Chrome and FF.

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


Navigation
Links