[wp-trac] [WordPress Trac] #37873: Consolidate customizer CSS

WordPress Trac noreply at wordpress.org
Sat Sep 17 19:40:32 UTC 2016


#37873: Consolidate customizer CSS
----------------------------------------+---------------------------
 Reporter:  helen                       |       Owner:  mrahmadawais
     Type:  defect (bug)                |      Status:  assigned
 Priority:  normal                      |   Milestone:  4.7
Component:  Customize                   |     Version:
 Severity:  minor                       |  Resolution:
 Keywords:  good-first-bug needs-patch  |     Focuses:  ui
----------------------------------------+---------------------------
Changes (by celloexpressions):

 * owner:   => mrahmadawais
 * status:  new => assigned


Comment:

 Thanks for volunteering to take this on @mrahmadawais!

 I think the first step is to research duplication of styles and
 duplication of selectors, and to evaluate the potential areas where things
 could be consolidated. We're probably best off adding any new common
 classes as opposed to removing any, to avoid JS issues and minimize back-
 compat concerns. Any changes will have compatibility implications due to
 the ability to extend the core UI with custom panels, sections, and
 controls. In general, we should move away from the `accordion-` classes as
 those are from UI that is no longer used.

 I believe that `customize-nav-menus.css` was largely duplicated from
 `customize-widgets.css` and intended to all be merged into `customize-
 controls.css` when the menu customizer plugin merged, but that didn't
 happen with all of the other work that was ongoing at that time.

 There are also numerous pending patches that would be impacted by these
 changes. Depending on how much we want to clean up, it may be best to work
 on an actual patch during 4.7 RC and commit early for 4.8.

 Also, previously and related: #30618, #36733, #30713.

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


More information about the wp-trac mailing list