Skip to content

[docs-infra] Fix layout shift toolbar#37693

Closed
oliviertassinari wants to merge 1 commit intomui:masterfrom
oliviertassinari:fix-layout-shift-docs
Closed

[docs-infra] Fix layout shift toolbar#37693
oliviertassinari wants to merge 1 commit intomui:masterfrom
oliviertassinari:fix-layout-shift-docs

Conversation

@oliviertassinari
Copy link
Member

@oliviertassinari oliviertassinari commented Jun 24, 2023

We changed the style of the demo toolbar in #37319 but we introduce a regression: CLS: https://web.dev/cls/. This effectively breaks the anchor links, e.g. open https://mui.com/material-ui/react-select/#controlling-the-open-state

actual ❌

Screenshot 2023-06-24 at 23 57 40

expected ✅

Screenshot 2023-06-24 at 23 57 49

@oliviertassinari oliviertassinari added type: bug It doesn't behave as expected. type: regression A bug, but worse, it used to behave as expected. scope: docs-infra Involves the docs-infra product (https://www.notion.so/mui-org/b9f676062eb94747b6768209f7751305). labels Jun 24, 2023
@mui-bot
Copy link

mui-bot commented Jun 24, 2023

Netlify deploy preview

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

Bundle size report

No bundle size changes

Generated by 🚫 dangerJS against cf652d4

Comment on lines -107 to -112
export function DemoToolbarFallback() {
const t = useTranslate();

return <Root aria-busy aria-label={t('demoToolbarLabel')} role="toolbar" />;
}

Copy link
Member Author

Choose a reason for hiding this comment

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

Dead code

@oliviertassinari
Copy link
Member Author

Continued in #37744

@oliviertassinari oliviertassinari deleted the fix-layout-shift-docs branch June 30, 2023 16:35
@oliviertassinari oliviertassinari removed the type: bug It doesn't behave as expected. label Apr 30, 2025
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: regression A bug, but worse, it used to behave as expected.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants