Skip to content

Conversation

@BenOsodrac
Copy link
Contributor

This pull request addresses z-index layering issues for popups and overflow menus to ensure proper stacking behavior when popups are open. The main focus is on improving how popups and overflow menus interact visually, especially when multiple UI layers are present.

Layering and z-index adjustments:

  • Changed the value of the --osui-popup-layer CSS variable from var(--layer-global-instant-interaction) to var(--layer-global-off-canvas) in _root.scss, aligning popups with other off-canvas elements for consistent stacking.
  • Added a new CSS rule in _overflowmenu.scss that increases the z-index of .osui-overflow-menu__balloon.osui-balloon--is-open when a popup is open on the same screen, using the :has(.show-as-popup) selector to apply the layer exception.

Checklist

  • tested locally
  • documented the code
  • clean all warnings and errors of eslint
  • requires changes in OutSystems (if so, provide a module with changes)
  • requires new sample page in OutSystems (if so, provide a module with changes)

@BenOsodrac BenOsodrac requested a review from a team as a code owner October 31, 2025 13:46
@BenOsodrac BenOsodrac added bug Something isn't working chore labels Oct 31, 2025
@cssecautomation
Copy link

cssecautomation commented Oct 31, 2025

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@cssecautomation
Copy link

cssecautomation commented Oct 31, 2025

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@sonarqubecloud
Copy link

@BenOsodrac BenOsodrac merged commit a3c010c into dev Nov 4, 2025
18 checks passed
@BenOsodrac BenOsodrac deleted the ROU-12311 branch November 4, 2025 11:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working chore

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants