Skip to content

fix: restore widgets theme support for WP Customizer#92

Open
WybeBosch wants to merge 1 commit into
mainfrom
fix/customizer-widgets-support
Open

fix: restore widgets theme support for WP Customizer#92
WybeBosch wants to merge 1 commit into
mainfrom
fix/customizer-widgets-support

Conversation

@WybeBosch
Copy link
Copy Markdown
Contributor

@WybeBosch WybeBosch commented Apr 9, 2026

Probleem : Wp Customizer werkte niet meer

image
  1. Op een bepaald moment tijdens initialisatie is de titel van een paneel "null", hier komt een warning
    voor.
  2. Maar sinds Acorn warnings converteert naar exceptions, crasht nu de pagina.
  3. Mogelijk was dit altijd al zo maar merkten we het niet omdat je voorheen enkel een warning kreeg.
  4. In development converteert Acorn dus warnings naar exceptions, maar in productie blijft het een
    warning. Daarom kon je tijdelijk het fixen door je WP_ENV te veranderen. (Al gingen dan wel opeens al
    je assets fout.)

Waarom werkte het vroeger wel?

Vroeger toen we nog footer deden via het "widgets menu" (038ec906 refactor: setup hooks) stond er een registerThemeSidebars() methode die register_sidebar() aanriep voor de footer:

#[Action('widgets_init')]
public function registerThemeSidebars(): void
{
    register_sidebar([
        'name' => __('Footer', 'sage'),
        'id'   => 'footer',
        ...
    ]);
}

register_sidebar() roept intern automatisch add_theme_support('widgets') aan.
Toen we dit weghaalde omdat we nu de footer doen vannuit de patronen (cfaa852b "remove sidebar widget" )
verdween dus ook impliciet de widgets support die gebruikt werd door de customizer.

De fix

In Setup.php expliciet add_theme_support('widgets') toevoegen. Dit zorgt ervoor dat de Customizer weer werkt zonder warnings, errors of WP_ENV shenanigans.

Het Widgets menu item onder Appearance is opnieuw verborgen via remove_submenu_page() in removeWidgetsMenuItem()

WordPress heeft wel aangegeven dat de Customizer in de toekomst depricated word, maar die dag is vandaag nog niet. So might as well put it back.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Restores WordPress Customizer functionality in the Sage theme (Brave stack) by explicitly enabling widget theme support, preventing PHP warnings (and Acorn warning-to-exception crashes) when the Customizer Widgets panel renders.

Changes:

  • Explicitly adds add_theme_support('widgets') during after_setup_theme.
  • Documents why widget support is required and why it previously disappeared (sidebar removal stopped implicitly enabling it).

@WybeBosch WybeBosch force-pushed the fix/customizer-widgets-support branch from e3b797b to ef6086b Compare April 9, 2026 08:17
@WybeBosch WybeBosch requested a review from Copilot April 9, 2026 08:18

This comment was marked as off-topic.

@WybeBosch WybeBosch force-pushed the fix/customizer-widgets-support branch from ef6086b to a656505 Compare April 9, 2026 08:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants