Currently, unchecking an invalid property unexpectedly removes the property. Steps: 1. Open http://nv.github.io/webkit-inspector-bugs/styles-redesign/tests/invalid.html 2. Inspect <body> 3. Click on the checkbox for `color: blah;` or `invalid-name: blah;` Expected: Invalid property disables the same way as a valid property. Actual: Property gets removed. <rdar://problem/36797202> --- Fixing this would require non-trivial CSS parser changes on the backend. The backend has a heuristic to distinguish commented out properties, such as /* font-size: 11px */ from other comments, such as /* make this red */ The CSS parser isn't smart enough to distinguish an invalid CSS property, such as `color: blah`, from a comment that isn't a CSS property at all. As a workaround, I suggest to hide checkboxes for invalid properties. This is exactly what Chrome does.
<rdar://problem/38807602>
Created attachment 336424 [details] Patch
Created attachment 336425 [details] [Animated GIF] With patch applied
Comment on attachment 336424 [details] Patch r=me
Comment on attachment 336424 [details] Patch Clearing flags on attachment: 336424 Committed r229941: <https://trac.webkit.org/changeset/229941>
All reviewed patches have been landed. Closing bug.