191428 - Paragraphs with display set to none still uses space

Status: VERIFIED INVALID

(Core :: CSS Parsing and Computation, defect)

Description

[José Jeria]

User-Agent:       Mozilla/4.0 (compatible; MSIE 5.01; Windows NT 5.0)
Build Identifier: 

Setting display:none; to <p> doesn't completely hide the paragrah.

Using build 2003013008.

Reproducible: Always

Steps to Reproduce:
See testcase.

Comment 1

[José Jeria]

Attachment: Testcase

Comment 2

[Boris Zbarsky [:bzbarsky]]

That's because we have a quirk that the first content node in a table cell gets
no top margin.  Changing display does not remove the presence of a content node...

In standards mode the space at the top would be there even when the first <p>
was visible.

Comment 3

[José Jeria]

Sorry, I didn't quite understand that, but shouldn't the display:none remove the
presence of the content node? 
IE and Opera 7 does it, are they incorrect?

Comment 4

[Boris Zbarsky [:bzbarsky]]

Style can't remove the presence of a content node.

There is no "correct" here -- we're talking about quirks mode (eg the fact that
there is no space at the top when the <p> is display:block is a bug, per CSS).

This particular issue with this quirk is known, and not worth fixing because
it's almost never encountered in quirks-mode pages in the wild.

Comment 5

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Bug 33784 would fix this, anyway.  But that's just a bug on making our incorrect
behavior in quirks mode invalid in more cases.

Comment 6

[Paul Wyskoczka]

v