search

brandonaaron / jquery-cssHooks

Collection of cssHooks that work with jQuery 1.4.3+
Other
478 stars 80 forks source link

more tests, fixed bug where margin/padding cssHooks were returning 3 space strings, changed to $.css #3

Closed burin closed 13 years ago

burin commented 13 years ago

added some tests to verify that the actual return values were matching the expected values fixed a bug where margin/padding css weren't returning anything, same with backgroundPosition

I switched it from $.style to use $.css, but only because $.style wasn't working when used this way ( $.style(elem, name ). I wasn't sure which was more "low level" and which was preferred, but switched it just to get the tests passing and the cssHooks to actually return something.

brandonaaron commented 13 years ago

Thanks for cleaning up my mess! Will push tomorrow. :)