Shift-click widget editing issues the Customizer

[hardeepasrani]

Hey folks,

I like the new Customizer API which allows us to add panels to the Customizer but it's also causing a simple problem with "Shift-click to edit this widget" option.

Shift-click feature is not working on the Customizer, unless you're editing the Widgets panel.

Thanks :)

[celloexpressions]

Confirmed. This is something that should be able to leverage the Customizer deep-linking functionality that westonruter is working on for 4.1 rather than all of the widgets-specific code it's currently using (which doesn't account for panels).

[celloexpressions]

No reason that this necessarily needs to be in 4.1, but it does depend on #28709 - we'll wait for a patch.

[westonruter]

Yeah, so once #28709 lands, as the patch exists right now, then the issue in this ticket can be resolved by first invoking (not final):

wp.customize.panel( 'widgets' ).toggle( true )

And actually, this should be automatic. So when something like this is called:

wp.customize.control( 'widget_text[123]' ).focus()

It should automatically cause the containing section and panel to also expand.

[westonruter]

This is now fixed in the ​most recent patch for #28709 (now on a ​GitHub PR). Namely, now WidgetControl.expandControlSection() is using the new API api.section( this.section() ).expand() which automatically expands the containing panel if it is not expanded.

[ocean90]

In 30102:

Improve/introduce Customizer JavaScript models for Controls, Sections, and Panels.

• Introduce models for panels and sections.
• Introduce API to expand and focus a control, section or panel.
• Allow deep-linking to panels, sections, and controls inside of the Customizer.
• Clean up accordion.js, removing all Customizer-specific logic.
• Add initial unit tests for wp.customize.Class in customize-base.js.

​https://make.wordpress.org/core/2014/10/27/toward-a-complete-javascript-api-for-the-customizer/ provides an overview of how to use the JavaScript API.

props westonruter, celloexpressions, ryankienstra.
see #28032, #28579, #28580, #28650, #28709, #29758.
fixes #29529.