[wp-trac] [WordPress Trac] #52148: CSS Optimization: Use shorthand 'border' value in wp-admin/css/admin-menu.css

WordPress Trac noreply at wordpress.org
Mon Feb 1 16:22:07 UTC 2021


#52148: CSS Optimization: Use shorthand 'border' value in wp-admin/css/admin-
menu.css
----------------------------------+----------------------------------------
 Reporter:  ankitmaru             |       Owner:  SergeyBiryukov
     Type:  enhancement           |      Status:  reviewing
 Priority:  normal                |   Milestone:  5.7
Component:  Administration        |     Version:  trunk
 Severity:  normal                |  Resolution:
 Keywords:  has-patch needs-      |     Focuses:  ui, css, coding-standards
  refresh                         |
----------------------------------+----------------------------------------

Comment (by sabernhardt):

 @ankitmaru Thanks for combining the patches and updating it in
 [attachment:"css_updated_52148.patch"].

 Please make two more changes:

 1. For the second change in customize-controls.css, remove the units with
 a zero value:
 {{{
 margin: 0 0 0 10px;
 }}}
 ([https://developer.wordpress.org/coding-standards/wordpress-coding-
 standards/css/#values see coding standards about zero value])

 2. Combine the 3 border properties in wp-includes/css/editor.css
 (comment:7)

 If you don't have time to update the patch today, I could do that.

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/52148#comment:21>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list