The default bug view has changed. See this FAQ.

MathML does not render properly with text-align:indent

RESOLVED FIXED in mozilla1.9.1b1

Status

()

Core
MathML
RESOLVED FIXED
9 years ago
9 years ago

People

(Reporter: fredw, Assigned: fredw)

Tracking

Trunk
mozilla1.9.1b1
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(4 attachments)

(Assignee)

Description

9 years ago
User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.0.1) Gecko/2008071618 Iceweasel/3.0.1 (Debian-3.0.1-1)
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.0.1) Gecko/2008071618 Iceweasel/3.0.1 (Debian-3.0.1-1)

Unexpected spaces appear in formulae when one uses text-align:indent

Reproducible: Always

Steps to Reproduce:
1.
2.
3.
(Assignee)

Comment 1

9 years ago
Created attachment 337307 [details]
testcase
(Assignee)

Comment 2

9 years ago
Created attachment 337434 [details] [diff] [review]
Add a text-indent:0 in mtable to prevent unexpected spaces.
[Checkin: Comment 12]
Attachment #337434 - Flags: review?(mozbugz)
Status: UNCONFIRMED → NEW
Component: General → MathML
Ever confirmed: true
Product: Firefox → Core
QA Contact: general → mathml
Version: unspecified → Trunk
Comment on attachment 337434 [details] [diff] [review]
Add a text-indent:0 in mtable to prevent unexpected spaces.
[Checkin: Comment 12]

Thanks, Frédéric.
I'm just trying to think whether maybe we should be doing this on the mtd or even on the anonymous nsMathMLmtdInnerFrame block frame.  It only makes a difference though if someone is going to set text-indent on an mtable, mtr, or mtd.  What do you think, Rob?
Attachment #337434 - Flags: superreview?(roc)
Comment on attachment 337434 [details] [diff] [review]
Add a text-indent:0 in mtable to prevent unexpected spaces.
[Checkin: Comment 12]

Let's do it on <mtable>, since the equivalent rule in html.css is on <table>.
Attachment #337434 - Flags: superreview?(roc)
Attachment #337434 - Flags: superreview+
Attachment #337434 - Flags: review?(mozbugz)
Attachment #337434 - Flags: review+
(Assignee)

Comment 5

9 years ago
Created attachment 337632 [details]
Bug when <math> is in an SVG foreignObject
(Assignee)

Comment 6

9 years ago
The previous attachment show the bug of text-indent when the <math> is inside a foreignObject: the formulae are shift to the right. Here again, it is fixable by attaching a text-indent:0 to -moz-svg-foreign-content
(Assignee)

Comment 7

9 years ago
Created attachment 337654 [details] [diff] [review]
Add a text-indent:0 to prevent the shift
[Checkin: Comment 12]
Attachment #337654 - Flags: review?(roc)
One thing to consider here is whether html within a foreignObject should inherit text-indent.

It looks like the issue in attachment 337632 [details] might be due to the <foreignObject> creating a block context for the inline <math>.  Maybe it needs to do that...
Attachment #337654 - Flags: superreview+
Attachment #337654 - Flags: review?(roc)
Attachment #337654 - Flags: review+
Comment on attachment 337654 [details] [diff] [review]
Add a text-indent:0 to prevent the shift
[Checkin: Comment 12]

It does need to use a block. I think this is right; <svg> should behave like <table> here.
(Assignee)

Updated

9 years ago
Keywords: checkin-needed
(Assignee)

Comment 10

9 years ago
Can someone commit my two patches, please?
It's marked checkin-needed so that should happen reasonably soon.
Checked these in, thank you:

http://hg.mozilla.org/mozilla-central/rev/2eef7a07820a
http://hg.mozilla.org/mozilla-central/rev/1b64769375b2
Assignee: nobody → fred.wang
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
Attachment #337434 - Attachment description: Add a text-indent:0 in mtable to prevent unexpected spaces. → Add a text-indent:0 in mtable to prevent unexpected spaces. [Checkin: Comment 12]
Attachment #337654 - Attachment description: Add a text-indent:0 to prevent the shift → Add a text-indent:0 to prevent the shift [Checkin: Comment 12]
Keywords: checkin-needed
Target Milestone: --- → mozilla1.9.1b1
You need to log in before you can comment on or make changes to this bug.