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

Cover Block: Add Background Image Size Selector #19223

Closed
bbertucc opened this issue Dec 18, 2019 · 1 comment
Closed

Cover Block: Add Background Image Size Selector #19223

bbertucc opened this issue Dec 18, 2019 · 1 comment

Comments

@bbertucc
Copy link

@bbertucc bbertucc commented Dec 18, 2019

Is your feature request related to a problem? Please describe.
Cover blocks currently show full-size images. Users upload images that are too large for the dimensions of the cover block, then have no way of selecting a more appropriately sized image. I see massive load times and resource use on large multisites with admins who have no image-editing experience.

Describe the solution you'd like
Until the overarching issue with image sizes is address (see #6177) is addressed, I'd like to add a single "Background Image Size" select.
{A1D3B70A-1E7B-497F-AFB1-2F5D75A5EF94} png

Describe alternatives you've considered
We tell users to load smaller images, but many users do not know how to optimize their images.

@draganescu

This comment has been minimized.

Copy link
Contributor

@draganescu draganescu commented Jan 15, 2020

Howdy @bbertucc and thanks for opening this issue.

Initially I thought it to be a good idea, options!, but then ... why would one ever select thumbnail for backgroung in a cover block? And the same case can be made about any other size. This is one of those situations where the options do more harm then good.

Adding a background size selector doesn't make much sense considering the large amount of screen sizes and densities on which content can potentially be shown.

I will close this as implementing this enhancement could potentially cause more problems than it solves, not a soution yet. Thank you :)

@draganescu draganescu closed this Jan 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.