[Menu Item | ui5-webcomponents-react |1.29.20]:

[rv97]

Bug Description

When the MenuItem has the busy attribute to true, it shows the busy indicator. But the alignment of the busy indicator is misaligned.

Here is the codesandbox reproducing the issue:
https://codesandbox.io/p/devbox/nice-williamson-forked-3w5p9z

The issue is reproducible with the following versions:
"@ui5/webcomponents": "1.24.24",
"@ui5/webcomponents-fiori": "1.24.24",
"@ui5/webcomponents-icons": "1.24.24",
"@ui5/webcomponents-react": "1.29.20",

In the previous version we had (before the upgrade), the issue was NOT reproducible:
"@ui5/webcomponents": "1.24.13",
"@ui5/webcomponents-fiori": "1.24.13",
"@ui5/webcomponents-icons": "1.24.13",
"@ui5/webcomponents-react": "1.29.14",

Affected Component

MenuItem

Expected Behaviour

The busy indicator should be aligned to the center of the item

Isolated Example

https://codesandbox.io/p/devbox/nice-williamson-forked-3w5p9z

Steps to Reproduce

1. Create a menu with menuItem component
2. Set the busy attribute to true and busyDelay to 0

Log Output, Stack Trace or Screenshots

No response

Priority

Very High

UI5 Web Components Version

1.24.24

Browser

Chrome

Operating System

Mac OS

Additional Context

No response

Organization

SAP

Declaration

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

[StefanDimitrov04]

Hello @UI5/ui5-team-ctr-bal,

The alignment of the busy indicator inside the MenuItem appears to be incorrect.
The issue is reproducible in the following example.
Could you please take a look?

Best Regards,
Stefan

[ilhan007]

Hello @UI5/ui5-team-ctr-bal,

The issue is reproducible the version 2 as well and can be seen in our docs:
https://ui5.github.io/webcomponents/components/Menu/#handling-focus-behaviour-when-adding-items-dynamically

Note: Downport to 1.24 will be also needed whenever the issue is fixed in main.

[ilhan007]

Hello @rv97, most of the consumers has upgraded to v2,
is this coming from SAP SAC, could you specify? Thanks!

[rv97]

@ilhan007 Yeah this is coming from SAP SAC. Also can you take this on very high priority because we have updated the app to use this new version and released. After upgrading we are noticing this issue and it looks very bad on the UI

[ilhan007]

Hello @rv97 , thanks for the input.

I will leave the @UI5/ui5-team-ctr-bal to lead further communication, to require more information if needed and decide on the priority