Last modified: 2014-02-12 23:53:31 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 T38651, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 36651 - MobileFrontend scrubs co-ordinate templates
MobileFrontend scrubs co-ordinate templates
Status: RESOLVED FIXED
Product: MobileFrontend
Classification: Unclassified
stable (Other open bugs)
unspecified
All All
: Normal normal
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-05-08 17:20 UTC by Yuvi Panda
Modified: 2014-02-12 23:53 UTC (History)
10 users (show)

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


Attachments

Description Yuvi Panda 2012-05-08 17:20:35 UTC
http://en.wikipedia.org/wiki/Great_Barr_Hall has its' co-ordinates scrubbed. Should be output at a relavant position.
Comment 1 Jon 2012-05-10 15:41:30 UTC
This seems to be scrubbed on purpose.. I'm not sure if there is a good reason.
The only reason I suspect is that due to the introduction of sections the coordinates appear in the top right of the section it is contained in rather than in the top right of the page.

The commit message 4bc466c1 isn't very helpful
"Started extracting wikitext-manipulating code into a separate, isolated class"
Comment 2 Max Semenik 2012-05-10 17:56:35 UTC
That commit simply moved code around, the selector in question was there since times forgotten, even Patrick doesn't quite remeber why. We have to find a way to position them though.
Comment 3 Jon 2012-05-10 18:00:42 UTC
We could simply leave it in the markup, hide it with css and use javascript to move it into a more appropriate place. Obviously this is inaccessible for users with javascript disabled (although not such an issue with the app and actually won't make any presentational difference to the current situation we find ourselves in)

Alternatively we could parse it (not sure if this a good idea) and reconstruct it at the top of the page.
Comment 4 Max Semenik 2012-05-10 18:07:01 UTC
They're normally just positioned with CSS, so the only problem with it on the mobile site is to make them not overlap the UI.
Comment 5 Jon 2012-05-10 18:13:14 UTC
In the example Yuvi gives they appear for some reason in the last section when not scrubbed on the mobile site.
Comment 6 Max Semenik 2012-05-10 18:17:20 UTC
So they're moved to the top of the desktop site page with position: absolute; top: 0; right: 0;
Comment 7 Jon 2012-05-10 22:51:35 UTC
Yes... but when in a section on mobile site they are absolute to the section they are contained in ...
Comment 8 Jon 2012-05-15 11:15:04 UTC
Fixed in https://gerrit.wikimedia.org/r/7720
Note the commit message that this may need a follow up as I notice many geographical coordinates are placed at the bottom of the page instead of at the top. I'm not sure what the correct behaviour is here (whether they should be in the last section or appear outside the sections) but this should be addressed in a separate bug.

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


Navigation
Links