Last modified: 2014-08-04 16:07:51 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 T39109, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 37109 - Sometimes, Learn > Upload .. headings are incorrectly centered
Sometimes, Learn > Upload .. headings are incorrectly centered
Status: VERIFIED FIXED
Product: MediaWiki extensions
Classification: Unclassified
UploadWizard (Other open bugs)
unspecified
All All
: Low trivial (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-05-25 02:00 UTC by Erik Moeller
Modified: 2014-08-04 16:07 UTC (History)
4 users (show)

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


Attachments

Description Erik Moeller 2012-05-25 02:00:32 UTC
This appears to be some kind of random load order issue; on some page loads, the headers are incorrectly centered. You can see it in this clip:

https://accounts.usertesting.com/ViewClip.aspx?file=7WK1sCsF5MGMmqHHvdW7BA%3d%3d
Comment 1 Mark Holmquist 2012-05-25 16:57:54 UTC
I think this may be a bug in the arrowSteps jQuery addon, but I have fixed it on our side of things: https://gerrit.wikimedia.org/r/8928
Comment 2 Erik Moeller 2012-05-25 17:07:02 UTC
The arrowSteps plug-in is "our side of things" too (it was developed by NeilK for Upload Wizard), it just lives in MediaWiki core.

https://gerrit.wikimedia.org/r/gitweb?p=mediawiki/core.git;a=blob;f=resources/jquery/jquery.arrowSteps.js;h=f9637545dc4fb49317917e8d95d8cf71edbbbf3b;hb=HEAD
Comment 3 Mark Holmquist 2012-05-25 17:13:00 UTC
I guess I meant "in this repository". I could try to figure out what's happening in the arrowSteps plugin, but I really don't see any modification of height in it....would you rather I dug deeper to find the height modification, or just let this fix stand?
Comment 4 Erik Moeller 2012-05-25 22:34:49 UTC
Dig a little deeper; the current fix looks pretty hackish.
Comment 5 Mark Holmquist 2012-05-25 23:10:31 UTC
I think it's better now! Patch still need review.
Comment 6 matanya 2012-06-26 11:21:51 UTC
This is no longaer an issue. closing.
Comment 7 Mark Holmquist 2012-08-24 18:56:28 UTC
Since we have closed this, and there don't appear to be any reopens coming down the pipeline, I'll abandon the patchset.

If this gets reopened, hopefully the patchset will still be there....
Comment 8 Erik Moeller 2012-08-24 20:11:12 UTC
*cough* Bugs usually don't fix themselves. This one is still quite easily reproducible (Chrome): Just reload https://commons.wikimedia.org/wiki/Special:UploadWizard a few times. Reopening.
Comment 9 Mark Holmquist 2012-08-24 20:14:17 UTC
Well, I can't reproduce that way. Maybe there's something else funky going on. Are you in debug mode? Anything else unusual?
Comment 10 Erik Moeller 2012-08-24 20:21:20 UTC
Nope. Chrome with a new test account to ensure it's not a pref or customization, after a few reloads:

http://imgur.com/tMtGy

Can't repro it in FF though.
Comment 11 Mark Holmquist 2012-08-24 21:33:32 UTC
OK, let's drill down to the heart of the matter: Which version of Chrome?

I'm on Chromium 18.0.1025.168 (Developer Build 134367 Linux) Ubuntu 12.04.

Maybe this happens over slower connections, but not faster ones? I'm on an ethernet connection, maybe you're on wireless? I'll test that now.
Comment 12 Mark Holmquist 2012-08-24 21:34:49 UTC
Wireless didn't change things.
Comment 13 Nischay Nahata 2013-02-24 16:12:15 UTC
Tried reloading several times on Chrome 24.0.1312.57 m but couldn't reproduce.
Comment 14 Nischay Nahata 2013-03-14 11:50:45 UTC
Tried again today and could easily reproduce, I also identified the problem but don't have a fix yet.

The bug:
var arrowWidth in the arrowSteps plugin is set to different values on reloads because it depends on background images. If the background images are already loaded the value is right, otherwise its wrong and we see the wrong padding.
Comment 15 Gerrit Notification Bot 2013-05-20 09:02:49 UTC
Related URL: https://gerrit.wikimedia.org/r/64556 (Gerrit Change Ide468a5259dd81bfbd1ca8117b091b4e9cfee7ce)
Comment 16 Nischay Nahata 2013-05-20 09:05:58 UTC
Trying to fix the plugin directly. Recent refreshes in latest Chrome shows less aberrant behavior.
Comment 17 Gerrit Notification Bot 2014-07-11 21:59:27 UTC
Change 64556 merged by jenkins-bot:
jquery.arrowSteps: Center arrowheads text

https://gerrit.wikimedia.org/r/64556
Comment 18 James Forrester 2014-07-11 23:31:02 UTC
(In reply to Nischay Nahata from comment #16)
> Trying to fix the plugin directly. Recent refreshes in latest Chrome shows
> less aberrant behavior.

Less aberrant or totally fixed?
Comment 19 Nischay Nahata 2014-07-21 20:20:58 UTC
Not working on this anymore. Would be great if someone else could verify and close this bug.
Comment 20 Andre Klapper 2014-07-22 09:31:45 UTC
(In reply to Nischay Nahata from comment #19)
> Not working on this anymore.

Resetting assignee very welcome in such cases. :)
Comment 21 James Forrester 2014-08-04 16:07:44 UTC
Confirmed that this is now fixed in production as of wmf14.

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


Navigation
Links