#42337 closed defect (bug) (fixed)
Unregistering the Publish Settings section prevents changes from being published
Reported by: | westonruter | Owned by: | westonruter |
---|---|---|---|
Milestone: | 4.9 | Priority: | normal |
Severity: | normal | Version: | 4.9 |
Component: | Customize | Keywords: | has-patch needs-testing |
Focuses: | Cc: |
Description
In plugins like AMP and Customizer Blank Slate, a stripped-down version of the Customizer is presented where all except for the plugin's own panels, sections, and controls are registered. However, when the Publish Settings section is unregistered then there is no way to change the previous drafted changeset to publish, since the section doesn't exist. The gear icon likewise is presented but it never becomes enabled.
Attachments (4)
Change History (14)
#1
@
7 years ago
- Keywords has-patch needs-testing added; needs-patch removed
First stab at fix is 42337.0.diff where we can register the publish_settings
section in JS to ensure that it exists. There was no need to register it in PHP anyway, as no controls were being registered that referenced it anyway.
#4
@
7 years ago
Applied the patch to 4.9-beta4-42021, but I can't see any changes, no php or js errors are reported. Meaning, the option wheel is still disabled.
#5
@
7 years ago
@PerS Try clearing your cache? With the AMP plugin I see amp-customizer-gear-icon-enabled.png and amp-customizer-publish-settings-expanded.png when I open AMP Customizer with changes previously drafted.
#6
@
7 years ago
It might be my use of blank slate, I've added blank slate as a mu-plugin: https://soderlind.no/resetting-customizer-wordpress-theme-blank-slate/
#7
@
7 years ago
In 42337.1.diff, unconditionally register core types for panels, sections, and controls so that the required control templates are available when all customize_register
actions are removed.
https://github.com/xwp/wordpress-develop/pull/294