search

jsdom / cssstyle

A Node.js implementation of the CSS Object Model CSSStyleDeclaration interface
MIT License
109 stars 70 forks source link

Fixing bug w/font shorthand returns "background" #31

Closed arvind closed 9 years ago

arvind commented 9 years ago

There seems to be an additional error somewhere, because I'm getting font: sans-serif 11px 11px; as the output, rather than font: 11px sans-serif;

chad3814 commented 9 years ago

probably a conflict between font-size accepting a length and line-height also accepting a length, so they are both taking it; which isn't right (I think). maybe line-height shouldn't be part of the font shorthand, I'll have to look at some docs