[webkit-reviews] review denied: [Bug 40916] Web Inspector: In Styles pane jump from a CSS property name to the value by typing colon : [Attachment 60471] Tab behavior restored

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Jul 9 12:30:43 PDT 2010


Joseph Pecoraro <joepeck at webkit.org> has denied Nikita Vasilyev <me at elv1s.ru>'s
request for review:
Bug 40916: Web Inspector: In Styles pane jump from a CSS property name to the
value by typing colon
https://bugs.webkit.org/show_bug.cgi?id=40916

Attachment 60471: Tab behavior restored
https://bugs.webkit.org/attachment.cgi?id=60471&action=review

------- Additional Comments from Joseph Pecoraro <joepeck at webkit.org>
Missing ChangeLog. r- for that.


> +	       } else {
> +		   delete section._afterUpdate;
>	       }

The else should not have braces. Even though it looks ugly (to me), its the
WebKit style rules and its better to have consistent style everywhere. =)


More information about the webkit-reviews mailing list