Closed
Bug 317148
Opened 19 years ago
Closed 19 years ago
Dynamically changing the colspan of a cell completely screws up the layout
Categories
(Core :: Layout: Tables, defect)
Core
Layout: Tables
Tracking
()
RESOLVED
FIXED
People
(Reporter: bugzilla-mozilla-20000923, Assigned: bernd_mozilla)
References
()
Details
(Keywords: regression, testcase)
Attachments
(2 files)
|
458 bytes,
text/html
|
Details | |
|
1.05 KB,
patch
|
bzbarsky
:
review+
bzbarsky
:
superreview+
|
Details | Diff | Splinter Review |
This bug was found while NSA (IRC) was developing an in-page editor system. It changes the colspan of some table cells to adjust the layout dynamically. This works fine in Firefox 1.0, and IE 6.0. It completely messes up in Firefox trunk (and maybe 1.5, I can't check - 1.5 crashes on startup here).
The attached testcase is simplicity itself; just change the colSpan property [1], and the cell jumps three places to the right of where it should be! NSA told me that this was working "a few weeks back", so if that rings any bells about changes to table code...
[1] This will also occur if you use setAttribute("colspan", ...), but since this doesn't work in IE, I chose to use the property for the testcase.
|
Reporter | |
Comment 1•19 years ago
|
||
|
||
Comment 2•19 years ago
|
||
This regressed between 2005-11-10 and 2005-11-12, the most likely culprit seems bug 271789 to me.
Blocks: 271789
Attachment #203698 -
Flags: superreview?(bzbarsky)
Attachment #203698 -
Flags: review?(bzbarsky)
|
||
Updated•19 years ago
|
Attachment #203698 -
Flags: superreview?(bzbarsky)
Attachment #203698 -
Flags: superreview+
Attachment #203698 -
Flags: review?(bzbarsky)
Attachment #203698 -
Flags: review+
fix checked in
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
|
||
Comment 6•19 years ago
|
||
*** Bug 322732 has been marked as a duplicate of this bug. ***
You need to log in
before you can comment on or make changes to this bug.
Description
•