[wp-trac] [WordPress Trac] #22834: Live Preview should map sidebars as done on activation

WordPress Trac noreply at wordpress.org
Wed Jul 31 22:36:13 UTC 2013

#22834: Live Preview should map sidebars as done on activation
 Reporter:  alexvorn2                 |       Owner:
     Type:  enhancement               |      Status:  new
 Priority:  normal                    |   Milestone:  Awaiting Review
Component:  Appearance                |     Version:  3.4
 Severity:  normal                    |  Resolution:
 Keywords:  needs-patch dev-feedback  |
Changes (by rmarks):

 * cc: rmarks@… (added)
 * keywords:  needs-patch => needs-patch dev-feedback


 I have looked over possibilities to fix this and think I'm close, but I
 need a little feedback please.  These changes are in /wp-includes/class-

 In {{{start_previewing_theme()}}}, I propose to add near all of the
 {{{update_option( 'theme_switched', $this->theme->stylesheet );}}}

 In {{{stop_previewing_theme()}}}, I propose to add
 {{{update_option( 'theme_switched', false );}}}

 I tested this code, but when I browse to /wp-
 admin/customize.php?theme=twentytwelve the first time, I do not see the
 sidebars.  When I refresh, I do see the sidebars.  I think I have the
 first update_option() call in the wrong place.  Am I on the right track?
 Can someone point me toward where this should be?

Ticket URL: <http://core.trac.wordpress.org/ticket/22834#comment:5>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software

More information about the wp-trac mailing list