fix(NcActions): hotfix for custom children

[ShGKme]

☑️ Resolves

• Hotfix for [NcActions] doesn't support non-direct NcAction* usage #5171
• Particularly a regression of fix(NcActions): keyboard navigation #5153

In NcActions we check what NcAction* components were passed as direct children. But some apps (Text, Mail) pass NcAction* not as direct children of NcActions, but as a sub-child.

<NcActions>
  <MyUserActionsList /> // renders <NcActionButton />
</NcActions>

The current NcActions implementation doesn't fully support this.

• In past it resulted in ignoring inline prop and incorrect render of single actions
• After mentioned PR it was completely broken:
  1. Menu is rendered
  2. There are no direct NcAction* interactive children (they are sub-children)
  3. Menu is considered a tooltip
  4. It should close on blur
  5. After opening, it focuses on an interactive element, causing blur and closing.

Unfortunately, in Vue 2 it is not possible to easily fix. When render function is executed, children are not rendered yet, so we don't know if they are to render NcAction* components or not.

This PR adds a hotfix that prevents component from being unusable. If children are not allowed custom components - consider menu a dialog with the default focus trap.

So the component works like it worked in the past. Yet it still has bad accessibility in this case.

🖼️ Screenshots

Docs

🏚️ Before	🏡 After

🚧 Tasks

• Document NCActions children limitations
• Add a hotfix - make NcActions be considered a semantic dialog if there are custom

🏁 Checklist

• ⛑️ Tests are included or are not applicable
• 📘 Component documentation has been extended, updated or is not applicable

[raimund-schluessler]

These custom wrappers for NcAction* are a pain. I know they are necessary sometimes, but they really interfere with how NcActions works. And it got worse with the focus trap and a11y stuff.

However, the hotfix brings back the functionality for now, so 👍

I just had a few nitpicks for the language 🙂

[Antreesy]

Tested with styleguide only

[susnux]

Please fix DCO and the fixup commments

[ShGKme]

Please fix DCO and the fixup commments

fixup commits are not supposed to be merged without squash

[susnux]

fixup commits are not supposed to be merged without squash

Thats what I meant: Squash the commits and add the sign off^^

[ShGKme]

/backport to next