Skip to content
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

[website] Polish some pages and stray components #40797

Merged
merged 17 commits into from
Jan 29, 2024

Conversation

danilo-leal
Copy link
Contributor

@danilo-leal danilo-leal commented Jan 26, 2024

This PR adds some general polish changes throughout many pages of the website. I was prompted to open it after I realized a previous PR of mine (#40752) tweaking the testimonials section and related components broke some stuff. And then, as it naturally goes, started to notice a few other refinement opportunities. Here's a list of highlights:

  • Simplified the Material UI demo on the homepage's product section
  • Simplified the "Start now" section used across a few pages
  • Made the Material UI "Community" section consistent with the Base UI one
  • General spacing and color tweaks throughout

Example pages:

@danilo-leal danilo-leal added design This is about UI or UX design, please involve a designer website Pages that are not documentation-related, marketing-focused. labels Jan 26, 2024
@danilo-leal danilo-leal self-assigned this Jan 26, 2024
@mui-bot
Copy link

mui-bot commented Jan 26, 2024

Netlify deploy preview

Bundle size report

No bundle size changes (Toolpad)
No bundle size changes

Generated by 🚫 dangerJS against 1257bf4

@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged label Jan 29, 2024
@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged label Jan 29, 2024
@danilo-leal danilo-leal marked this pull request as ready for review January 29, 2024 12:27
Copy link
Contributor

@zanivan zanivan left a comment

Choose a reason for hiding this comment

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

Love these stray polishing PRs of yours! As always, below are some nits I found 😅


Missing space between project/faster:

Screenshot 2024-01-29 at 13 33 33

This is something unrelated to this PR, but might be worth it to solve:

Screenshot 2024-01-29 at 13 39 34

On the /core page, the start now section redirects the user to the same page—maybe we could have something linking to the discord channel?

Copy link
Contributor

@zanivan zanivan left a comment

Choose a reason for hiding this comment

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

🚀

@danilo-leal danilo-leal merged commit 0492cf2 into mui:master Jan 29, 2024
19 checks passed
@danilo-leal danilo-leal deleted the add-website-polish branch January 29, 2024 18:42
mostafa-rio pushed a commit to mostafa-rio/material-ui that referenced this pull request Feb 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design This is about UI or UX design, please involve a designer website Pages that are not documentation-related, marketing-focused.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants