Skip to content

[FIX] web_responsive: app-menu-container hides open mail chat windows - #3640

Open
jp-sft wants to merge 1 commit into
OCA:19.0from
BRM-Teck:fix/app-menu-container-zindex-chat-window
Open

[FIX] web_responsive: app-menu-container hides open mail chat windows#3640
jp-sft wants to merge 1 commit into
OCA:19.0from
BRM-Teck:fix/app-menu-container-zindex-chat-window

Conversation

@jp-sft

@jp-sft jp-sft commented Aug 13, 2026

Copy link
Copy Markdown

Bug

.app-menu-container (the full-screen apps grid overlay, apps_menu.scss) sets z-index: 1024 !important. Mail's .o-mail-ChatWindow (core mail module) uses z-index: $zindex-sticky, which is 1020 in Bootstrap 5's default z-index scale.

Both elements are position: fixed panels that span overlapping regions of the viewport (the apps grid covers the full width from just below the navbar down; chat windows are anchored fixed-bottom). Since 1024 > 1020, opening the apps grid always paints over any already-open chat window, hiding it completely for as long as the apps grid stays open.

Fix

Lower .app-menu-container's z-index to $zindex-sticky - 1 (1019), so it still sits comfortably above ordinary page content, dropdowns ($zindex-dropdown = 1000) and this module's own .o_searchview_autocomplete (999), but no longer covers .o-mail-ChatWindow or anything else at the sticky tier.

To reproduce (before fix)

  1. Open a chat window (Discuss bubble in the bottom-right corner).
  2. Click the apps grid button in the navbar.
  3. The open chat window disappears behind the apps grid overlay until it is closed.

🤖 Generated with Claude Code

https://claude.ai/code/session_01BoYMgd2neGjAAx8N2xwYur

.app-menu-container (apps grid overlay) used z-index: 1024 !important,
which is higher than mail's .o-mail-ChatWindow z-index ($zindex-sticky,
1020 in Bootstrap 5's default scale). Both are `position: fixed` panels
that span overlapping screen regions, so opening the apps grid always
painted the fixed z-index:1024 overlay on top of any already-open chat
window, hiding it completely.

Lower the overlay's z-index to just below $zindex-sticky so it still
sits above ordinary page content and dropdowns, but no longer covers
chat windows (or any other sticky-tier UI at the same level).

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01BoYMgd2neGjAAx8N2xwYur
@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hi @SplashS, @Tardo,
some modules you are maintaining are being modified, check this out!

@CristianoMafraJunior CristianoMafraJunior left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants