Summary: | REGRESSION (r15069): getPropertyValue() returns null when element has display: none (affects Technorati.com) | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Stef Pause <stef.pause> | ||||||
Component: | DOM | Assignee: | Dave Hyatt <hyatt> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | ap, dev+webkit, hyatt, mepard, mitz, mrowe | ||||||
Priority: | P1 | Keywords: | HasReduction, InRadar, Regression | ||||||
Version: | 523.x (Safari 3) | ||||||||
Hardware: | Mac (PowerPC) | ||||||||
OS: | OS X 10.4 | ||||||||
URL: | http://technorati.com/ | ||||||||
Attachments: |
|
Description
Stef Pause
2007-03-17 16:31:24 PDT
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.
*** 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). 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
|