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

It is possible to edit wp_navigation type posts via Appearance -> Navigation menus when a non-FSE theme is enabled #36258

Closed
anton-vlasenko opened this issue Nov 5, 2021 · 0 comments · Fixed by #36257
Assignees
Labels
[Block] Navigation Affects the Navigation Block [Type] Bug An existing feature does not function as intended

Comments

@anton-vlasenko
Copy link
Contributor

anton-vlasenko commented Nov 5, 2021

Description

We must not allow to manage wp_navigation type posts when the current theme doesn's support FSE.

Step-by-step reproduction instructions

  1. Enable a non-FSE theme like Twenty Twenty-One.
  2. Check the Appearance pane.

Expected behaviour

There must be no Navigation Menu menu item.

Actual behaviour

Navigation Menu menu item is present when a non-FSE theme is enabled.

Screenshots or screen recording (optional)

WordPress information

  • WordPress version: "5.9-alpha-51272-src"
  • Gutenberg version: "11.8.0" (trunk)
  • Are all plugins except Gutenberg deactivated? "Yes"
  • Are you using a default theme (e.g. Twenty Twenty-One)? "Yes"

Device information

  • Device:
  • Operating system:
  • WordPress app version:
@anton-vlasenko anton-vlasenko added the Mobile App - i.e. Android or iOS Native mobile impl of the block editor. (Note: used in scripts, ping mobile folks to change) label Nov 5, 2021
@anton-vlasenko anton-vlasenko added this to the Gutenberg 11.9 milestone Nov 5, 2021
@anton-vlasenko anton-vlasenko added this to Inbox in Navigation editor via automation Nov 5, 2021
@anton-vlasenko anton-vlasenko self-assigned this Nov 5, 2021
@anton-vlasenko anton-vlasenko added [Block] Navigation Affects the Navigation Block [Type] Bug An existing feature does not function as intended and removed Mobile App - i.e. Android or iOS Native mobile impl of the block editor. (Note: used in scripts, ping mobile folks to change) labels Nov 5, 2021
Navigation editor automation moved this from Inbox to Done Nov 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Block] Navigation Affects the Navigation Block [Type] Bug An existing feature does not function as intended
Projects
No open projects
1 participant