Opened 6 years ago
Closed 5 years ago
#46530 closed defect (bug) (fixed)
CSS line-height values should be unitless - media.css
Reported by: | ianbelanger | Owned by: | ianbelanger |
---|---|---|---|
Milestone: | 5.3 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Media | Keywords: | has-patch needs-testing good-first-bug |
Focuses: | ui, administration, coding-standards | Cc: |
Description
As outlined in #44643, CSS line-height values should be unitless unless necessary to be defined as a specific pixel value
. It was suggested that we break up 44643 by stylesheet in order to better track them.
This ticket covers much of wp-admin/css/media.css
, see patch notes below
Note: The patch was tested only on a Windows machine in Chrome, Firefox, IE11 and Edge. As per https://core.trac.wordpress.org/ticket/44643#comment:23 it should probably be tested on other OS's and supported browsers.
Patch Notes:
Unchanged line-height values that use units (I could not find a good unitless solution that would work across all browsers and admin pages)
line 1255 - .edit-attachment-frame .edit-media-header .right:before, .edit-attachment-frame .edit-media-header .left:before
Unable to find selectors in the DOM
line 162 - .media-item .progress (couldn't keep it on the page long enough to inspect it) line 187 - .media-item .progress .percent (same as above)
Attachments (2)
Change History (8)
#3
@
5 years ago
I tested attached patch successfully without any error in Ubuntu 18.04.2 LTS with Google Chrome Version 73.0.3683.75 and Firefox Version 65.0.1
#4
@
5 years ago
/* This does not belong here, it has nothing to do with media */ #poststuff .inside .the-tagcloud { … }
Originally introduced in [9518], accidentally ended up in Media section after the reorganization in [16136].
Same for these styles for the update form in wp-admin/update-core.php
:
ul#dismissed-updates { display: none; } form.upgrade { margin-top: 8px; } form.upgrade .hint { font-style: italic; font-size: 85%; margin: -0.5em 0 2em 0; }
They ended up accidentally duplicated in both Notifications and Media sections as part of the same reorganization. They still exist in wp-admin/css/common.css, the duplicates can be removed.
HI @ianbelanger. Similar to #46529, I think this would be a nice update.