Validation fixes

[SLaks]

I fixed two issues with CSS property value validation:

• Allow border-bottom-color: inherit
• Require units for <length> values

I didn't check to make sure that there aren't any properties that only allow <length> and should also allow <integer> or <number>.

[SLaks]

I didn't include an ant build; if you want to, I can add another commit with that.

[nzakas]

I don't agree with the change to not allow unit-less values, so I'm not able to merge this in. In the future, please limit each pull request to one fix, that way I can merge in what I want and not have to either take or leave the entire thing.

[SLaks]

Why? Unit-less lengths are not valid per the CSS specs. (although older IEs did support them)

[nzakas]

Well, you yourself mentioned that you didn't do any research to figure out
if this change would break any existing behavior, so I'm not confident that
the change is correct. I'm not in favor of changing things that seem wrong
without understanding a) if it was causing a bug before and b) what are the
effects if the code is changed.

On Sat, Mar 9, 2013 at 4:43 PM, SLaks notifications@github.com wrote:

Why? Unit-less lengths are not valid per the CSS specs. (although older
IEs did support them)

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/48#issuecomment-14673853
.

---

Nicholas C. Zakas
@SlickNet

Author, Professional JavaScript for Web Developers
Buy it at Amazon.com:
http://www.amazon.com/Professional-JavaScript-Developers-Nicholas-Zakas/dp/1118026691/ref=sr_1_3