Closed
Bug 37388
Opened 25 years ago
Closed 24 years ago
navquirks: need to ignore empty P in table cells
Categories
(Core :: Layout, defect, P3)
Tracking
()
VERIFIED
INVALID
M18
People
(Reporter: ekrock, Assigned: karnaze)
Details
(Keywords: compat)
Bug report of a need for a quirk for backward compatibility with Nav4 (and IE)
from raul@onshore.com who writes for Digital Chicago. Reportedly affects N6b1
across platforms. (This is a core Mozilla layout quirks enhancement request; do
not INVALID!)
Apparently, when there was an empty P in a table cell, Nav4 and IE5 essentially
ignored its presence and sized the cell to match content. We need a quirk to do
the same. It's important that we do this because Claris Home Page outputs such
markup, so there are now lots of pages out there with this expectation built in.
You can see how the various browsers format this markup in the screen shot at
this URL:
http://www.silva.com/article/images/netscape6_table.gif
Raul, could you please provide a URL of a page that demonstrates this problem?
*** REPORT FROM RAUL: ***
The Digital Chicago people discovered something very disturbing. It appears that
Claris Home Page inserts a <P> tag at the beginning of every table cell <TD>
declaration. All the older browsers simply ignored the <P> tag and made the cell
shrink to match the content. Netscape 6 however is very loyal to the code and
adds paragraph spacing at the bottom of the cell.
<http://www.silva.com/article/images/netscape6_table.gif>
The editor of Digital Chicago told me they have hundreds of pages created on
Home Page that look terrible because of this.
Is there any plan to have Netscape interpret these extraneous <P> tags the way
the are interpreted in every other browser?
*** COMMENT FROM PIERRE ***
We already have a Nav quirk to not draw backgrounds and borders for empty cells.
We could do the same quirk for cells that only have empty children.
*** COMMENT FROM RICKG ***
This seems reasonable. We don't want to yank (valid) but useless markup.
*** COMMENT FROM KARNAZE ***
The behavior Pierre mentions is not a quirk. Empty cells don't get backgrounds
or borders. CSS 2 defines the empty-cells property to give borders to cells with
no visible content (but there is no way to give backgrounds!).
"No visible content" means that none of the descendant frames of the cell has
any size. So, the code today works the way Pierre suggests.
I think Steve and I need to work this out. The first table below looks identical
in Nav and Gecko while the 2nd has the symptoms reported. I haven't checked yet
whether Claris' page contains something similar.
<table border><tr><td>foo</td><td><p> </p></td></tr></table> <table
border><tr><td>foo</td><td><p> </p></td></tr></table>
*** COMMENT FROM PIERRE ***
My comments were related to bug 8113 and bug 33244. With 'empty-cells: show',
you can indeed have a background on an empty cell in IE (Mac + Win). I don't
know whether it's the result of peterl's suggestion at the WG but that's how
they implemented it.
I said it was a quirk because the default value for empty-cells is 'hide' in
quirks mode and 'show' in strict mode, meaning that by default empty cells do
have borders and background in strict mode.
You may want to reopen bug 8113. It's currently Verified/Remind.
Reporter | ||
Comment 1•25 years ago
|
||
Changing Summary to read "ignore EMPTY P in table cells" which is more accurate.
Summary: navquirks: need to ignore extra P in table cells → navquirks: need to ignore empty P in table cells
Assignee | ||
Updated•24 years ago
|
Status: NEW → ASSIGNED
Target Milestone: --- → M18
Assignee | ||
Comment 2•24 years ago
|
||
If someone can produce a test case or url relevant to claris' problem (not just
any old hypothetical test case, please reopen. I couldn't add raul@onshore.com
to the CC list.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → INVALID
Comment 3•24 years ago
|
||
Were ok its Clairs that is the problem. Marking as Verified.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•