Skip to content

sap.m.plugins.UploadSetwithTable has wrong z-index #4244

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
rubentuider opened this issue Mar 24, 2025 · 5 comments
Open

sap.m.plugins.UploadSetwithTable has wrong z-index #4244

rubentuider opened this issue Mar 24, 2025 · 5 comments

Comments

@rubentuider
Copy link

What is the expected behavior?

That the UploadSetwithTable is "behind" the busy z-index

What is the current behavior?

When the dialog is set to busy, the dialog gets a busy-layer
Since the UploadSetwithTable has an own z-index (which is 2), the table-header is above the busy-layer

How we can reproduce the issue?

OpenUI5 version

1.134.0

Operating System

Win11

Browser

Google Chrome 134.0.6998.118 (Official Build) (64-bit)

Log Output / Stack Trace

@s-todorova
Copy link
Contributor

Hello @rubentuider ,

Thank you for sharing this finding. I've created an internal incident DINC0451599. The status of the issue will be updated here in GitHub.

Regards,
Siyana

@saurabhkr7
Copy link

Hello @rubentuider,

I am working this incident.
As I was following the given steps to reproduce the issue, and landed on this

Image

here i checked by pressing the buttons but not able to open any dialog,
can you please help me to reproduce the issue?

regards,
Saurabh

@rubentuider
Copy link
Author

Hi @saurabhkr7 ,

it is important to cd com.myorg.mypp and do npm start inside this folder. Plus ensure that you checked out branch fix/uploadTable.

Then the app should look like this:
Image

Regards,
Ruben

@AshwinKumarBolar
Copy link
Contributor

Hello @rubentuider,

The problem here is not on the UploadSetwithTable Plugin rather on the sap.m.Table which is consumed in the application.
The internally created incident: DINC0451599 has been moved to the Table component. Colleagues from Table will revert back on their findings here.

Thanks,
Ashwin

@boghyon
Copy link
Contributor

boghyon commented Apr 9, 2025

Some more examples: https://plnkr.co/edit/aCyB5pG7rju2xiPf which shows the same issue also when using 1.71+.
Setting the entire Page busy (contentOnlyBusy="false") shows no issue unlike when setting only the Panel or Popup/Dialog busy. Also the cursor on Switch changes if the theme is sap_horizon* with 1.108+.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants