Type checking length units

[jacobwardio]

Resolves issue: #3023 Incorrect type-checking for CSS length units

[seven-phases-max]

#2326 ?

[jacobwardio]

@seven-phases-max please read issue #3023, which this PR resolves.

[seven-phases-max]

#3023 is OK (see complete LU list below though), but could you please annihilate theis festival?

https://www.w3.org/TR/css-values-3:

em, ex, ch, rem, cm, mm, q, in, pt, pc, px, vw, vh, vmin, vmax

P.S. Ideally the test should be case-insensitive (not a big deal usually, but counting they're using both q and Q, this won't harm anyway).

[jacobwardio]

@seven-phases-max sorry, I understand what you mean now.

Please excuse my sleepy head late last night 😴

I'll refactor it to use isunit rather than introducing a load of isfoo/isbar/isbaz type methods later today. And yes, you're correct, the tests should really be case insensitive.

[jacobwardio]

• Use isunit instead of the isFoo festival.
• Add case-insensitivity to unit matcher.
• Add all units from the complete LU list.

[jacobwardio]

@seven-phases-max I have resolved your comments and updated my PR for issue #3172 :)

[seven-phases-max]

Thanks!

[matthew-dean]

Thanks! I'll try to tag and push a release soon on NPM.