Last modified: 2014-02-12 23:55:26 UTC
There are various bits of HTML that are added in MobileFormatter that would better belong in the Skin itself. These include back to top link, content_block, section_heading and section classes. It would be better for the skin to be given a data structure describing the sections similar to what the API returns and let the template (MinervaTemplate) decide how to format this. It would also be useful to make this less mobile specific so it can be used on the desktop Minerva skin.
https://gerrit.wikimedia.org/r/#/c/64104/ was a first stab to try and explain this better.
This would also allow us to ensure that content_0 is present on all pages and allow us to run photo uploads CTA on the desktop version of Minerva.
Max appears to be working on this.
To clarify this bug was about moving more control to the template (I got confused with the word skin). It seems ideally that some changes would need to be upstreamed to core for this to happen. Proof of concept: https://gerrit.wikimedia.org/r/#/c/70736/
I would like to propose we move the formatter code as it is now into SkinMinerva/the Skin class in core so that the desktop Minerva skin can make use of it are there any blockers to doing this?
Since SkinMinerva and the formatter have changed drastically this problem doesn't seem so relevant now so closing.
