[wp-trac] [WordPress Trac] #35941: Eliminate WP_Customize_Site_Logo_Control

WordPress Trac noreply at wordpress.org
Sun Feb 28 06:44:30 UTC 2016


#35941: Eliminate WP_Customize_Site_Logo_Control
------------------------------+--------------------
 Reporter:  celloexpressions  |       Owner:
     Type:  defect (bug)      |      Status:  new
 Priority:  normal            |   Milestone:  4.5
Component:  Customize         |     Version:  trunk
 Severity:  normal            |  Resolution:
 Keywords:  has-patch         |     Focuses:
------------------------------+--------------------
Changes (by celloexpressions):

 * keywords:  needs-patch => has-patch


Comment:

 [attachment:35941.diff] eliminates the unnecessary
 `WP_Customize_Site_Logo_Control`. Note that there are no UI changes
 whatsoever (once #35542 is also committed). For developers/the customizer
 API, this clarifies that `WP_Customize_Media_Control` should be used for
 basic media such as image controls that don't require specialized UI for a
 specific purpose.

 The patch also adds two changes that benefit core beyond this cleanup:
 - All of the media controls now send media attachment information to the
 preview when a new file is selected, facilitating the use of postMessage
 setting transport without additional ajax calls or custom controls.
 - There was a double-bottom-margin on all the media controls that I
 noticed when cleaning up the CSS; this is now removed so that the intended
 rhythm of control spacing is maintained (by using the margin on the
 control, and removing the excessive one on the last element within the
 control).

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


More information about the wp-trac mailing list