[menu]: when using Menu in Table on mobile devices app crashes

[rvsia]

Describe the bug

When using a Menu component in Table on mobile devices, the app is crashing:

Error
Web component processed too many times this task, max allowed is: 10
▼ 2 stack frames were expanded.
    at RenderQueue.process (https://prk2z7.codesandbox.io/node_modules/
ui5/webcomponents-base/dist/RenderQueue.js:45:15
    at eval (https://prk2z7.codesandbox.io/node_modules/
ui5/webcomponents-base/dist/Render.js:84:34

Isolated Example

https://codesandbox.io/s/smoosh-tdd-cywz6k

Reproduction steps

1. Open the example
2. Open the preview in a new window
3. Set the window to responsive mode (simulate mobile devices)
4. Click on menu

Expected Behaviour

Menu should be open

Screenshots or Videos

`

UI5 Web Components for React Version

latest

UI5 Web Components Version

latest

Browser

Chrome, Edge

Operating System

MacOS

Additional Context

• This will be probably a webcomponents issue but I do not know how to reproduce the issue there

Relevant log output

No response

Declaration

• I’m not disclosing any internal or sensitive information.

[Lukas742]

Hi @rvsia

Thanks for reporting! I'll forward this issue to our UI5 Web Components Colleagues as the affected components are developed in their repository.

Also, it's not recommended rendering modals or popups inside other components. You can find out more about this here. In this case the issue is also occurring when the Menu is rendered outside, but keeping this in mind will help avoid future problems.

---

Hi colleagues,
the issue is reproducible using ui5-webcomponents without our wrapper: https://codesandbox.io/s/ui5-webcomponents-forked-3zzmdn?file=/index.html

You can follow the steps provided by @rvsia with this codeSandbox as well.

[rvsia]

@Lukas742

Also, it's not recommended rendering modals or popups inside other components.

Thank you for pointing this up, we forget to use the portal here as it is working well for us on normal screens. In other places where we encounter issues, we use Portal. 👍

[nnaydenow]

Hi @SAP/ui5-webcomponents-topic-b,

Could you please check this issue? It's reproducible with native ui5-webcomponets as well.
https://codesandbox.io/s/ui5-webcomponents-forked-k8fklj?file=/index.html

[Lukas742]

Hi colleagues,
as the linked issue above shows, the error is also thrown when a menu is used outside of a Table. I've edited my codeSandbox from above to include this as well: https://codesandbox.io/s/ui5-webcomponents-forked-3zzmdn?file=/index.html

[TobiasKoehler01]

Hi colleagues,
we have unfortunately the same problem and using the menu in a live project and now need to hold back a new feature that uses the menu component. Any news on this?