Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Site Editor: Remove the Navigation Menus Sidebar #71278

Closed
Copons opened this issue Dec 16, 2022 · 5 comments
Closed

Site Editor: Remove the Navigation Menus Sidebar #71278

Copons opened this issue Dec 16, 2022 · 5 comments
Assignees
Labels
[Feature] Full Site Editor The site editor. [Status] Blocked / Hold T-Rex [Type] Enhancement Changes to an existing feature — removing, adding, or changing parts of it

Comments

@Copons
Copy link
Contributor

Copons commented Dec 16, 2022

The "Navigation Menus" sidebar is in a rough beta state and is more confusing than helpful.

Screenshot 2022-12-16 at 17 39 38

We should remove it for the time being. It will likely show up again in a better state somewhere else in the Gutenberg UI (for example in the dark left sidebar).

Originally requested in p1670010686600519-slack-C048CUFRGFQ and pc4f5j-2Fb-p2#comment-3228

@Copons Copons added [Type] Enhancement Changes to an existing feature — removing, adding, or changing parts of it [Goal] Full Site Editing T-Rex [Feature] Full Site Editor The site editor. and removed [Goal] Full Site Editing labels Dec 16, 2022
@Copons
Copy link
Contributor Author

Copons commented Dec 21, 2022

This looks much more complicated than anticipated, as the Navigation Menus sidebar is not registered via hook, but it depends on the process.env.IS_GUTENBERG_PLUGIN feature flag (which is inherently enabled on Dotcom and, as you can imagine, is not something we can disable haphazardly).

https://github.com/WordPress/gutenberg/blob/c05d7c5535a38ae18661c8c7e99c1aec72720722/packages/edit-site/src/components/sidebar-edit-mode/index.js#L67-L75

Furthermore, the Navigation Menus is connected to the Navigation block's "Open list view" button (currently re-labelled "Open navigation list view" in the current Gutenberg trunk).

Screenshot 2022-12-21 at 14 16 44

The functionality is undergoing radical changes with the off-canvas navigation editor experiment (__experimentalEnableOffCanvasNavigationEditor) that, if I understand it correctly, should open the navigation menu in the block's setting sidebar.

WordPress/gutenberg#46335

Given that, in this instance, the feature flag changed from plugin to experimental:

- if ( process.env.IS_GUTENBERG_PLUGIN )
+ if ( window?.__experimentalEnableOffCanvasNavigationEditor === true )

I have the feeling that the Navigation Menus will be either removed altogether or gated by a different feature flag, because otherwise I'm not quite sure what is the point of it anymore.

@scruffian I see that you are closely involved with the new approach. Do you have any insider intel about the fate of the Navigation Menus sidebar?

@scruffian
Copy link
Member

What we could do is just hide this feature behind the window?.__experimentalEnableOffCanvasNavigationEditor. I created a PR for this here: WordPress/gutenberg#46710

@Copons
Copy link
Contributor Author

Copons commented Dec 21, 2022

@scruffian nice! I wasn't sure if the two features were related or if the inspector view would make the nav sidebar obsolete. Thanks for opening the PR! 🙇‍♂️

@Copons Copons self-assigned this Dec 21, 2022
@Copons
Copy link
Contributor Author

Copons commented Dec 21, 2022

WordPress/gutenberg#46710 has been merged and should be included in 14.9. Dotcom can wait. 🙂

@Copons
Copy link
Contributor Author

Copons commented Jan 18, 2023

Circling back, this has been fixed a while ago when Gutenberg 14.9 was merged on Dotcom! 🎉
Thanks again @scruffian!

@Copons Copons closed this as completed Jan 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Full Site Editor The site editor. [Status] Blocked / Hold T-Rex [Type] Enhancement Changes to an existing feature — removing, adding, or changing parts of it
Projects
None yet
Development

No branches or pull requests

2 participants