angular-masonry icon indicating copy to clipboard operation
angular-masonry copied to clipboard

Masonry directive as an element doesn't create columns

Open vrockai opened this issue 8 years ago • 2 comments

In the README.md, the masonry directive is used as an element. It doesn't work properly (creates only a single column) for me when I use it as E directive like:

<masonry>
    ...
</masonry>

but it works well when used as A directive like:

<div masonry>
    ...
</div>

vrockai avatar Mar 18 '16 17:03 vrockai

This also happened to me, using bower install.

elstanto avatar Apr 06 '16 16:04 elstanto

I can reproduce this with the demo. The documentation does seem to be at odds. It says:

You have to include the masonry attribute on the element holding the bricks.

But then the examples that follow use masonry as an element.

Seems like a pull request could:

  • fix the docs
  • change restrict: AE to restrict: A

@passy Unless there's a bug disallowing use of masonry as an element, I could submit some changes.

jodytate avatar Apr 09 '16 23:04 jodytate