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 SEO issues #34537
[docs] Fix SEO issues #34537
Conversation
c64cd8b
to
518af4f
Compare
@@ -9,13 +9,11 @@ import BookmarkAdd from '@mui/icons-material/BookmarkAddOutlined'; | |||
|
|||
export default function BasicCard() { | |||
return ( | |||
<Card variant="outlined" sx={{ minWidth: '320px' }}> | |||
<Box sx={{ display: 'flex', flexDirection: 'column', gap: 0.5 }}> |
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.
Unecessary, removed
<AspectRatio minHeight="120px" maxHeight="200px" sx={{ my: 2 }}> | ||
<img | ||
src="https://images.unsplash.com/photo-1527549993586-dff825b37782?crop=entropy&auto=format&fit=crop&w=3270" |
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.
alt="" | ||
/> | ||
</AspectRatio> | ||
<Box sx={{ display: 'flex' }}> | ||
<div> | ||
<Typography level="body3">Total price:</Typography> | ||
<Typography fontSize="lg" fontWeight="lg"> | ||
$2900 | ||
$2,900 |
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.
en-US locale number format
@@ -39,7 +39,7 @@ const vars = [ | |||
}, | |||
]; | |||
|
|||
export default function GroupedAvatars() { | |||
export default function CardVariables() { |
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.
Match filename
@@ -137,7 +137,7 @@ | |||
"stars": "GitHub stars", | |||
"stickyFooterDescr": "Attach a footer to the bottom of the viewport when page content is short.", | |||
"stickyFooterTitle": "Sticky footer", | |||
"strapline": "MUI is a simple and customizable component library to build faster, beautiful, and more accessible React applications. Follow your own design system, or start with Material Design.", | |||
"strapline": "MUI provides a simple, customizable, and accessible library of React components. Follow your own design system, or start with Material Design.", |
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.
This was too long & out of sync with the description on https://mui.com/index.html.
@@ -11,7 +11,7 @@ import References, { CORE_CUSTOMERS } from 'docs/src/components/home/References' | |||
import AppFooter from 'docs/src/layouts/AppFooter'; | |||
import AppHeaderBanner from 'docs/src/components/banner/AppHeaderBanner'; | |||
|
|||
export default function Home() { | |||
export default function Core() { |
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.
Match pathname
@@ -31,7 +31,7 @@ export const getStaticProps = () => { | |||
}; | |||
}; | |||
|
|||
const PostPreview = (props: BlogPost) => { | |||
function PostPreview(props: BlogPost) { |
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.
Component convention.
@@ -377,7 +377,7 @@ const appList = [ | |||
title: 'HiFiveWork', | |||
description: 'HiFiveWork, the cool tool for leave management', | |||
image: 'hifivework.png', | |||
link: 'https://www.hifivework.com/#/', | |||
link: 'https://hifivework.com/', |
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.
301
alt="" | ||
/> | ||
</CardCover> | ||
<CardContent sx={{ justifyContent: 'center', gap: 1 }}> |
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.
I could see no impacts from this CSS.
alt="" | ||
/> | ||
</AspectRatio> | ||
<Box> | ||
<Box sx={{ ml: 0.5 }}> |
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.
This Box duplicates with the div and the gap prop.
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 some of the issues in https://app.ahrefs.com/site-audit/3524616/32/issues.
Before: https://mui.com/joy-ui/react-card/
After: https://deploy-preview-34537--material-ui.netlify.app/joy-ui/react-card/