Bug 15834 – There are many subtle bugs in the Styles pane of the Web Inspector

• Status: RESOLVED
• Resolution: FIXED
• Component: Web Inspector (Deprecated)

[Timothy Hatcher]

The styles pane has a few bugs.

- Poor handling of duplicate properties in the same rule. Some of this can't be fixed since we can't only get the "winning" value for duplicate properties. So we should only show one entry per unique property name.
- Computed style does not show font shorthand properties if font was used.
- Property priority is broken, the wrong properties are crossed out.
- Border shows null for the shorthand value.
- Shorthands don't show their priority.
- HSL and HEX colors have now preview swatch blocks.
- Complex code, making it hard to reuse for console.log later.

See attached file for some test cases.

[Timothy Hatcher]

Created attachment 17037 [details]
Test case

[Timothy Hatcher]

Created attachment 17038 [details]
Patch

[Adam Roben (:aroben)]

Comment on attachment 17038 [details]
Patch

+        Bug 15834: There are many subtile bugs in the Styles pane of the Web Inspector

Typo: Subtile -> Subtle

+        for (var i = 0; i < boxes.length; ++i) {
+            var name = boxes[i];

Even though you've defined name here, you still use boxes[i] later on.

It looks like PropertiesSidebarPane.js has a bunch of tabs in it.

I'm not really sure what the changes to the TreeOutline code are supposed to be doing (the ChangeLog didn't mention them).

r=me

[Timothy Hatcher]

Fixed the issues Adam mentioned.
Landed in r r27431. http://trac.webkit.org/projects/webkit/changeset/27431