During investigation of T143768 we discovered that certain browser extensions, for example MathML, result in output of 2 math equations being loaded on the mobile site as a result of lazy loaded images.
To replicate:
Maybe the diversity of platforms on mobile is not as huge as for desktop, and there is a way to figure out if the client supports mathml.
In that case the fallback image should not be displayed at all.
Especially on pages with many formulae it makes a big difference for example on edge connections. With MathML on the page is just there (the mathml is in the dom anyhow). Without MathML every single image needs to be loaded indvidually and the overall amout of data to be transferred is much higher.
In reply to your comment https://phabricator.wikimedia.org/T134281#2811490:
@Jdlrobson the FF extension is only one of a few alternatives that use that mechanism. There are corresponding for Chrome (using MathJax) and other browsers. It's documented here https://www.mediawiki.org/wiki/Extension:Math/advancedSettings#CSS_for_the_MathML_with_SVG_fallback_mode and here https://www.mediawiki.org/wiki/Extension:Math#Viewing_math
@Jdlrobson I just realize that there is no problem with duplicated math images in previw mode. cf. https://phabricator.wikimedia.org/T134281#2895521
Do you understand why that looks better?
Yup only desktop browsers who have opted into browser extensions. I ended up investigating this issue out of interest (effectively doing the spike... feel free to pull into sprint @ovasileva if you want to get a final answer)
Looking at the code
.mwe-math-mathml-inline + .mwe-math-fallback-image-inline,
.mwe-math-mathml-display + .mwe-math-fallback-image-display {
display: none !important;
}We can fix this by doing 2 things
<Div class="lazy-image-placeholder loaded" data-class="mwe-math-fallback-image-inline"></div>
#2 is the tricky bit. We can deploy a hack that whitelists this class or explore what happens when we use replaceWith rather than appendTo (I forget why we decided to use appendTo but I think there was a good reason).
@bmansurov what do you think?
I don't remember either. If we end up working on the task, I think it's worth finding out.
Another approach is that we submit a patch to https://github.com/fred-wang/Mathzilla/blob/b5648ec71a704ce5d316f917583eefcd6de5c9b3/native-mathml/data/mediawiki.css#L16 and add the selector. Looks straight forward but last commit was April 2016. Although this fix may help that extension it may not help others.
@Jdlrobson I'm happy to help. The overall goal is that people can read formulae as integral parts of the text. Especially on mobile it's a huge advantage to load a page with only one request instead of a few hundert requests (which is the case for pages with many math expressions in it). If you are aware of other ways to determine if a mobile client supports mathml that would also be an option.
@Jdlrobson @fredw is still actively maintaining the package as far as I know... how would you change the selector?
Le 08/02/2017 à 19:13, Physikerwelt a écrit :
Physikerwelt added a subscriber: fredw.
Physikerwelt added a comment.@Jdlrobson https://phabricator.wikimedia.org/p/Jdlrobson/ @fredw
https://phabricator.wikimedia.org/p/fredw/ is still actively maintaining the
package as far as I know... how would you change the selector?
Adapting the CSS and updating the extension would be straightforward,
hopefully this can be done in a backward compatible way (i.e. the new
version of the addon would still work will previous versions of mediawiki).
Similar CSS is used in the mediawiki doc, Safari extension etc so they
would need to be updated too.
Great! I've submitted a patch that should fix this issue in this particular extension: https://github.com/fred-wang/Mathzilla/pull/40
Thanks for the merge! I've added this to a sign off column simply as a reminder to test the new release fixes this.