Making WordPress.org

Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#3142 closed defect (bug) (fixed)

Figcaption is hidden when the screenshot image height is bigger than 600px

Reported by: mirucon's profile Mirucon Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: Priority: normal
Component: Plugin Directory Keywords:
Cc:

Description

When I have a screenshot image that height is bigger than 600px, the bottom part of image is cut off, and even the caption is hidden.

https://wordpress.org/plugins/coldbox-addon/#screenshots

Attachments (1)

Screen Shot 2017-09-20 at 16.15.14.png (71.3 KB) - added by Mirucon 7 years ago.
Caption located on wrong place

Download all attachments as: .zip

Change History (3)

@Mirucon
7 years ago

Caption located on wrong place

#1 @SergeyBiryukov
7 years ago

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

In 5938:

Plugin Directory: Make sure tall screenshots fit into the slider, which is limited to 600px in height.

Fixes #3142.

#2 @aurovrata
7 years ago

wouldn't it be simpler to simply remove the css styling:

.plugin-screenshots .image-gallery-slides {
    max-height: 600px;
}

and let the screenshot image height vary? I think its easier to let users add any ratios (width/height) of screenshots rather than constrain them to landscape ratios.

Note: See TracTickets for help on using tickets.