[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