Navigation Submenu: Add Layout Support

[Rishit30G]

What?

Part of #43248

Why?

The block was missing the layout support

How?

The PR adds the following to the block.json of the block to support the 'Layout'

 "supports": {
    "layout": true
 }

Testing Instructions

• Open the editor
• Insert the Menu block
• Add a menu and then create a submenu for it
• Select the submenu block and see the right panel
• Observe that the 'Layout' section is present in the settings tab in the right panel

Screenshots or screencast

Screen.Recording.2025-02-18.at.8.53.18.AM.mov

[github-actions]

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: Rishit30G <[email protected]>
Co-authored-by: carolinan <[email protected]>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

[carolinan]

I have tested the PR with the navigation block only, not the page list.
I don't think that this is the right solution, or I am not even sure what this PR is trying to solve; why does the submenu need layout support? This question needs to be answered.

Here are some notes and questions that I have from testing the PR:
The options increases the complexity of a block that is already difficult to work with.
I like the idea of being able to align the text of the menu link in the navigation block, but I don't think it should be limited to submenus.
Why can I enter a wide width option, when the submenu can never be wide?
It is difficult to understand what the content width option does, because it does not change the width of the submenu.
The design on the front does not match what I see in the editor.
If I enable "Inner blocks use content width" the submenu link text is centered by default, but neither the center, left, or right aligned options are working on the front of the site.
On the front, the arrow icon is miss aligned.

Editor:

Front:

[Rishit30G]

Hi @carolinan

Thanks for sharing the feedback and apologies for the PR if it's not appropriate enough,

The PR comes from the comment shared here: #43248 (comment)

The options increases the complexity of a block that is already difficult to work with.
I like the idea of being able to align the text of the menu link in the navigation block, but I don't think it should be limited to submenus.
Why can I enter a wide width option, when the submenu can never be wide?
It is difficult to understand what the content width option does, because it does not change the width of the submenu.
The design on the front does not match what I see in the editor.
If I enable "Inner blocks use content width" the submenu link text is centered by default, but neither the center, left, or right aligned options are working on the front of the site.
On the front, the arrow icon is miss aligned.

Thanks for this, to me based on the above points it looks like that we should not include the layout option for this menu
Should I go ahead a close the PR ?

[carolinan]

We can keep it open for a bit since others may have something to add.