Compared to Vector, Monobook, the fallback and API skins and most other skins, Mobile has the footer in a different order.
On desktop:
On mobile:
It should swap data['reporttime'] and data['bottomscripts']. Or perhaps use the printTrail() from BaseTemplate class which does this automatically (instead of hardcoding).
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Use default order of footer elements | mediawiki/extensions/MobileFrontend | master | +1 -4 |
@Krinkle Does this have any practical effects? Is it skewing any performance metrics?
Just trying to clarify the urgency of this :)
@Jhernandez It isn't causing any problems that I can measure at this time. However load and execution order are sensitive areas in an application that should not needlessly vary between desktop and mobile. It can bite us unexpectedly later. Especially because these variables (reporttime, bottomscripts) are quite generic and have been repurposed and extended a lot over time. It's quite likely people make assumptions about their order in the HTML stream.
Change 229963 had a related patch set uploaded (by Florianschmidtwelzow):
Use default order of footer elements