Twenty Thirteen: First image in featured gallery can be blurry if in landscape

[obenland]

​Reported in IRC, if a landscape oriented image happens to be the first image in a featured gallery, it appears blurry. Since the medium image size does not crop, the image is most likely not 300px high. This causes the image to be magnified and to appear blurry.

We previously discussed the issue of the best image size to use in IRC, I couldn't find the log in a reasonable amount of time though.

[knutsp]

Replying to obenland:

We previously discussed the issue of the best image size to use in IRC, I couldn't find the log in a reasonable amount of time though.

This? ​https://irclogs.wordpress.org/chanlog.php?channel=wordpress-dev&day=2013-05-23&sort=asc#m616985

#24307 seems related

[lancewillett]

I think we can detect "landscape" class value from core for the gallery image, and change the size to make sure it's not blurry.

[lancewillett]

I tried a lot of variations on layout changes to account for landscape images being 200 or so in height, at "medium" default media cropped size.

Nothing works well. :/

I'd like to consider -- bringing up to core leadership team -- re-instating the custom image size of 300x300. I think it's worth the extra file storage and overhead to make these galleries look great with good quality images.

[lancewillett]

After considering this a lot more, and weighing our options (a customized layout versus the overhead it requires) we've decided to simplify things. Going with a thumbnail grid, without a larger first image. Basically styling index and single exactly the same, for galleries.

This means removing twentythirteen_gallery_atts() and the CSS that changes the index layout for gallery post formats.

See also #23649 #23823 and #24307. Lots of IRC discussion, also, most recently ​here.

[lancewillett]

@obenland is going to work on a patch to get this ready.

[lancewillett]

Patch is a good start, but I think as I mentioned I think index and single should look the same. Centered, thumbs.

[lancewillett]

.2 removes more special styles for blog/search/index and ensures similar view for single view. I don't think we need the 1, 2, and 3 column max-width rules.

[obenland]

Replying to lancewillett:

.2 removes more special styles for blog/search/index and ensures similar view for single view.

I'm not sure I understand why index and single should look the same. They don't currently.

I don't think we need the 1, 2, and 3 column max-width rules.

We have to constrain them in width so galleries work with image sizes bigger than thumbnails.

[nacin]

I <3 all of the red in 24555.2.diff​.

[lancewillett]

Replying to obenland:

Replying to lancewillett:

.2 removes more special styles for blog/search/index and ensures similar view for single view.

I'm not sure I understand why index and single should look the same. They don't currently.

They don't currently and that's a bug in my opinion. Since we're showing a thumbnail grid now (not the big first image as before) they should look the same on both views.

Why would you want them to look different?

[rdall]

When the page is resized the images are shown differently on the home page then they are when it is resized on the single page. See Screenshot that is has both. Left is the front page right is the single post page. I agree with lance it is still a bug.

​http://cl.ly/image/1f3O0H3T2n3w

[nacin]

I'm +100 on 24555.2.diff​.

N.B. If anything, it'd be the single view that should have a bigger photo.

[rdall]

I removed the

.gallery-item:first-child 

which seems to be causing the layout problem I mentioned above. As this if my first submitted patch I hope I submitted that properly… 

[obenland]

Replying to lancewillett:

They don't currently and that's a bug in my opinion. Since we're showing a thumbnail grid now (not the big first image as before) they should look the same on both views.

Why would you want them to look different?

It would be a standardized look for all galleries on index, no matter the individual settings. I just thought the bigger first image was maybe not the only reason to have them different in the first place.

But I can get behind having them look the same on both single and index.

[lancewillett]

Replying to obenland:

But I can get behind having them look the same on both single and index.

:) -- let's do it and get this ticket closed. Want to test my patch? .2

[rdall]

Seem I applied only patch2 instead of both patches… Disregard my patch… 

[lancewillett]

Another pass, includes removing mobile and RTL styles

[lancewillett]

Minor fix to selector, props obenland

[lancewillett]

In 24720:

Twenty Thirteen: improve gallery styles to ensure both single and index views are displayed consistently. Closes #24555.