WordPress.org

Making WordPress.org

Opened 3 months ago

Last modified 3 weeks ago

#5834 new defect

Adjust themes screenshot in the themes directory

Reported by: kafleg Owned by:
Milestone: Priority: normal
Component: Theme Directory Keywords: has-patch
Cc:

Description

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.

Attachments (3)

5834.patch (486 bytes) - added by dufresnesteven 3 months ago.
Screenshot on 2021-08-03 at 09-28-12.png (142.2 KB) - added by dufresnesteven 3 months ago.
Screenshot on 2021-08-03 at 09-28-29.png (1.4 MB) - added by dufresnesteven 3 months ago.

Download all attachments as: .zip

Change History (10)

#1 @dufresnesteven
3 months ago

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.

#2 @poena
3 weeks ago

  • Keywords has-patch added; needs-patch removed

#3 @poena
3 weeks ago

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.

This ticket was mentioned in Slack in #meta by poena. View the logs.


3 weeks ago

#5 @dd32
3 weeks ago

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)

#6 @poena
3 weeks ago

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

#7 @poena
3 weeks ago

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

Note: See TracTickets for help on using tickets.