dynamically created MathML content sometimes doesn't layout as expected

RESOLVED FIXED in mozilla1.0

Status

()

Core
MathML
RESOLVED FIXED
17 years ago
17 years ago

People

(Reporter: steve.swanson, Assigned: rbs)

Tracking

Trunk
mozilla1.0
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(3 attachments)

(Reporter)

Description

17 years ago
(See testcase, which includes instructions on how to recreate the bug.)

In certain cases, adding content to MathML documents causes existing content 
(in the run of MathML) to layout strangely. The only cases I've observed are 
with stretchy <mo>s.  (The testcase uses a big operator, but I've also seen the 
problem with fences.)
(Reporter)

Comment 1

17 years ago
Created attachment 76145 [details]
testcase

includes instructions on how to reproduce the bug
(Reporter)

Comment 2

17 years ago
Created attachment 76240 [details]
better testcase using fences

Previous testcase only failed when patch for bug 129560 was applied. This
version seems to always fail.
(Assignee)

Comment 3

17 years ago
-> m1.0. Great testcase...
Status: NEW → ASSIGNED
OS: Windows 2000 → All
Hardware: PC → All
Target Milestone: --- → mozilla1.0
(Assignee)

Comment 4

17 years ago
Found the problem. A state in the stretchy goes out of sync. Specifically, when
the layout of '(x)' finished, the stretchy chars for '(' and ')' recorded that 
they didn't stretch by setting their mDirection to -1. Now, when 'x' is replaced 
with a fraction, the old state causes the rendering to mess up. When the state 
is reset, the rendring works fine.
(Assignee)

Comment 5

17 years ago
Created attachment 76533 [details] [diff] [review]
patch to fix the bug
(Assignee)

Comment 6

17 years ago
as usual, pinging roc & attinasi for r/sr...
Summary: dynamically created MathML content sometimes doesn't layout as expected → [fix, awaiting r/sr/a] dynamically created MathML content sometimes doesn't layout as expected

Comment 7

17 years ago
Comment on attachment 76533 [details] [diff] [review]
patch to fix the bug

sr=attinasi, as usual
Attachment #76533 - Flags: superreview+
(Assignee)

Updated

17 years ago
Summary: [fix, awaiting r/sr/a] dynamically created MathML content sometimes doesn't layout as expected → [fix, have sr, awaiting r/a] dynamically created MathML content sometimes doesn't layout as expected
Whiteboard: have sr=attinasi
Comment on attachment 76533 [details] [diff] [review]
patch to fix the bug

r=dbaron
Attachment #76533 - Flags: review+
(Assignee)

Updated

17 years ago
Summary: [fix, have sr, awaiting r/a] dynamically created MathML content sometimes doesn't layout as expected → [fix, have r/sr, awaiting a] dynamically created MathML content sometimes doesn't layout as expected
Whiteboard: have sr=attinasi → have r=dbaron, sr=attinasi

Comment 9

17 years ago
Comment on attachment 76533 [details] [diff] [review]
patch to fix the bug

a=asa (on behalf of drivers) for checkin to the 1.0 trunk
Attachment #76533 - Flags: approval+
(Assignee)

Comment 10

17 years ago
Checked-in.
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED
Summary: [fix, have r/sr, awaiting a] dynamically created MathML content sometimes doesn't layout as expected → dynamically created MathML content sometimes doesn't layout as expected
Whiteboard: have r=dbaron, sr=attinasi
You need to log in before you can comment on or make changes to this bug.