ui5-webcomponents icon indicating copy to clipboard operation
ui5-webcomponents copied to clipboard

[Menu & MenuItem | 1.14.0]: The Dynamic Menu is not working on the second sub-level

Open rv97 opened this issue 2 years ago • 12 comments

Bug Description

When the main menu has two levels of sub-menu where the second level sub-menu should be rendered dynamically, then the loading indicator is stuck and does not render the content automatically. We need to go one level back and then open the sub-menu to show the rendered content. The same works in the case of the first-level sub-menu.

Affected Component

Menu, MenuItem

Expected Behaviour

How many ever sub-levels are there, the busy indicator should be gone automatically and render the content.

Isolated Example

https://codesandbox.io/s/prod-bird-mjtv35?file=/src/App.js

Steps to Reproduce

  1. Open the sandbox link shared.
  2. Click on "Show Menu"
  3. Open the "New File" sub-menu.
  4. Again open the "Open" sub-menu ...

You can observe that the sub-menu inside a sub-menu does not render the content automatically.

Log Output, Stack Trace or Screenshots

No response

Priority

High

UI5 Web Components Version

1.14.0

Browser

Chrome

Operating System

MacOS

Additional Context

No response

Organization

SAP Labs

Declaration

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

rv97 avatar Jul 31 '23 06:07 rv97

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

This issue is for us.

Best regards, Boyan

unazko avatar Jul 31 '23 07:07 unazko

Hi @unazko ,

Is there any update on this?

rv97 avatar Nov 27 '23 04:11 rv97

Hi @rv97,

Apologies for the late reply. We've got stuck with merging the previously related PR. Currently there is a new related PR, which will most probably fix this issue. We're currently testing if the issue will be resolved with the PR.

Best regards, Boyan

unazko avatar Feb 12 '24 09:02 unazko

Hello @rv97,

I've managed to fully test the provided sample and unfortunately the previously linked PR's aren't going to fix this issue: I suspect you're already using the following workaround: https://codesandbox.io/p/sandbox/boring-solomon-92qknd?file=%2Fsrc%2FApp.js

There is a workaround at least. We're going to continue debugging this behavior.

Best regards, Boyan

unazko avatar Mar 05 '24 08:03 unazko

Hi @unazko,

I'm from @rv97' team working on this issue. The workaround you proposed is not working in our case. We have a complex reusable component which is used to render all the menus across the entire application. Introducing the workaround in our application brings in a lot of code refactoring and unnecessary errors. Hence, we request the UI5 Web Components Team to check if it is possible to fix this issue in the Menu component itself.

Thanks & Regards, Shubham.

shubhamnazare avatar Apr 30 '24 06:04 shubhamnazare

Hi @rv97 & @shubhamnazare,

We're actively addressing your concern and will update you shortly. Thank you for your patience!

Regards, Diana

DMihaylova avatar May 08 '24 14:05 DMihaylova

Hi @DMihaylova ,

Thanks for the update.

Regards, Vignesh R

rv97 avatar May 09 '24 04:05 rv97