[website] Add content about the Sync plugin in the Material UI page #40515
Conversation
danilo-leal
added
the
website
Netlify deploy previewhttps://deploy-preview-40515--material-ui.netlify.app/ Bundle size report |
danilo-leal
changed the title
github-actions
bot
added
the
PR: out-of-date
Signed-off-by: Danilo Leal <[email protected]>
github-actions
bot
removed
the
PR: out-of-date
github-actions
bot
added
the
PR: out-of-date
Signed-off-by: Danilo Leal <[email protected]>
github-actions
bot
removed
the
PR: out-of-date
danilo-leal
added
the
needs cherry-pick
zanivan
left a comment
zanivan
left a comment
There was a problem hiding this comment.
It looks good! Just pushed a small fix for layout shift, and have a couple of remarks:
- Here, I don't think we should have colorful icons, although I like them, it's not like this anywhere—so it's a bit odd to me.
- The card layout is broken, cutting off the buttons
On a different note, I think it's worth changing the Design kits mentions to something broader, like “Design resources” or jut “Design”
@zanivan that makes sense, but there are some other related tasks (like fine-tuning the actual Design Kit description on the Store) that I'd find good to do before committing to this change. This design kit is very Material UI-centric, naturally, and we should go toward this clarity! |
zanivan
left a comment
zanivan
left a comment
There was a problem hiding this comment.
Found one more issue, the focus style doesn't seem to be working properly here:
zanivan
left a comment
zanivan
left a comment
There was a problem hiding this comment.
Found a bug with the hover animation on the Design kit image:
Screen.Recording.2024-04-30.at.11.57.19.mov
zanivan
left a comment
zanivan
left a comment
There was a problem hiding this comment.
Design-wise looks good to me :)
…40515) Co-authored-by: David Cnoops <[email protected]> Co-authored-by: zanivan <[email protected]>
7 participants
This PR adds content about the Material UI Sync plugin to the Material UI and Design Kits pages.