Opening/Closing the 'Add Items' slideout in the Customizer Menu editor loses the check icon

[garrett-eclipse]

Hello,

While testing 46830 I came across a separate issue I'll open here. The checkbox to indicate an item (page/post) is in the menu gets reverted to the + add icon if you click 'All Items' button to open and close the select menu item drawer.

It would be nice if the checkbox icon was persistent to continue to indicate to the user what items are selected in the menu.

Thank you

[garrett-eclipse]

Gif to illustrate the lose of state on menu items with the check reverting to the plus icon when the 'Add Items' drawer is open/closed.

[dlh]

Hey @garrett-eclipse,

Could you elaborate a bit on the preferred behavior? Is the suggestion to keep the items checked for as long as the Customizer is open? As long as the current changeset is unpublished (and therefore, potentially, across Customizer sessions)? Something else?

[garrett-eclipse]

Hi @dlh in my opinion I feel it would be beneficial when the 'Add Items' panel is opened it does a check on what's in the current menu and sets the checkmarks for all those items. In that way it doesn't matter if it's the first time opening customizer or if you're in the middle of working with it those checks would always indicate what's currently included in the menu. It could possibly just be tied to the open action on 'Add Items' and run the check then that way it's there on first open as well as if you close and reopen it.

I hope that clarifies,
Thank you
P.S. #46830 is related in that it affects those checks when no more items remain in the menu.

[dlh]

Thanks @garrett-eclipse! To my ears this sounds closer to enhancement than bug, but I'll leave that call to you.

[garrett-eclipse]

Thanks @dlh I'm fine with it being addressed as an enhancement, I appreciate your review.

[garrett-eclipse]

As I've been playing with Twenty Twenty I found another instance it would be nice to have the check applied is when Starter Content is loaded. In that case items can already be in the menu so would be good that on load they get checked.

[celloexpressions]

The add menu items panel is generally not designed to account for tracking items present in a given menu. The checkmark hints at that but was originally intended mostly as a secondary validation that an item was successfully added. That would be a nice enhancement, but some things to keep in mind for implementing:

• The same add-items panel is used for all menus, so the checkmarks would need to sync to the current active menu section
• The available menu item accordions lazy-load additional items; this process would need to re-check for items in a menu
• Should items also be shown as checked in the available menu item search section?