[Webkit-unassigned] [Bug 127616] Web Inspector: Value 'inherit' should be allowed for any CSS property, even if it is a non-inheritable property

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=127616





--- Comment #6 from Diego Pino <dpino at igalia.com>  2014-01-25 17:09:06 PST ---
Created an attachment (id=222241)
 --> (https://bugs.webkit.org/attachment.cgi?id=222241&action=review)
Test of initial and inherit values on non-inherited property (background-image)

I have attached a page showing how different boxes are rendered setting different values (initial, inherit, etc) for background-image (non-inherited property).

At this point I can conclude the following:

   * When an inherited property is not set, its value is set to the value of its parent.
   * When a non-inherited property is not set, its value is set to its initial value (spec specifies what that value is, http://www.w3.org/TR/CSS2/propidx.html).
   * Setting an inherited property to inherit is the same as setting no value for that property.
   * Setting a non-inherited property to initial is the same as setting no value for that property.
   * Initial and inherit can be applied to either inherited and non-inherited properties.

As Joseph told me on IRC, setting "inherit" to a non-inherited property in the current implementation is valid, because it's not crossed through. The point is that the autocompletion doesn't suggest "inherit" on non-inherited properties. When the user starts typing "in" it will only suggest "initial" for non-inherited properties (and suggests "inherit" and "initial" for inherited properties). The good thing of the current implementation is that it's easy to know what properties are inherited and which ones are not just by looking at the values suggested (if inherit is not suggested it means the property is non-inherited).

With regard to "unset", I agree with removing it.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.