Opened 10 years ago
Closed 10 years ago
#30888 closed defect (bug) (fixed)
Manage in Customizer lacks return param for Widgets admin page
Reported by: | westonruter | Owned by: | tywayne |
---|---|---|---|
Milestone: | 4.2 | Priority: | normal |
Severity: | normal | Version: | 4.1 |
Component: | Customize | Keywords: | good-first-bug has-patch |
Focuses: | Cc: |
Description
In #28032 we introduced a “Manage in Customizer” link on the Widgets admin page. This link opens the Customizer and auto-opens the Widgets panel (#28650). In #25457 we added return
URL param for Customizer links in the WP Admin so that when the user clicks “Close” in the Customizer they are returned to the URL that they were on before being navigated to the Customizer. This was not done for the “Manage in Customizer” link on the Widgets admin page. When clicking this link and then clicking “Close” in the Customizer, the user should be taken back to the Widgets admin page as opposed to the Themes admin page.
Attachments (1)
Change History (11)
#2
@
10 years ago
The widgets admin page is the only place where you can fully delete widgets (remove them from the inactive widgets sidebar). Also, I don't think the widgets admin page is going anywhere. I believe the decision to go with a “Manage in Customizer” link at the top instead of going with an nag notice was because of this as well, that we're not trying to direct users away from the widgets admin page (not yet, anyway).
#3
@
10 years ago
After reading through the discussion on #25457 I don't see Widgets being an exception to the return after closing approach. Patch added in case that is the consensus.
This ticket was mentioned in Slack in #core by drew. View the logs.
10 years ago
#7
@
10 years ago
- Keywords needs-patch added; 2nd-opinion has-patch removed
- Owner set to tywayne
- Status changed from new to assigned
Patch still applies and works as expected, though I would question why we're pointing the user back to the REQUEST_URI
instead of explicitly pointing them to widgets.php.
#8
@
10 years ago
REQUEST_URI
was used to ensure a return to whatever view of widgets.php they actually came from.
For example, let's say a theme or plugin provided a filtered view of widgets.php, showing only widget areas for specific layouts/templates using a query string (iThemes Builder does exactly that.)
If we explicitly point them to widgets.php, they'd lose the filtered view they came from.
Actually, I thought that was intentional. The point of that link is to get them to start using the Customizer instead of the admin page, so why should we take them back to the admin page when they're done?
Thinking more broadly, I'm starting to think that the front page on the front end might be a more appropriate default return location that themes in the admin, although I don't think we're quite ready for that yet.