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)

Product:
Core
Component:
MathML
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla1.0

People

(Reporter: steve.swanson, Assigned: rbs)

Details

Attachments

(3 files)

testcase
1.42 KB, application/xhtml+xml
Details
better testcase using fences
1.58 KB, application/xhtml+xml
Details
patch to fix the bug
4.89 KB, patch
dbaron
: review+
attinasi
: superreview+
asa
: approval+
Details | Diff | Splinter Review 
(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.