Skip to content

[docs-infra] Render footer in SSR#39710

Merged
oliviertassinari merged 1 commit intomui:masterfrom
oliviertassinari:footer-ssr
Nov 2, 2023
Merged

[docs-infra] Render footer in SSR#39710
oliviertassinari merged 1 commit intomui:masterfrom
oliviertassinari:footer-ssr

Conversation

@oliviertassinari
Copy link
Member

@oliviertassinari oliviertassinari commented Nov 2, 2023

I have noticed this while I was looking at #39704.

  • Fix layout shift in https://mui.com/material-ui/getting-started/design-resources/ when loading the page. Overall, it feels like it makes more sense to SSR the footer. In the future, we might be able to make it a Server component. The alternative fix would be to simply force the scrollbar to reserve space.
  • Fix the animation by adding a <div>. Before
Screen.Recording.2023-11-02.at.02.02.04.mov

Preview: https://deploy-preview-39710--material-ui.netlify.app/material-ui/getting-started/design-resources/

@oliviertassinari oliviertassinari added type: bug It doesn't behave as expected. scope: docs-infra Involves the docs-infra product (https://www.notion.so/mui-org/b9f676062eb94747b6768209f7751305). labels Nov 2, 2023
@mui-bot
Copy link

mui-bot commented Nov 2, 2023

Netlify deploy preview

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

Bundle size report

No bundle size changes (Toolpad)
No bundle size changes

Generated by 🚫 dangerJS against bd5d8f3

Copy link
Collaborator

@danilo-leal danilo-leal left a comment

Choose a reason for hiding this comment

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

Much better — thanks for tackling it! I had noticed this, too, but didn't know how to fix it. 🤙

@oliviertassinari oliviertassinari merged commit 0adb435 into mui:master Nov 2, 2023
@oliviertassinari oliviertassinari deleted the footer-ssr branch November 2, 2023 19:33
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: bug It doesn't behave as expected.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants