Make WordPress Core

Opened 11 years ago

Closed 11 years ago

Last modified 7 years ago

#27897 closed defect (bug) (fixed)

Theme preview empties sidebar on active theme

Reported by: la-geek's profile La Geek Owned by: nacin's profile nacin
Milestone: 3.9.1 Priority: high
Severity: critical Version: 3.9
Component: Customize Keywords: has-patch
Focuses: Cc:

Description

How to reproduce:

1.) Insert at least one widget into a sidebar. You can insert one with Appearance -> Widgets or Appearance -> Customize -> Widgets. This makes no difference in its (bug) result.

2.) Go to Appearance -> Themes. Click "Live Preview" on an inactive theme. Wait until you see the Live Preview. Now click on left top the button Cancel.

3.) Go to frontend and check, if the widget is still there. It is not. The sidebar is empty. In my test this concerns also older widgets, imo, it concerns all widgets.

Attachments (1)

27897.patch (780 bytes) - added by ocean90 11 years ago.

Download all attachments as: .zip

Change History (21)

#1 @jynmeyer
11 years ago

This was a major issue for me.
I had a whole sidebar of various widgets- some that were customized with code- and after going in to explore what the new customize features were and then canceling out, the whole sidebar was gone.
Most were dumped into the "Unused Widgets with saved settings" box however the customized settings were gone and I had to redo everything.

Im afraid to go into the Customize page now.

#2 @ocean90
11 years ago

  • Keywords needs-patch added
  • Milestone changed from Awaiting Review to 3.9.1
  • Priority changed from normal to high

Confirmed.

This ticket was mentioned in IRC in #wordpress-dev by lancewillett. View the logs.


11 years ago

#4 @obenland
11 years ago

Tested it a bit on WordPress.com, and it looks like widgets do not disappear, if the active and the previewed theme use the same sidebar IDs. Like when testing with newer default themes.

#5 @westonruter
11 years ago

  • Component changed from Themes to Appearance

#7 follow-up: @westonruter
11 years ago

Agree with gcorne. The logic needs to be inverted, as when in the customizer, no change should be made yet. Introduced in r28124 for #27767.

For background on the introduction of the 'customize' value for the $theme_changed argument, see https://irclogs.wordpress.org/chanlog.php?channel=wordpress-dev&day=2014-04-14&sort=asc#m832791

@ocean90
11 years ago

#8 in reply to: ↑ 7 ; follow-up: @ocean90
11 years ago

  • Keywords has-patch added; needs-patch removed
  • Severity changed from normal to critical

Replying to westonruter:

For background on the introduction of the 'customize' value for the $theme_changed argument, see https://irclogs.wordpress.org/chanlog.php?channel=wordpress-dev&day=2014-04-14&sort=asc#m832791

Was surprised as I saw this, because there was no patch with 'customize' value which I have tested. Next time we should create a new patch, test again and then commit, when we handle which such a complex system as the Widget Customizer is.

#9 @La Geek
11 years ago

  • Resolution set to worksforme
  • Status changed from new to closed

@Test
27897.patch works like a charme! Can switch themes, theme previews, activate and deactivate now and all widgets remain :)

Last edited 11 years ago by La Geek (previous) (diff)

#10 @chipbennett
11 years ago

  • Resolution worksforme deleted
  • Status changed from closed to reopened

Patch has not been committed yet.

#11 in reply to: ↑ 8 @westonruter
11 years ago

Replying to ocean90:

Was surprised as I saw this, because there was no patch with 'customize' value which I have tested. Next time we should create a new patch, test again and then commit, when we handle which such a complex system as the Widget Customizer is.

Absolutely. The change to allow retrieve_widgets('customize') happened right before commit, as noted above. I also should have double-checked what got committed in r28124 since it didn't come from a tested patch.

#12 @nacin
11 years ago

  • Owner set to nacin
  • Resolution set to fixed
  • Status changed from reopened to closed

In 28161:

Fix logic inversion in [28124] to ensure widgets stay with themes during a theme switch.

fixes #27897.

#13 @nacin
11 years ago

In 28162:

Fix logic inversion in [28124] to ensure widgets stay with themes during a theme switch.

Merges [28161] to the 3.9 branch.

props gcorne.
fixes #27897.

#14 @westonruter
11 years ago

I've been seeing a few reports of the widgets being all deactivated upon upgrade to 3.9 even though they say they never visited the Customizer: https://wordpress.org/support/topic/all-widgets-are-gone-after-update-to-39?replies=3

#15 @westonruter
11 years ago

#28060 was marked as a duplicate.

#16 @nacin
11 years ago

#22834 was marked as a duplicate.

#17 @westonruter
11 years ago

In #28183 it is reported that the upgrade to 3.9.1 causes the widgets to vanish from the sidebars. I've asked for further information.

This ticket was mentioned in Slack in #core-customize by westonruter. View the logs.


8 years ago

This ticket was mentioned in Slack in #core by obenland. View the logs.


7 years ago

This ticket was mentioned in Slack in #core by westonruter. View the logs.


7 years ago

Note: See TracTickets for help on using tickets.