search

eiriklv / react-masonry-component

A React.js component for using @desandro's Masonry
MIT License
1.44k stars 145 forks source link

add onImagesLoaded props #33

Closed tonytonyjan closed 8 years ago

tonytonyjan commented 8 years ago

we might need this to solve some problems like this one: https://css-tricks.com/forums/topic/masonry-loading-images-displaynone-issue/

afram commented 8 years ago

@tonytonyjan Thanks for your contribution!

Do you mind please adding a section in the Readme about this new handler as well as updating the tests?

afram commented 8 years ago

@tonytonyjan cheers for the changes. Are you planning on adding tests for your new functionality?

tonytonyjan commented 8 years ago

@afram I am not sure, I have never written test for reactjs before, I can try if I am available next week.