Bug 13103 – REGRESSION (r15069): getPropertyValue() returns null when element has display: none (affects Technorati.com)

• Status: RESOLVED
• Resolution: FIXED
• Component: DOM

[Stef Pause]

Go to Technorati.com and click the "Sign In" link at the top-right.  This onclick JavaScript should cause the div id="signin" to change its style (via a slide effect) to "display: block;" which it does correctly in Safari v2.0.4, Firefox, Camino, etc.  However, in the latest Webkit nightly it slides down then immediately changes back to "display: none;".  This obviously makes it somewhat difficult to sign in ;)

The relevant JavaScript is:
onclick="if(document.all) $('select').hide();if($e('signin').style.display=='block') {$('#signin').slideUp('fast'); } else { $('#signin').slideDown('fast');$e('username').focus()};return false"

I'm not entirely sure if it's a Javascript bug or a DOM bug, I suspect the former, so I've filed it under "New Bugs".

[Matt Lilek]

Created attachment 13701 [details]
Reduction

The heart of this bug is that when an element has display: none, getPropertyValue() returns null instead of 'none' like Firefox/Opera.

This works in Safari 2.0.4 because jQuery (Technorati's JS framework of choice) checks if the getComputedStye() returns null, which Safari 2.0.4 does when an element has display: none.  In that case they then check if they're looking for the display property and return 'none'.

The reason this disappears in ToT after the animation is because we don't return 'none', jQuery's animation gizmo never gets sets the display as block permanently when it's finished, presuming we're handling that ourselves (ie: we want it to be display: inline) and removes the temporary display: block it added during animation.

[Maciej Stachowiak]

<rdar://problem/5072345>

*** Bug 13193 has been marked as a duplicate of this bug. ***

Regressed in <http://trac.webkit.org/projects/webkit/changeset/15069>. Safari 2 returns null from getComputedStyle() if the element doesn't have a renderer. r15069 and later return a CSSStyleDeclaration (which subsequently return null for all properties, see bug 12384).

[Dave Hyatt]

Created attachment 13859 [details]
Honor display property when we have no renderer.

*** Bug 13223 has been marked as a duplicate of this bug. ***

Comment on attachment 13859 [details]
Honor display property when we have no renderer.

Needs a change log and a layout test. Also make sure that existing tests pass or are updated as needed. r=me

[Dave Hyatt]

Fixed in r20570.