WordPress.org

Making WordPress.org

Opened 6 weeks ago

Last modified 4 days ago

#4499 reopened defect

Prevent tall screenshots overflowing container

Reported by: dd32 Owned by: dd32
Milestone: Priority: normal
Component: Theme Directory Keywords:
Cc:

Description

Sometimes theme authors accidentally add screenshots which are too tall for the theme directory, see the below slack conversation for an example, we should limit the height of screenshots.

https://wordpress.slack.com/archives/C02RP4Y3K/p1559876587250600

Attachments (1)

themes-2-across.jpg (104.9 KB) - added by SergeyBiryukov 4 days ago.

Download all attachments as: .zip

Change History (5)

#1 @dd32
6 weeks ago

  • Owner set to dd32
  • Resolution set to fixed
  • Status changed from new to closed

In 8932:

Theme Directory: Styles: Prevent tall screenshots overflowing the container.

Fixes #4499.

#2 @dd32
6 weeks ago

In 8933:

Theme Directory: Cache bump after r8932.

See #4499.

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


4 days ago

#4 @SergeyBiryukov
4 days ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

As reported by @joyously, this caused issues with stretching the screenshot images on mobile views (see the screenshot above). Per Slack discussion, reopening to either revert or find a better CSS fix.

Note: See TracTickets for help on using tickets.