[webkit-reviews] review denied: [Bug 11888] REGRESSION (r18320): Web Inspector panes broken : [Attachment 11943] Patch

bugzilla-request-daemon at macosforge.org bugzilla-request-daemon at macosforge.org
Wed Dec 20 21:35:42 PST 2006


Alexey Proskuryakov <ap at webkit.org> has denied Alexey Proskuryakov
<ap at webkit.org>'s request for review:
Bug 11888: REGRESSION (r18320): Web Inspector panes broken
http://bugs.webkit.org/show_bug.cgi?id=11888

Attachment 11943: Patch
http://bugs.webkit.org/attachment.cgi?id=11943&action=edit

------- Additional Comments from Alexey Proskuryakov <ap at webkit.org>
Could we wait just a bit more before reverting a behavior that's (a)
standard-compliant AND (b) matches WinIE? So far, there were no reports of Web
compatibility problems (except for www.apple.com/getamac, where we share IE
behavior now).

To correctly remove a property, one should just set it to empty:
"myElement.style.display = ''".

I believe this Inspector JS bug needs to be fixed even if we decide to disable
exceptions in this case, so r-.



More information about the webkit-reviews mailing list