Skip to content

Comments

[docs-infra] Reapply Cookie Banner with Design Fixes#47744

Open
dav-is wants to merge 12 commits intomui:masterfrom
dav-is:davis/reapply-cookie-banner
Open

[docs-infra] Reapply Cookie Banner with Design Fixes#47744
dav-is wants to merge 12 commits intomui:masterfrom
dav-is:davis/reapply-cookie-banner

Conversation

@dav-is
Copy link
Member

@dav-is dav-is commented Feb 10, 2026

@dav-is dav-is added type: enhancement It’s an improvement, but we can’t make up our mind whether it's a bug fix or a new feature. scope: docs-infra Involves the docs-infra product (https://www.notion.so/mui-org/b9f676062eb94747b6768209f7751305). website Pages that are not documentation-related, marketing-focused. labels Feb 10, 2026
@mui-bot
Copy link

mui-bot commented Feb 10, 2026

Netlify deploy preview

https://deploy-preview-47744--material-ui.netlify.app/

Bundle size report

Bundle Parsed size Gzip size
@mui/material 0B(0.00%) 0B(0.00%)
@mui/lab 0B(0.00%) 0B(0.00%)
@mui/system 0B(0.00%) 0B(0.00%)
@mui/utils 0B(0.00%) 0B(0.00%)

Details of bundle changes

Generated by 🚫 dangerJS against 5e68ed3

@dav-is dav-is marked this pull request as ready for review February 10, 2026 17:55
@Janpot
Copy link
Member

Janpot commented Feb 11, 2026

I've updated #47721. I propose we wrap the cookiebanner in this theme provider so that we inherit most of the styling from the branding theme instead of trying to recreate it. This ensures maximum compatibility with the them and prevents drift over time.

@dav-is
Copy link
Member Author

dav-is commented Feb 11, 2026

@Janpot ok, I applied your PR and it seems to be working

Copy link
Member

@Janpot Janpot left a comment

Choose a reason for hiding this comment

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

Ok from my end.

Copy link
Contributor

@noraleonte noraleonte left a comment

Choose a reason for hiding this comment

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

From what I see the dark theme and the docs theme are applying correctly now

Leaving just a few nits about the original copy and some small design tweaks. After that, LGTM 👌

dav-is and others added 3 commits February 16, 2026 10:42
Co-authored-by: Nora <72460825+noraleonte@users.noreply.github.com>
Signed-off-by: Connor Davis <mail@connordav.is>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope: docs-infra Involves the docs-infra product (https://www.notion.so/mui-org/b9f676062eb94747b6768209f7751305). type: enhancement It’s an improvement, but we can’t make up our mind whether it's a bug fix or a new feature. website Pages that are not documentation-related, marketing-focused.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants