-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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] Improve chart SEO #10170
[docs] Improve chart SEO #10170
Conversation
3c34590
to
387aa16
Compare
Netlify deploy previewNetlify deploy preview: https://deploy-preview-10170--material-ui-x.netlify.app/ Updated pages
These are the results for the performance tests:
|
387aa16
to
9c06a36
Compare
Signed-off-by: Olivier Tassinari <[email protected]>
9c06a36
to
ff90933
Compare
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.
Make sense with the pickers title
Would it make sense to change the data grid pages as well?
@flaviendelangle I'm not sure, these make sense to me:
One downside of this change is that the title in the browser is a bit less helpful. I have explored if we could only change the SEO title, but I couldn't find something obviously simpler. Google also has runtime engines, so he could detect that the title after hydration is different. I have too many tabs open to be able to read the table title, I just look at the logos, so it might be enough. Speaking of tabs experience optimization. @danilo-leal I guess we could change the favicon based on the docs page we are on with the new logos. |
Per the following search volume data, I think we should change the SEO title.
https://app.ahrefs.com/keywords-explorer/list/new/db2c322fa1f3891b3d2c32da458fe4a5/google/us/overview
on this pages, and the other chart pages.
Preview: https://deploy-preview-10170--material-ui-x.netlify.app/x/react-charts/gantt/