#27897 closed defect (bug) (fixed)
Theme preview empties sidebar on active theme
Reported by: | La Geek | Owned by: | 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)
Change History (21)
#2
@
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
@
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.
#6
@
11 years ago
gcorne identified L1424 and L1509 in widgets.php
to possibly have the wrong logic.
See https://irclogs.wordpress.org/chanlog.php?channel=wordpress-dev&day=2014-04-18&sort=asc#m836556
#7
follow-up:
↓ 8
@
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
#8
in reply to:
↑ 7
;
follow-up:
↓ 11
@
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
@
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 :)
#10
@
11 years ago
- Resolution worksforme deleted
- Status changed from closed to reopened
Patch has not been committed yet.
#11
in reply to:
↑ 8
@
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
@
11 years ago
- Owner set to nacin
- Resolution set to fixed
- Status changed from reopened to closed
In 28161:
#14
@
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
#17
@
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 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.