Skip to content

Conversation

@albozek
Copy link
Contributor

@albozek albozek commented Oct 23, 2025

🎫 Issue IBX-10848

Related PRs:

Description:

In addition to changes in design, changing current maximum z-index in the system from 99999 to 10300 after introducing integrated-help feature. The change is made in order to avoid excessively high indices in the project.
After the changes are implemented, the highest index will be set as follows:

  1. 10300 - notifications as elements with the highest z-index
  2. 10200 - help center trigger button/dialog
  3. 10100 - image editors/picker

Description:

For QA:

Documentation:

@albozek albozek changed the base branch from main to 4.6 October 23, 2025 14:00
@albozek albozek requested review from a team and Copilot October 24, 2025 11:48
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates the z-index value of notifications to ensure they appear above the help center button.

Key Changes:

  • Increased the z-index of the notification container from 50000 to 100010

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@pawlakadrian
Copy link
Contributor

pawlakadrian commented Oct 31, 2025

Still we have 2 situation to resolve:

  1. The notification is covering the button with the help icon, but it's still partially visible and unevenly covered.
Zrzut ekranu 2025-10-30 o 13 43 37
  1. This is the location selection view, where the "Discard" button is immediately covered by the icon.
Zrzut ekranu 2025-10-30 o 13 38 29

@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 3, 2025

@pawlakadrian
Copy link
Contributor

QA Approved on DXP Commerce 4.6.x-dev

@dew326 dew326 merged commit 09a7a45 into 4.6 Nov 4, 2025
28 checks passed
@dew326 dew326 deleted the IBX-10848 branch November 4, 2025 12:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants