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

Tiled gallery block: don't use full size image, use large image instead #29544

Closed
simison opened this issue Dec 18, 2018 · 0 comments
Closed
Labels
[Block] Tiled Gallery [Goal] Gutenberg Working towards full integration with Gutenberg [Pri] High Address as soon as possible after BLOCKER issues

Comments

@simison
Copy link
Member

simison commented Dec 18, 2018

Currently, if you upload 5mb image to the gallery, it'll use that 5mb image for the tiny square.

Photonizing URLs will help (#29495) but if the block should work without it, then we should load large image instead.

Here's a PR to Gutenberg Gallery block that uses "large" image as a base instead of full images: WordPress/gutenberg#11682

Additionally, for smaller tiles "medium" size might be enough so there should be some logic to figure out which image to load for the tile. This isn't really needed if we just use Photon.

@simison simison added [Pri] High Address as soon as possible after BLOCKER issues [Goal] Gutenberg Working towards full integration with Gutenberg [Block] Tiled Gallery labels Dec 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Block] Tiled Gallery [Goal] Gutenberg Working towards full integration with Gutenberg [Pri] High Address as soon as possible after BLOCKER issues
Projects
None yet
Development

No branches or pull requests

1 participant