Opened 4 years ago
Closed 4 years ago
#51685 closed defect (bug) (fixed)
Media Element CSS Issue : Some layout display crop
Reported by: | ravipatel | Owned by: | sabernhardt |
---|---|---|---|
Milestone: | 5.6 | Priority: | normal |
Severity: | normal | Version: | 5.6 |
Component: | Media | Keywords: | has-patch |
Focuses: | ui, css | Cc: |
Attachments (4)
Change History (16)
This ticket was mentioned in Slack in #core by ravi. View the logs.
4 years ago
#2
@
4 years ago
@ravipatel I can't reproduce the issue on Chrome/OSX. Which browser/OS are you using?
#3
@
4 years ago
@audrasjb
i have checked both Firefox and chrome (Version 86.0.4240.183). Both browser on same issue
1920x1080 - Screen resolution
#4
@
4 years ago
Hi there, thanks for the ticket.
Just noting that MediaElement.js is an external library, any suggested changes to its files should be submitted upstream: https://github.com/mediaelement/mediaelement.
If the fix is included in a future version of MediaElement.js, it will be in WordPress core as well as part of updating the library, there is no need to have an extra Trac ticket for that.
If the issue only exists in WordPress admin, perhaps some of the admin CSS should be adjusted instead.
#5
@
4 years ago
- Milestone changed from Awaiting Review to 5.6
Seems related to [49343], moving to the milestone for investigation.
#6
@
4 years ago
- Keywords needs-refresh added
- Owner set to sabernhardt
- Status changed from new to assigned
Yes, I see that in Firefox/Windows with trunk but not 5.5.2. [49343] overrides content-box
on .mejs-time
.
I'd rather not edit the height value, but I can check if there's a better option than
.wp-core-ui .mejs-time { box-sizing: content-box; }
#7
@
4 years ago
- Keywords needs-refresh removed
Caused by both the box-sizing: border-box
and a hidden overflow, Windows browsers Firefox, Chrome and Edge (not IE) cut off the bottom of the time value on the Media attachment page and in the Attachment Details dialog.
51685.1.patch resets the time container to content-box
, to match the front end.
I also considered showing the overflow instead, but then the horizontal spacing is off in the Attachment Details dialog.
Media element css changes