New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[docs] Fix small regressions API pages #36972
[docs] Fix small regressions API pages #36972
Conversation
Netlify deploy previewhttps://deploy-preview-36972--material-ui.netlify.app/ Bundle size report |
@@ -305,7 +305,7 @@ export const getDesignTokens = (mode: 'light' | 'dark') => | |||
fontWeight: 700, | |||
}, | |||
allVariants: { | |||
scrollMarginTop: 'calc(var(--MuiDocs-header-height) + 72px)', | |||
scrollMarginTop: 'calc(var(--MuiDocs-header-height) + 32px)', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -4,7 +4,7 @@ import Container from '@mui/material/Container'; | |||
|
|||
const StyledAppContainer = styled(Container)(({ theme }) => { | |||
return { | |||
paddingTop: 'calc(var(--MuiDocs-header-height) + 36px)', | |||
paddingTop: `calc(var(--MuiDocs-header-height) + ${theme.spacing(4)})`, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for consistency with the other padding
}, | ||
}), | ||
...(hasTabs && { | ||
'&& .component-tabs .MuiTabs-root': { | ||
marginBottom: 193, | ||
marginBottom: 198, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix #35938 (comment)
There is still a layout shift with Joy UI, but it's because the theme are not correctly isolated, the Ad renders with Joy UI theme, not with MUI's branding theme, a problem for another day.
<Box> | ||
<div | ||
style={{ | ||
'--MuiDocs-header-height': `${AppFrameHeight + TabsHeight}px`, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix #35938 (comment)
@@ -111,6 +111,8 @@ const Root = styled('div')( | |||
}, | |||
'& h1, & h2, & h3, & h4': { | |||
position: 'relative', | |||
// Reserve space for the end of the line action button | |||
paddingRight: 26 * 2 + 10, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix #35938 (comment)
@@ -103,7 +103,7 @@ const pages = [ | |||
], | |||
}, | |||
{ | |||
title: 'Component API', | |||
title: 'APIs', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copied from #35938 (comment)
A few low-hanging fruits from the items left in #35938.