Last Comment Bug 689641 - (mlabeledtr-support) Add support for mlabeledtr
: Add support for mlabeledtr
Status: NEW
: dev-doc-needed, helpwanted
Product: Core
Classification: Components
Component: MathML (show other bugs)
: Trunk
: All All
P4 normal with 4 votes (vote)
: ---
Assigned To: Nobody; OK to take it and work on it
: Anthony Jones (:kentuckyfriedtakahe, :k17e)
Mentors: Frédéric Wang (:fredw)
Depends on:
Blocks: mathml-2 557086 mathml-in-mathjax 958947
  Show dependency treegraph
Reported: 2011-09-27 10:17 PDT by Frédéric Wang (:fredw)
Modified: 2014-06-17 08:09 PDT (History)
11 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

testcase (3.97 KB, text/html)
2011-09-27 10:17 PDT, Frédéric Wang (:fredw)
no flags Details

Description User image Frédéric Wang (:fredw) 2011-09-27 10:17:39 PDT
Created attachment 562803 [details]
Comment 2 User image Frédéric Wang (:fredw) 2012-09-01 03:15:03 PDT
Because of the vertical alignment requirement, it seems that the most easy way to implement this is to follow the REC's  suggested model:

"To place a label, an implementor might think in terms of creating a larger table, with an extra column on both ends. The columnwidth attributes of both these border columns would be set to "fit" so that they expand to fill whatever space remains after the inner columns have been laid out. Finally, depending on the values of side and minlabelspacing, the label is placed in whatever border column is appropriate, possibly shifted down if necessary, and aligned according to columnalignment."

So maybe we should do something in nsCSSFrameConstructor to add extra column on both ends of an <mtr> or <mlabeledtr> element (with the label placed in the appropriate extra cell in the latter case). We will need to change the implementation of rowlines, columnlines etc to take into account that we have these extra columns. That would give a slightly incorrect implementation of mtable@width, though.
Comment 3 User image Frédéric Wang (:fredw) 2012-11-17 09:47:22 PST
An add-on to workaround this bug:
Comment 4 User image Frédéric Wang (:fredw) 2012-11-22 08:43:38 PST
Someone from the Webkit MathML team contacted me about this bug today. Adding me as mentor, although I don't know well the code in nsCSSFrameConstructor to change. I think even something like
Comment 5 User image Frédéric Wang (:fredw) 2012-11-22 08:45:01 PST
I think even something like what David Carlisle's XSLT stylesheet do would be OK (adding a column to <mtable>'s that have a <mlabeledtr> child).
Comment 6 User image Frédéric Wang (:fredw) 2012-11-30 08:32:52 PST
Mass change: setting priority to 2 for bugs that would be nice fix if Gecko's MathML support is enabled by default in MathJax but that are not in my opinion strictly required or for which a workaround could be written in the MathJax code.

Note You need to log in before you can comment on or make changes to this bug.