[wp-meta] [Making WordPress.org] #4888: Caption overflow in screenshots of plugin single page

Making WordPress.org noreply at wordpress.org
Mon Dec 9 01:48:56 UTC 2019


#4888: Caption overflow in screenshots of plugin single page
------------------------------+-----------------------------
 Reporter:  YordanSoares      |       Owner:  dufresnesteven
     Type:  defect            |      Status:  accepted
 Priority:  normal            |   Milestone:
Component:  Plugin Directory  |  Resolution:
 Keywords:                    |
------------------------------+-----------------------------
Changes (by dufresnesteven):

 * owner:  (none) => dufresnesteven
 * status:  new => accepted


Comment:

 In [attachment:"4888.diff"]:

 This diff sets the slide parent to `display: table-cell` and its parent to
 `display: table` . ([[https://caniuse.com/#feat=mdn-
 css_properties_display_table_values|Browser Support Chart]])

 **Why was it happening?**
 We have hardcoded values for the slide `max-height: 600px` and the image
 `max-height: 550px`. This was done to make sure large images didn't extend
 too far down the page and make the slider hard to use. If the captions
 would be larger than `50px`, it would hide under the bottom bounds of the
 slide.

 **Considerations**
 - I considered a JS solution where we could get references to the image
 and the caption to obtain the height and adjust the `max-height` property
 for the image but the JS (in part because its React) makes it a bit messy
 to gracefully store and read the DOM references.
 - We are also looking to replace the slider completely, so I didn't feel
 it necessary to invest too much time. See #2273.


 **Browser Test Matrix via BrowserStack**
 ||= Environment =|| Detail || Result || Notes ||
 || Win 10/8.1   ||  FF 68   ||  ✅ || ||
 || Win 10/8.1   ||  Opera 55   ||  ✅ || ||
 || Win 10/8.1   ||  Edge 15   ||  ✅ || ||
 || Win 10/8.1  ||  IE 11   ||  ❌ || Not a regression. Already exists. ||
 || Win 8  ||  IE 9/10   ||  ❌ || Not a regression. Already exists. ||
 || MacOS Catalina || Safari 13     ||  ✅ || ||
 || MacOS Sierra || Safari 10     ||  ✅ ||  ||
 || MacOS El Capitan || Safari 9     ||  ✅ ||  ||
 || iPhone 6 || Safari   ||  ✅ ||  ||
 || Android Galaxy 9  || Chrome   ||  ✅ ||  ||
 || Android Galaxy 6  || UC Browser   ||  ✅ ||  ||

-- 
Ticket URL: <https://meta.trac.wordpress.org/ticket/4888#comment:2>
Making WordPress.org <https://meta.trac.wordpress.org/>
Making WordPress.org


More information about the wp-meta mailing list