ded / morpheus

A Brilliant Animator
504 stars 57 forks source link

line-height units should not be ignored #20

Closed mantoni closed 10 years ago

mantoni commented 12 years ago

If lineHeight is specified with px, morpheus animates it without the unit which is interpreted as em.

morpheus($nodes, {
  lineHeight: "21px"
});

Removing lineHeight from the unitless object fixes the issue for me, but not sure about side effects with other use cases.

ded commented 12 years ago

yeah, i can't remove it, but we just need a special case around it since it's the ONE AND ONLY property that can accept a unit, and also not (jerk css gods). will see what we can do