[wp-trac] [WordPress Trac] #31334: Customizer JS API should handle container removal from document

WordPress Trac noreply at wordpress.org
Sat May 23 06:59:07 UTC 2015


#31334: Customizer JS API should handle container removal from document
---------------------------------+-----------------------------------------
 Reporter:  Aniruddh             |       Owner:
     Type:  enhancement          |      Status:  new
 Priority:  normal               |   Milestone:  Future Release
Component:  Customize            |     Version:  4.1
 Severity:  normal               |  Resolution:
 Keywords:  has-patch needs-     |     Focuses:  javascript, administration
  refresh                        |
---------------------------------+-----------------------------------------
Changes (by celloexpressions):

 * keywords:   => has-patch needs-refresh
 * milestone:  Awaiting Review => Future Release


Comment:

 @Aniruddh: can you update your patch to:
 - Use tabs instead of spaces for indentation
 - Add brackets around the if statement, per the coding standards (even
 though it's one-line)
 - Please patch relative to the root of WordPress, ie the patch should be
 something like `src/wp-admin/js/customize-base.css`

 @westonruter: this looks good to me and makes sense for the API to handle,
 do you see any issues?

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


More information about the wp-trac mailing list