New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Row layout with fixed height and dynamic width #57
Comments
Hi, thanks for the feedback. I've had the same idea (#53) and I am glad there is more support for this layout. The linked post is a good explanation and I will look into it more. |
Awesome, I hadn't seen you were already looking in to this! Looking forward to it seeing it implemented. |
I've implemented this in the "justified" branch if you want to try it before it is merged and released |
This is exactly what I was hoping for, thank you so much! I did come across two small issues (though I'd still use it whether or not these get fixed):
|
Oh, I think both of those are bugs - 1. definitely I've done some work since and merged it into master, could you please try it with the updated version? If there are still problems, could you please send me an address so that I could look at it? Either here in the issue, or at genabitu@gmail.com. Thank you for your feedback and help. |
Working exactly as expected now. Thanks again! |
ok :-) |
Fantastic plugin. I'm wondering if there's any way to include an option where each row would have a static height while the widths of the images would change dynamically? For example all images could be set at 300 pixels tall? The number of images in each row would then also be dynamic.
In fact Google Photos does something even more advanced. Within an album, each row might vary from 300-400 pixels tall (on my screen). The specific height of each row is automatically calculated to accommodate the total width of the row (multiple photos) to be 1440 pixels. But within each row, every photo is always the same height as it's neighbor which makes for a nice gallery setup. Here's an example of the CSS to implement this:
xieranmaya/blog#6
The text was updated successfully, but these errors were encountered: