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.

Sign up for GitHub

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

Jump to bottom

[website] Update the about page #38733

Merged
merged 79 commits into from
Sep 22, 2023
Merged

[website] Update the about page #38733

merged 79 commits into from
Sep 22, 2023

Conversation

danilo-leal
Copy link
Contributor

@danilo-leal danilo-leal commented Aug 30, 2023 •
edited by oliviertassinari
Loading

Tweaking the About & Careers page to add the new company values! :)

https://deploy-preview-38733--material-ui.netlify.app/about/

@danilo-leal danilo-leal added the website Pages that are not documentation-related, marketing-focused. label Aug 30, 2023
@danilo-leal danilo-leal self-assigned this Aug 30, 2023
@mui-bot
Copy link

mui-bot commented Aug 30, 2023 •
edited
Loading

Netlify deploy preview

https://deploy-preview-38733--material-ui.netlify.app/

Bundle size report

No bundle size changes (Toolpad)
No bundle size changes

Generated by 🚫 dangerJS against cc8fea0

@danilo-leal danilo-leal marked this pull request as ready for review September 5, 2023 20:51
mnajdova
mnajdova approved these changes Sep 6, 2023
View reviewed changes
Copy link
Member

@mnajdova mnajdova left a comment

Choose a reason for hiding this comment

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

Love it! Copywriting looks good to me, I love the values section 👍 Long term it would be interesting to allow people to click on the images and see them in a gallery like way - I kept pressing the images as they were rotating 😄

@oliviertassinari
Copy link
Member

applied in a different PR, so it could be merged in independently

Ok, removed from this PR and applied in master directly 4930c3b.

@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged label Sep 17, 2023
@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged label Sep 17, 2023
siriwatknp
siriwatknp approved these changes Sep 18, 2023
View reviewed changes
Copy link
Member

@siriwatknp siriwatknp left a comment

Choose a reason for hiding this comment

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

👍 Love it!

@danilo-leal danilo-leal merged commit 1417736 into mui:master Sep 22, 2023
1 check passed
@danilo-leal danilo-leal deleted the about-page-updates branch September 22, 2023 10:28
@oliviertassinari
Copy link
Member

The map, so smooth 😍

mnajdova pushed a commit to mnajdova/material-ui that referenced this pull request Sep 25, 2023
@danilo-leal @oliviertassinari @mnajdova
[website] Update the about page (mui#38733)
eda9814 
Co-authored-by: Olivier Tassinari <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mnajdova mnajdova mnajdova approved these changes

@oliviertassinari oliviertassinari oliviertassinari approved these changes

@siriwatknp siriwatknp siriwatknp approved these changes

Assignees

@danilo-leal danilo-leal

Labels
website Pages that are not documentation-related, marketing-focused.
Projects
Joy UI
Status: Recently completed
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@danilo-leal @mui-bot @oliviertassinari @mnajdova @siriwatknp