Skip to content

[18.0][FIX] web_widget_x2many_2d_matrix: fix sticky header/column overlap#3404

Merged
OCA-git-bot merged 1 commit into
OCA:18.0from
ForgeFlow:18.0-imp-web_widget_x2many_2d_matrix
Jan 8, 2026
Merged

[18.0][FIX] web_widget_x2many_2d_matrix: fix sticky header/column overlap#3404
OCA-git-bot merged 1 commit into
OCA:18.0from
ForgeFlow:18.0-imp-web_widget_x2many_2d_matrix

Conversation

@RicardCForgeFlow

Copy link
Copy Markdown
Contributor

Problem:
When scrolling in the matrix widget, the content overlaps with the sticky headers (days), footers (totals), and the first column (names) because they are transparent by default.
image
image

Solution:
Added a background colour to all position: sticky elements in the web_widget_x2many_2d_matrix SCSS to ensure they properly hide the scrolling content behind them. Used standard Odoo colour variables ($o-gray-100 and $o-view-background-color) for consistency.
image

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hi @JasminSForgeFlow, @hbrunn, @DavidJForgeFlow,
some modules you are maintaining are being modified, check this out!

@JoanSForgeFlow

Copy link
Copy Markdown
Contributor

Love it! Thank you @RicardCForgeFlow

@LoisRForgeFlow LoisRForgeFlow left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

image

@legalsylvain

Copy link
Copy Markdown
Contributor

/ocabot merge patch

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 18.0-ocabot-merge-pr-3404-by-legalsylvain-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 05eeb35 into OCA:18.0 Jan 8, 2026
9 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at ff7aa1e. Thanks a lot for contributing to OCA. ❤️

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.

6 participants