Skip to content
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

ISBAT crop or auto-height images in Carousel Block #944

Closed
AnthonyLedesma opened this issue Oct 3, 2019 · 3 comments
Closed

ISBAT crop or auto-height images in Carousel Block #944

AnthonyLedesma opened this issue Oct 3, 2019 · 3 comments
Labels
[Priority] Low This issue/pull request is not immediate [Type] Enhancement Something new that adds functionality

Comments

@AnthonyLedesma
Copy link
Member

AnthonyLedesma commented Oct 3, 2019

Is your feature request related to a problem? Please describe.
The Carousel block will apply auto-height to images dependent on the image height. We should explore methods to give the user better control over whether the image will effectively crop or will resize accordingly.

Describe the solution you'd like
Something we could consider is adding a "Crop" toggle option which when toggled essentially makes it autoheight and one image across only.

Describe alternatives you've considered
At this time the alternatives are custom CSS or editing the image prior to upload to Carousel block.

Inspired by wp.org support request
https://wordpress.org/support/topic/carousel-block-object-fit-contain/
https://wordpress.org/support/topic/images-uncropped-in-the-carousel-block

@AnthonyLedesma AnthonyLedesma added [Type] Enhancement Something new that adds functionality request labels Oct 3, 2019
@richtabor richtabor added [Priority] Low This issue/pull request is not immediate and removed request labels Feb 10, 2020
@lorenzaME
Copy link

lorenzaME commented Apr 3, 2020

Hi I think that the Carousel block should not crop images, is there a chance to have this behavior?
Hope so and that you will assign this task to improve the carousel behavior because it is quite frustrating having photos cropped.
Thanks
Lorenza

@gmariani
Copy link

gmariani commented Feb 4, 2021

Still an issue. I have square images that get the top and bottoms cropped off. And can't display just a single image per slide either, so you see 1/4 of the images before and after, unlike any other slide show plugin you'd expect.

@AnthonyLedesma
Copy link
Member Author

Thank you for contributing. I am closing this issue and linking back to a primary issue regarding refactoring of the Carousel gallery block where we plan to address this ask.
#2077

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Priority] Low This issue/pull request is not immediate [Type] Enhancement Something new that adds functionality
Projects
None yet
Development

No branches or pull requests

4 participants