| • Tnegrin | |
| Jan 2 2017, 4:45 PM |
| F5222061: fullsizeoutput_b82.jpeg | |
| Jan 3 2017, 1:02 PM |
| F5217790: valley-of-kings.jpg | |
| Jan 2 2017, 4:45 PM |
From a wikipedia weekly thread:
Who here is interested in rendering problems for Wikipedia on Android? From [[Valley of the Kings]]
This is caused because the labels are relatively positioned using pixel offset, but the image itself is being scaled down, from the original size that the author included in the Desktop version of Wikipedia and related to the usage of the template:
https://en.wikipedia.org/wiki/Template:Image_label_begin
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Make noresize non-specific to thumbnails and cleanup | mediawiki/extensions/MobileFrontend | master | +14 -14 |
[Comments by @TheDJ on the original thread]
1: the screenshots of mobile web show the problem where responsive images do not work because of lazy loaded images.. (Not sure how that is still not fixed, i reported it almost the same day that lazy loaded images went live...). The image is therefor allowed to be wider than the viewport.
2: The original issue that Chris shows in the app. This does NOT use lazy loaded images, and thus the responsive image is scaled down to fit the viewport. The template used (Template:Image label begin) to built this map however uses fixed pixels to position the links, and of course these do not scale along with the image. Template:Location map+ uses percentages and is less sensitive to this problem, although the text because relatively larger.
Solutions non-existant, though long term should probably be sought in something like the Kartographer extension (even though that itself seems to have some mobile view issues atm).
This seems to be https://en.m.wikipedia.org/wiki/Valley_of_the_Kings#Tomb_development (steps to reproduce are welcome in bug reports).
This seems to be both about Android-app-Bugs (some mobile app) or MobileFrontend (some browser)?
What is hoped to be achieved here by making this a Reading bug? Is it purely to track?
As @TheDJ states there's not much that can be done here, apart from rethink the template from scratch / make it behave differently on mobile / adopting SVG or Kartographer for this purpose.
For situations where responsive images are not used (apparently in the mobile apps, which is an inconsistency that makes me wonder what other CSS the apps are missing), I could add to the template: max-width: 100%;overflow-x: scroll; to basically make this behave similar to what we usually do with wide tables. (now done ).
But this does not fix the responsive images case, and I'm also not aware of a good way to deal with that...
Further improved this by using noresize and adding the thumb class to this specific graph...
But i'm not sure why this noresize class would be limited to thumbs. It's not as if responsive images are limited to thumbs either are they ? Should we just remove the thumb class from that particular selector ?
Change 334754 had a related patch set uploaded (by TheDJ):
Make noresize non-specific to thumbnails and cleanup
Change 334754 merged by jenkins-bot:
Make noresize non-specific to thumbnails and cleanup
