I absolutely hate this kind of layout, especially when combined with infinite scrolling. Makes it impossible to systematically go over all items.
1) Fixed height rows: lower information density (excess white space around some of the items), but easier to scan systematically
2) Masonry layout: higher information density (no excess white space around items), but harder to scan systematically
Nobody is “right” here, it’s simply user-by-user preferences about what you prefer.