Juicy / juicy-tile-grid

Masonry-like Custom Element for sortable tiles that packs efficiently without changing HTML structure (changes CSS only), and adapts it to CSS Grid Layout. Extended renderer for juicy-tile-list.
http://juicy.github.io/juicy-tile-grid/
MIT License
8 stars 1 forks source link

Fractions, first impression #11

Closed miyconst closed 9 years ago

miyconst commented 9 years ago

Continuation to https://github.com/Juicy/juicy-tile-grid/issues/8

  1. Min width of a dynamic parent container is set to the width of window. If a layout should work with screen size 1000px - 2000px, I have to manually resize window to 1000px and then apply layout.
  2. Sometimes I want to resize window to check how the new styles work. After window size increased and styles recalculated the min width of the parent container is set to the current width.
  3. Somehow not clear behaviour with flexible tiles: juicy-tile-grid-2

Why do the first and the third tiles change their size?

tomalec commented 9 years ago
  1. Should be handled by RWD range picker,
  2. It seems like a bug, by recalculated, do you mean that refresh method is called?
  3. Are there any other tiles below?what is the initial container size for both, what is the gutter?
miyconst commented 9 years ago
  1. Okay.
  2. I did not check the code yet, but this happens when I resize window and then click on a tile. If reduce window size right after increasing then it works fine.
  3. Yes, there are some other tiles after those three, the width of the container is auto, the gutter is 25. See the picture: juicy-tile-grid-4
tomalec commented 9 years ago
miyconst commented 9 years ago

After your explanation, I still don't understand why 50% + 25% + 25% equals to 50% + 250px + 25% when 250px is less than 25%.

tomalec commented 9 years ago

why 50% + 25% + 25% equals to 50% + 250px + 25% when 250px is less than 25%.

It does not,as there are no columns with such sizes. Could you show me sizes of other tiles?

tomalec commented 9 years ago

@miyconst Is there anything left, that requires further explanation/changes/are there any features missing?

Can we close this issue?

miyconst commented 9 years ago

Yes. I close this issue, and will create new if something comes up.