Allow vertical-align: inherit; as browsers do use it

[aripalo]

If I understood right, vertical-align shouldn't take value of inherit if one is to believe MDN, but then again browsers seem to implement it with td elements for example:

• http://trac.webkit.org/browser/trunk/Source/WebCore/css/html.css#L250
• http://mxr.mozilla.org/mozilla-central/source/layout/style/html.css

That is why I think inherit should be added to https://github.com/CSSLint/parser-lib/blob/master/src/css/Properties.js#L526:

"vertical-align"                : "auto | use-script | baseline | sub | super | top | text-top | central | middle | bottom | text-bottom | <percentage> | <length>",

[mattiacci]

'inherit' is indeed a valid value of 'vertical-align':
http://www.w3.org/TR/CSS21/visudet.html#propdef-vertical-align.

On Mon, Apr 20, 2015 at 4:03 PM, Ari Palo notifications@github.com wrote:

If I understood right, vertical-align shouldn't take value of inherit if
one is to believe MDN
https://developer.mozilla.org/en/docs/Web/CSS/vertical-align, but then
again browsers seem to implement it with td elements for example:

• http://trac.webkit.org/browser/trunk/Source/WebCore/css/html.css#L250
• http://mxr.mozilla.org/mozilla-central/source/layout/style/html.css

That is why I think inherit should be added to
https://github.com/CSSLint/parser-lib/blob/master/src/css/Properties.js#L526
:

"vertical-align" : "auto | use-script | baseline | sub | super | top | text-top | central | middle | bottom | text-bottom | | ",

—
Reply to this email directly or view it on GitHub
#151.

[lorenz068]

I have the same problem, please add this value! Thanks...to release a new version too 👍