Closed
Bug 165772
Opened 22 years ago
Closed 22 years ago
Table splitting can be incorrect with rowspan cells
Categories
(Core :: Layout: Tables, defect, P3)
Tracking
()
RESOLVED
FIXED
mozilla1.2beta
People
(Reporter: karnaze, Assigned: karnaze)
References
()
Details
Attachments
(5 files, 4 obsolete files)
|
378 bytes,
text/html
|
Details | |
|
357 bytes,
text/html
|
Details | |
|
378 bytes,
text/html
|
Details | |
|
359 bytes,
text/html
|
Details | |
|
26.58 KB,
patch
|
karnaze
:
review+
karnaze
:
superreview+
|
Details | Diff | Splinter Review |
test case 1
There are still a number of problems involving splitting of rowspan cells. The
url (from bug 127128) is one example and there are others which I will add
later. nsTableRowGroupFrame::SplitRowGroup and SplitSpanning cells has a number
of flaws that need to be fixed.
|
Assignee | |
Updated•22 years ago
|
Severity: normal → major
Status: NEW → ASSIGNED
Priority: -- → P3
Target Milestone: --- → mozilla1.2beta
|
|
Assignee | |
Comment 1•22 years ago
|
||
|
|
Assignee | |
Comment 2•22 years ago
|
||
|
|
Assignee | |
Updated•22 years ago
|
Attachment #101360 -
Attachment is obsolete: true
|
|
Assignee | |
Comment 3•22 years ago
|
||
|
|
Assignee | |
Comment 4•22 years ago
|
||
|
|
Assignee | |
Updated•22 years ago
|
Attachment #101361 -
Attachment is obsolete: true
|
|
Assignee | |
Comment 5•22 years ago
|
||
|
|
Assignee | |
Comment 6•22 years ago
|
||
|
|
Assignee | |
Comment 7•22 years ago
|
||
|
|
Assignee | |
Comment 8•22 years ago
|
||
Attachment #100692 -
Attachment is obsolete: true
Comment on attachment 101367 [details] [diff] [review]
patch to fix the bug
sr=kin@netscape.com
Just add the comments, assertions and the firstOverflow to aRow change in
UndoContinuedRow() as we discussed over the phone.
Attachment #101367 -
Flags: superreview+
|
||
Comment 10•22 years ago
|
||
Comment on attachment 101367 [details] [diff] [review]
patch to fix the bug
r= alexsavulov
looks good to me, no additional requests
Attachment #101367 -
Flags: review+
|
|
Assignee | |
Comment 11•22 years ago
|
||
|
|
Assignee | |
Updated•22 years ago
|
Attachment #101367 -
Attachment is obsolete: true
|
|
Assignee | |
Updated•22 years ago
|
Attachment #101531 -
Flags: superreview+
Attachment #101531 -
Flags: review+
|
|
Assignee | |
Comment 12•22 years ago
|
||
The patch is in the trunk.
Status: ASSIGNED → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
This patch caused a rather serious regression, bug 212984 (see also its
duplicates and dependencies), and also see bug 212984 comment 7 for date of
regression.
You need to log in
before you can comment on or make changes to this bug.
Description
•