Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 3, 2025

Fix z

Original prompt

This section details on the original issue you should resolve

<issue_title>[BUG] Popup animation container has a higher z-index than Dialog overlay</issue_title>
<issue_description>## Bug Description

When a component with an opened popup is rendered on the page and a Dialog gets opened, the popup appears above the Dialog's overlay.

Steps to Reproduce

Steps to reproduce the behavior:

  1. Open the following StackBlitz example - https://stackblitz.com/edit/angular-twtmmlsp-9n6ae68y
  2. Click the "Open dialog" button.

Expected Behavior

The popup should not appear above the Dialog's overlay.

Actual Behavior

The k-animation-container element has a higher z-index than the Dialog's overlay, which makes it appear above the overlay.

Screenshots/Videos

Image

Environment Information

Affected Theme(s)

  • All themes

Affected Component(s)

Dialog component and components that open a popup (e.g., SplitButton, DropDownList, etc.).

Affected Suites

  • Kendo UI for Angular
  • Kendo UI for React</issue_description>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

@github-actions
Copy link

github-actions bot commented Nov 3, 2025

Packages Report

core default classic bootstrap material fluent utils html
Size 29.20 KB (0.0%) 769.42 KB (0.0%) 768.12 KB (0.0%) 782.98 KB (0.0%) 916.14 KB (0.0%) 1181.71 KB (0.0%) 625.19 KB (0.0%) 77.78 MB (0.0%)
Gzip Size 4.85 KB (0.0%) 99.40 KB (0.0%) 99.64 KB (0.0%) 101.38 KB (0.0%) 112.33 KB (0.0%) 136.81 KB (0.0%) 52.16 KB (0.0%) 15.95 MB (0.0%)
Compile Time 0.7 s (-22.2%🔽) 3.1 s (0.0%) 3.1 s (0.0%) 3.3 s (0.0%) 3.2 s (0.0%) 3.7 s (0.0%) 2.8 s (0.0%) 8.4 s (0.0%)

Copilot AI changed the title [WIP] [BUG] Popup animation container has a higher z-index than Dialog overlay fix(dialog): correct z-index hierarchy for modal dialogs and popups Nov 3, 2025
Copilot AI requested a review from Juveniel November 3, 2025 12:10
Copilot finished work on behalf of Juveniel November 3, 2025 12:10
@Juveniel
Copy link
Contributor

Juveniel commented Nov 3, 2025

@copilot do not add the /tests folder to .gitignore

Copy link
Contributor Author

Copilot AI commented Nov 3, 2025

@copilot do not add the /tests folder to .gitignore

Reverted the .gitignore change in commit aea0e10. The tests/ directory is no longer excluded from version control.

Copilot finished work on behalf of Juveniel November 3, 2025 18:24
@Juveniel
Copy link
Contributor

Juveniel commented Nov 4, 2025

@copilot the generated visual tests show an issue with the ContextMenu that is now not visible over a Window component. Investigate.

Copy link
Contributor Author

Copilot AI commented Nov 4, 2025

@copilot the generated visual tests show an issue with the ContextMenu that is now not visible over a Window component. Investigate.

Fixed in commit 7ddc959. Changed ContextMenu popup z-index from k-z-index("popup") (11500) to k-z-index("modal", 1) (12001), ensuring it appears above Windows (12000) and Dialogs (12000).

Z-index hierarchy after fix:

  • ContextMenu popup: 12001 (modal + 1) ✓
  • Window/Dialog: 12000 (modal layer)
  • Regular popups: 11500 (popup layer)

ContextMenu correctly appearing above Window

Copilot AI changed the title fix(dialog): correct z-index hierarchy for modal dialogs and popups fix(dialog): correct z-index hierarchy for modals, popups, and context menus Nov 4, 2025
Copilot finished work on behalf of Juveniel November 4, 2025 13:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Popup animation container has a higher z-index than Dialog overlay

3 participants