Page MenuHomePhabricator
Create Task
Maniphest T95498

MathJax does not work on MobileFrontend
Closed, DeclinedPublic
Actions

Description

If you turn on MathJax rendering in the standard MediaWiki options (e.g., for English Wikipedia, go to your appearance preferences and select "Client-side MathJax" under "Math"), viewing pages on mobile using the MobileFrontend displays math in pages as the raw LaTex-style math markup (e.g., $ L=D-A, $), not as a rendered mathematical statement.

Selecting other math rendering options (e.g., "MathML with SVG or PNG") works as expected on mobile, and renders the mathematical symbols.

It seems like selecting MathJax as a rendering option disables server-side math rendering, but MobileFrontend does not include MathJax.

Details

SubjectRepoBranchLines +/-
WIP: Do not submitmediawiki/extensions/Mathmaster+10 -3
Customize query in gerrit

Related Objects
Search...

Event Timeline

int3h created this task.Apr 8 2015, 10:47 PM
int3h raised the priority of this task from to Needs Triage.
int3h updated the task description. (Show Details)
int3h added a project: MobileFrontend.
int3h subscribed.
Restricted Application added a project: Web-Team-Backlog. · View Herald TranscriptApr 8 2015, 10:47 PM
Restricted Application added a subscriber: Aklapper. · View Herald Transcript
Jdlrobson edited projects, added Math, Mobile; removed Web-Team-Backlog, MobileFrontend.Apr 9 2015, 1:57 AM
Jdlrobson set Security to None.
Jdlrobson added a project: MobileFrontend.
Restricted Application added a project: Web-Team-Backlog. · View Herald TranscriptApr 9 2015, 1:58 AM
Jdlrobson subscribed.Apr 9 2015, 1:58 AM

This is an issue in the MathJax extension. If you find a maintainer of it I'll be able to help out.

Physikerwelt subscribed.Apr 9 2015, 2:45 AM
In T95498#1192881, @Jdlrobson wrote:

This is an issue in the MathJax extension. If you find a maintainer of it I'll be able to help out.

Do you mean the MathJax or the Math extension. I do not know if the mathjax resource modules (and there are quite a few) have been disabled by mobile frontend on purpose. If all the of the provided MathJax data would be downloaded that might be bad for low Budget data plans. I.e. since a WMF specific Version of MathJax 2.3 is used. See also T78046

Physikerwelt triaged this task as Medium priority.Apr 9 2015, 2:46 AM
Jdlrobson added a comment.Apr 9 2015, 3:07 AM

I meant MathJax sorry. Where should Mathjax bugs get filed? Mathjax will need to test itself out in the mobile skin (many client side code are incompatible) and then explicitly turn itself on by updating its ResourceLoader modules. Who can I talk to about that?

Legoktm subscribed.Apr 9 2015, 3:33 AM

https://www.mathjax.org/ --> https://github.com/mathjax/MathJax

Pkra subscribed.Apr 9 2015, 6:41 AM

Duplicate of https://phabricator.wikimedia.org/T47816, I think.

TheDJ subscribed.Apr 9 2015, 2:10 PM

I would say that if MathJax is not supported on mobile (it's a beast, we better not load it), we should ignore the user selected preference and fall back to another setting, that is supported.

So that means, use MathML with SVG or PNG images.

Physikerwelt added a comment.Apr 9 2015, 2:46 PM
In T95498#1194133, @TheDJ wrote:

I would say that if MathJax is not supported on mobile (it's a beast, we better not load it), we should ignore the user selected preference and fall back to another setting, that is supported.

So that means, use MathML with SVG or PNG images.

+1

Jdlrobson added a comment.Apr 10 2015, 5:52 PM

This bug keeps coming up so I'd be super happy to resolve this in some satisfactory way once and for all. Alas I've never had the time to look into this extension. What do you need from MobileFrontend to do this? You can use Output::getTarget to determine whether you are on mobile or not. Please raise some specific sub tasks for any MobileFrontend changes you'll need to do this.

Jdlrobson removed projects: Web-Team-Backlog, MobileFrontend.Apr 10 2015, 5:52 PM
Physikerwelt claimed this task.Apr 10 2015, 6:28 PM

I can implement the change suggested by TheDJ it's probably two or three lines of code

TheDJ added a comment.Apr 10 2015, 6:30 PM

Is parser output vary'ied per target ? otherwise it might be a bit difficult to override the user's preferences.

Physikerwelt moved this task from Inbox to Doing on the Math board.Apr 13 2015, 4:19 PM
Physikerwelt added a comment.Apr 13 2015, 4:38 PM

I can not find the getTarget method
https://doc.wikimedia.org/mediawiki-core/master/php/classParserOutput.html

gerritbot subscribed.Apr 13 2015, 4:43 PM

Change 203857 had a related patch set uploaded (by Physikerwelt):
WIP: Do not submit

https://gerrit.wikimedia.org/r/203857

gerritbot added a project: Patch-For-Review.Apr 13 2015, 4:43 PM
Jdlrobson added a comment.Apr 13 2015, 5:04 PM

setTarget / getTarget can be found on OutputPage not ParserOutput

Physikerwelt moved this task from Doing to Inbox on the Math board.Apr 13 2015, 5:29 PM
Florian subscribed.Apr 13 2015, 7:32 PM
Nemo_bis subscribed.Apr 24 2015, 4:39 PM
gerritbot added a comment.May 16 2015, 3:26 PM

Change 203857 abandoned by Physikerwelt:
WIP: Do not submit

Reason:
This approach does not work. More work is needed to bypass caching issues

https://gerrit.wikimedia.org/r/203857

Physikerwelt closed this task as Declined.Jul 19 2015, 10:35 PM

MathJax will be removed T99369

Physikerwelt closed subtask T95918: Figure out how to determine if page is visited in mobile mode from onParserFirstCallInit as Invalid.Dec 3 2015, 5:45 AM
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL