WordPress.org

Make WordPress Core

Opened 2 months ago

Last modified 7 weeks ago

#47569 reviewing enhancement

Remove unnecessary !important from wp-admin/css/common.css

Reported by: foack Owned by: SergeyBiryukov
Milestone: 5.3 Priority: normal
Severity: normal Version:
Component: Administration Keywords: has-patch needs-testing needs-design-feedback
Focuses: ui, administration Cc:

Description

There are 21 !important statements in common.css, that sometimes simply overwrite CSS changes made in JS, overwrite other !important statements in the same file or other files. Overall, it doesn't help with clarity :)

Attachments (6)

common.css (69.0 KB) - added by foack 2 months ago.
47569.diff (12.3 KB) - added by foack 2 months ago.
updated version of postbox.js
47569-postbox.js.diff (12.2 KB) - added by foack 2 months ago.
Updated postbox.js
47569-2.diff (2.2 KB) - added by foack 2 months ago.
Updated common.css
47569-3.diff (2.8 KB) - added by foack 2 months ago.
Updated common-rtl.css
47569-postbox.js.2.diff (428 bytes) - added by foack 7 weeks ago.
postbox.js without formatting changes.

Download all attachments as: .zip

Change History (14)

@foack
2 months ago

@foack
2 months ago

updated version of postbox.js

#1 in reply to: ↑ description @foack
2 months ago

Replying to foack:

There are 21 !important statements in common.css, that sometimes simply overwrite CSS changes made in JS, overwrite other !important statements in the same file or other files. Overall, it doesn't help with clarity :)

Please ignore the uploaded common.css and 47569.diff - those are here forever but don't actually contribute to clarity and do not move the ticket further.

#2 @foack
2 months ago

Updated postbox.js and the statement that added overflow:hidden to #wpbody-content, so that the !important statement on overflow:visible in common.css line 29 can be removed.

@foack
2 months ago

Updated postbox.js

@foack
2 months ago

Updated common.css

@foack
2 months ago

Updated common-rtl.css

#3 @foack
2 months ago

  • Keywords has-patch needs-testing needs-design-feedback added; needs-patch removed
  • Type changed from defect (bug) to enhancement

#4 @SergeyBiryukov
8 weeks ago

  • Component changed from General to Administration

#6 @SergeyBiryukov
7 weeks ago

  • Keywords needs-refresh added

Hi @foack, welcome to WordPress Trac! Thanks for the ticket and the patches.

Could you upload a version of 47569-postbox.js.diff without the formatting changes? It's hard to see what's actually changed there.

@foack
7 weeks ago

postbox.js without formatting changes.

#7 @foack
7 weeks ago

Sure thing @SergeyBiryukov, sorry that I forgot to disable my auto-formatting the first time. WCEU was too exciting :)

Let me know if the new diff helps to clarify!

#8 @SergeyBiryukov
7 weeks ago

  • Keywords needs-refresh removed
  • Milestone changed from Awaiting Review to 5.3
  • Owner changed from foack to SergeyBiryukov
  • Status changed from assigned to reviewing
Note: See TracTickets for help on using tickets.