Twenty Thirteen: display issues if gallery images are smaller than 300x300

[professor99]

WordPress 3.6-beta2-24227

Browser: Only tested on Firefox 20

Gallery excerpts on the home page are magnified.

Also missing captions on both excerpt and gallery page.
This may be addressed by #23584 "closed defect (bug) (fixed)
Twenty Thirteen: Gallery captions are hidden" but I'm not sure if captions were still meant to be hidden or displayed.

On the gallery page the centered images look out of place. Much better to displayed them from the left side as done for 2012.

See attached files for screenshots.

Also attached 2012 gallery screenshot for comparison which is the same on both the excerpt and the gallery page.

Andy Bruin

[professor99]

Gallery 2013 Excerpt

[professor99]

Gallery 2013 Main

[professor99]

Gallery 2012

[professor99]

Forgot to add image details

Zen Size:98px * 98px, Title: test, Caption: zen
ZipIcon Size: 32px * 32px, Title: ico_zip, Caption: howdy

[kovshenin]

Reproduced with trunk. If the gallery images are less than 300x300, the tiled gallery resizing looks really weird.

[lancewillett]

This seems pretty edge case, but we should try to address anyway.

[lancewillett]

I'd also like to relook at how we load all large versions for the gallery images on index. We should try a different technique that only loads the first image in large size, and the rest thumbnail.

[lancewillett]

Patch is a quick attempt at getting the first image in a gallery, and outputting "large" for only that one image. Not working fully ... but it's the idea.

[lancewillett]

Alternate idea using a custom image size

[obenland]

I think 24307.patch​ would set a bad precedent.

I do like .1, though we have to be aware that galleries that were created prior to switching to Twenty Thirteen, will fall back to the 'full' image size.

[obenland]

Centers galleries on index pages.

[obenland]

Patch builds on lancewillett's patch, cropping the image and serving only thumbnails on mobile views, where we display galleries in two columns of thumbnail sized images.

[lancewillett]

Oh, @obenland -- pro tip for the "new Trac" -- if you use .patch it'll look nicer in the diff view. I found it out via Westi. :)

[lancewillett]

In 24285:

Twenty Thirteen: ensure galleries are displayed correctly (centered) on both single and indexed views, props obenland. See #24307.

[tyshkev]

removing these two rules seems to solve this issue

[lancewillett]

@tyshkev But does it change the layout? The designer's intent was for the first image in the gallery to be 300px by 300 px.

[lancewillett]

In 24333:

Twenty Thirteen: create new image size for galleries, at 300 x 300 so that "large" image size doesn't load for a resulting thumbnail on index views. See #24307.

[lancewillett]

In 24338:

Twenty Thirteen: revert adding new image size for galleries., Using existing medium size instead, and thumbnail for mobile views. See #24307.

[lancewillett]

Last item to work on before closing this ticket -- how to handle case of images smaller than 300? I can't think of a way with pure CSS. We might just have to live with it.

Opinions?

[lancewillett]

I've thought about this a bit more, doesn't seem a simple way to avoid using the special styles for images smaller than 300. It'll just be a known issue, which can be avoided by 1) not using small images or 2) using a standard post for the gallery.