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

Dara: Featured Image srcset issue #3

Closed
laurelfulford opened this issue Feb 24, 2018 · 4 comments
Closed

Dara: Featured Image srcset issue #3

laurelfulford opened this issue Feb 24, 2018 · 4 comments
Labels
Milestone

Comments

@laurelfulford
Copy link
Contributor

laurelfulford commented Feb 24, 2018

Originally reported by @senff in 4348-wpcom-themes:

The Featured Image on the home page is being cropped inconsistently. On page load it is cropped, but when switching to iPad view, and then back to normal view, it is not cropped anymore. Even after a refresh, it stays uncropped.

Steps to replicate

  1. Go here: ​https://jswbaileylaw.com/

  2. The Featured Image is a bit cropped (you can not see the boats at the bottom of the image, first screenshot). Refreshing the page will not change this.

  1. Open the inspector into iPad view. The boats are now visible at the bottom (second screenshot)

  1. Close inspector panel. The window is back to the original size, but now the boats are visible. Refreshing the page will "reset" that (third screenshot)

  1. Open the site again in a new tab. Now the boats are gone again even though the window is the exact same size as the other tab.

Testing

I tried to replicate this on a test site, but I can not seem to reproduce it. So I'm not sure if it's a bug or just a quirky edge case. Also, it only appears on Chrome/Windows, no issues on Mac.

@laurelfulford laurelfulford added this to the Dara milestone Feb 24, 2018
@laurelfulford
Copy link
Contributor Author

After investigating, @sixhours had the following comments:

... the correct image with the appropriate crop for that page is here: ​https://jswbaileylaw.files.wordpress.com/2017/01/0221-annapolis-2.jpg?w=1180&h=435&crop=1

This is reflected in the theme's post thumbnail size for this page:

// Full-width and grid page template
add_image_size( 'dara-page-thumbnail', 1180, 435, true );

For me (Chrome on Mac), the image is first shown un-cropped, then is cropped properly after loading the Inspector.

I don't know why the image is showing un-cropped at any given time. There's nothing in the theme that's changing the crop of the image save for the declared thumbnail size above. I'm noticing the same thing happens on the user's site when testing Sela (via Theme Butler), which tells me this isn't a theme-specific issue.

We've had some luck with users re-uploading the image in question when Photon quirks like this pop up. Please have the user try that.

@sixhours
Copy link
Contributor

I'm inclined to wontfix this; it's a bit of an edge case (happens after a window resize) and is fixed by a refresh. The theme's code is working as intended, and it's not specific to Dara.

@alaczek
Copy link
Contributor

alaczek commented Mar 5, 2018

I'm for wontfixing this one as well.

@sixhours
Copy link
Contributor

sixhours commented Mar 5, 2018

I can't reproduce this any more, either. Wontfixing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants