Replace default height for "banner image" in grid listing by max-height?
#2990
Replies: 2 comments 6 replies
-
|
I definitely think it is worth considering making our grid more flexible. The |
Beta Was this translation helpful? Give feedback.
-
|
Yeah that sounds very good. We could also consider exposing the |
Beta Was this translation helpful? Give feedback.
-
|
There is a JavaScript library that could be nice as well and related to let the cards in grid layout have flexible height. |
Beta Was this translation helpful? Give feedback.
-
|
That is really interesting. It is almost like it should be another grid layout altogether. I have it on my todo list to make grid layouts extensible using extensions- it would be awesome to request a listing layout |
Beta Was this translation helpful? Give feedback.
-
|
It might be actually a better idea to make a new layout option with no |
Beta Was this translation helpful? Give feedback.
-
|
Yeah I sort of like that approach. Masonry looks very cool for making really compact layouts of wildly different grid cards which is both very cool but perhaps different enough than our simple grid to warrant a new layout type! |
Beta Was this translation helpful? Give feedback.
-
|
Closing this in favour of #2999 |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
-
Hi,
currently, the grid listing has a default height of 150px for top image in the card which causes the image to be "cropped" in nearly all situations.
Instead, I would suggest to not set
heightproperty butmax-heightto allow responsive scaling of the image without cropping the image.For example, on the left, manually defined max-height, while on the right it's the default from the grid listing.
A video to show the resizing issue of setting
height.resizing.mov
Beta Was this translation helpful? Give feedback.
All reactions