Closed Bug 689641 (mlabeledtr-support) Opened 13 years ago Closed 4 years ago

Add support for mlabeledtr


(Core :: MathML, defect, P4)






(Reporter: fredw, Unassigned, Mentored)




(Whiteboard: [lang=c++])


(1 file)

Attached file testcase
      No description provided.
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.
An add-on to workaround this bug:
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
Whiteboard: [mentor=fredw][lang=c++]
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).
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.
Priority: -- → P2
Priority: P2 → P4
Blocks: 958947
Whiteboard: [mentor=fredw][lang=c++] → [lang=c++]

Hi, I would like to work on this, but I need mentoring. Can someone advise me on the first steps? Where can I find the documentation for the relevant code as I can't find them on the Mozilla website and the commenting in the source code is pretty sparse. Thanks.

Hi, sorry this bug is probably going to be wontfix per 1495813. I invite you to look for another bug ;-)

Keywords: helpwanted

WONTFIX per bug 1495813 since <mlabeledtr> is not part of core

Closed: 4 years ago
Resolution: --- → WONTFIX

(In reply to Frédéric Wang (:fredw) from comment #8)

Hi, sorry this bug is probably going to be wontfix per 1495813. I invite you to look for another bug ;-)

Hey Frédéric Wang,
Please notify me, if you come across any bug fixes.

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