Adjust themes screenshot in the themes directory

[kafleg]

At first, check this ​link. You can see many themes with different sizes of screenshots. The reason behind this is, theme authors submitted themes with invalid screenshot sizes.

As per the ​theme guidelines, the valid/proper size of the screenshot should be in a 4:3 ratio. Many theme authors follow this and submit themes with proper screenshot size. However, many authors submit /update themes with improper size. Because of that, the themes directory listing page looks bad.

Theme reviewers used to check screenshot size by manual process. But after the new version of the ​Theme Check plugin is added, it's handled by the plugin itself. But, the Theme Check plugin won't work for child themes. And still, there is a chance that the theme author submits themes with an invalid screenshot size.

The current themes screenshot requirement is applied to make the themes page good. There are no other reasons for this. If the theme directory can be updated to display the correct size, the requirement is not needed.

[dufresnesteven]

The current themes screenshot requirement is applied to make the themes page good. There are no other reasons for this. If the theme directory can be updated to display the correct size, the requirement is not needed.

I think it's better to remove this as a requirement and update .org.

[poena]

I have tested the patch and it works well as long as support for IE11 is not expected.
IE11 does not have support for aspect-ratio.

[dd32]

​aspect-ratio is only supported in browsers as of January 2021, there do exist ​some polyfills that might be worth looking into as an option here too.

Before a restriction is lifted on dotorg though, some confirmation that no fixes are required within core for mis-sized images would be good (ie. on the Install themes page, not necessarily the theme listing page)

[poena]

You are right, I think there was one view that was not double checked,
the image in the customizer preview is not displayed correctly.

[poena]

I'd consider this customizer view low priority but I put the ticket in the 5.9 milestone.
​https://core.trac.wordpress.org/ticket/54188

[dufresnesteven]

Don't use aspect-ratio but instead its generated polyfill which supports more browsers.

[dufresnesteven]

With the latest patch applied.

[dufresnesteven]

In 11876:

wporg-themes: Adjust themes screenshot in the themes directory.

Props kafleg, poena, dd32.
Fixes #5834.

[dufresnesteven]

Hmm... Changes were working in Sandbox but not on the live site. Investigating...

[dufresnesteven]

In 11877:

wporg-themes: Break theme css cache.

See: #5834.