Closed
Bug 271955
Opened 20 years ago
Closed 20 years ago
"empty" table cells don't include borders in size even when 'empty-cells: show'
Categories
(Core :: Layout: Tables, defect, P2)
Core
Layout: Tables
Tracking
()
RESOLVED
FIXED
mozilla1.8alpha6
People
(Reporter: dbaron, Assigned: dbaron)
References
()
Details
(Keywords: css2, Whiteboard: [patch])
Attachments
(1 file)
2.60 KB,
patch
|
bernd_mozilla
:
review+
roc
:
superreview+
|
Details | Diff | Splinter Review |
In http://dbaron.org/css/test/intrinsic/block-width-max , the third of the four boxes is 4 pixels narrower than all the others. This is because the intrinsic sizing of the table cells determines that the cell is "empty" (I'm not sure whether that's correct, but that's another bug) and therefore doesn't add in the width of the borders. The code that accounts for cells being smaller because of lack of borders needs to check the 'empty-cells' property.
Assignee | ||
Updated•20 years ago
|
Status: NEW → ASSIGNED
Priority: -- → P2
Whiteboard: [patch]
Target Milestone: --- → mozilla1.8alpha6
Assignee | ||
Comment 1•20 years ago
|
||
Perhaps I should also rename |contentEmptyBeforeReflow| to |noBorderBeforeReflow|?
Assignee | ||
Updated•20 years ago
|
Attachment #167169 -
Flags: superreview?(roc)
Attachment #167169 -
Flags: review?(bernd.mielke)
Comment on attachment 167169 [details] [diff] [review] patch If you like to rename it its OK with me. The whole empty cell code is a nightmare and needs to be rewritten but thats another story.
Attachment #167169 -
Flags: review?(bernd.mielke) → review+
Attachment #167169 -
Flags: superreview?(roc) → superreview+
Assignee | ||
Comment 3•20 years ago
|
||
Fix checked in to trunk, 2004-11-30 19:50 -0800.
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 4•19 years ago
|
||
*** Bug 210710 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
•