[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 17:51:38 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 ankitmaru):

 I have fixed the point no. 2, Can you please share more info about point
 no. 1?
 Replying to [comment:21 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:22>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list