Closed Bug 133429 Opened 21 years ago Closed 21 years ago

dynamically created MathML content sometimes doesn't layout as expected

Categories

(Core :: MathML, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla1.0

People

(Reporter: steve.swanson, Assigned: rbs)

Details

Attachments

(3 files)

(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.)
Attached file testcase
includes instructions on how to reproduce the bug
Previous testcase only failed when patch for bug 129560 was applied. This
version seems to always fail.
-> m1.0. Great testcase...
Status: NEW → ASSIGNED
OS: Windows 2000 → All
Hardware: PC → All
Target Milestone: --- → mozilla1.0
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.
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 on attachment 76533 [details] [diff] [review]
patch to fix the bug

sr=attinasi, as usual
Attachment #76533 - Flags: superreview+
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+
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 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+
Checked-in.
Status: ASSIGNED → RESOLVED
Closed: 21 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.