Layout: Show vertical alignment toolbar with allowSwitching enabled

[Infinite-Null]

Fixes: #67016

What?

Enable vertical alignment toolbar to display when block layout has allowSwitching enabled and flex layout is active.

Why?

Currently, the vertical alignment toolbar is hidden when allowSwitching is set to true, even when flex layout is selected.

Testing Instructions

1. Go to any post
2. Add any block which have:

"supports": {
    "layout": {
        "allowSwitching": true
    }
}

3. Add the block and test the block toolbar
   

Screenshots or screencast

Before Fix:

After Fix:

[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: Infinite-Null <[email protected]>

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

[Mamaduka]

I think this early return condition needs to be fixed. It should probably return null when both allowJustification and allowVerticalAlignment are false.

@ntsekouras, can you confirm? I think you worked on the initial implementation of this feature.

[ntsekouras]

It's been so long and even by looking at old PRs, I'm still not 100% sure why this check was added. If I remember correctly at that time we explicitly didn't want to show the justification controls in inspector controls too. Having said that, back then there were fewer layout props and there was no vertical alignment.

The allowJustification prop was added specifically for constraint layout but in docs says that can also be applied to flex and defaults to true. There are no checks for it now in flex layout, so I guess we need to take into account both in inspectorControls and toolBarControls.

So, I'm not sure whether the allowSwitching check is still needed, but the other checks (allowVerticalAlignment, allowJustification) should be added.

I'll cc @tellthemachines if she has more context.