metafizzy / outlayer

:construction_worker: the brains & guts of a layout library
163 stars 63 forks source link

Switch Vendor prefixes to Object #47

Closed felipearosemena closed 8 years ago

felipearosemena commented 8 years ago

I just recently ran into an issue while testing the Masonry plugin on Safari 8 (both on Desktop and on my IPad). The items were not transitioning at all, and that was also messing up the layout.

Reason for this I found is that the proto.css method was trying to apply the transform property, while this browser required webkitTransform. By changing the vendorProperties variable to an object, the css method can reference the prefixed properties by accessing them by their non-prefixed name.

This change fixed Masonry for me.

desandro commented 8 years ago

Thank you for this contribution. I managed to uncover the same bug yesterday! I appreciate this fix, but looks like I no longer need it. :kissing_closed_eyes: See SHA: aaa786769

felipearosemena commented 8 years ago

No worries, thanks for updating the plugin! 🤘🏼