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] Developer survey 2022 #35407

Merged
merged 21 commits into from
Dec 12, 2022
Merged

[website] Developer survey 2022 #35407

merged 21 commits into from
Dec 12, 2022
+142 −56

Conversation

joserodolfofreitas
Copy link
Member

@joserodolfofreitas joserodolfofreitas commented Dec 9, 2022 •
edited by oliviertassinari
Loading

Summary

Add callout to take the Developer survey 2022.

  • Add a top banner on the main website
  • Add a side banner on the docs.
  • Add a notification for the docs.

Preview

Website:
https://deploy-preview-35407--material-ui.netlify.app/

Docs:
https://deploy-preview-35407--material-ui.netlify.app/material-ui/getting-started/overview/

Note

Alternatively to replacing the hiring ad, we can use the top banner by toggling the switches in the featureToggle.js file.

@joserodolfofreitas
Add routes to the survey 2022

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
GPG key ID: 4AEE18F83AFDEB23
Expired
Verified
Learn about vigilant mode
4c293f8
@joserodolfofreitas
Add banner on the website, right side of docs and prepare alternative…

Partially verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
We cannot verify signatures from co-authors, and some of the co-authors attributed to this commit require their commits to be signed.
ae9d619 
… with docs navbar
@joserodolfofreitas
Add notification to the docs
1b61432
@joserodolfofreitas joserodolfofreitas added docs Improvements or additions to the documentation website Pages that are not documentation-related, marketing-focused. labels Dec 9, 2022
@mui-bot
Copy link

mui-bot commented Dec 9, 2022 •
edited
Loading

Messages
📖 Netlify deploy preview: https://deploy-preview-35407--material-ui.netlify.app/

No bundle size changes

Generated by 🚫 dangerJS against ba74077

mnajdova
mnajdova reviewed Dec 9, 2022
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.

image

We should fix the contrast on hover on the banner on home page.

joserodolfofreitas and others added 2 commits December 9, 2022 14:21
@joserodolfofreitas @mnajdova
Apply Marija's suggestions from code review
70f99ad 
Co-authored-by: Marija Najdova <mnajdova@gmail.com>
Signed-off-by: José Rodolfo Freitas <joserodolfo.freitas@gmail.com>
@joserodolfofreitas
fix link onhover
1383222
@joserodolfofreitas
Copy link
Member Author

joserodolfofreitas commented Dec 9, 2022 •
edited
Loading

We should fix the contrast on hover on the banner on home page.

Thank you for the catch, @mnajdova!
It's done.

oliviertassinari
oliviertassinari reviewed Dec 9, 2022
View reviewed changes
Copy link
Member

@oliviertassinari oliviertassinari left a comment

Choose a reason for hiding this comment

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

Looks good 👍

oliviertassinari
oliviertassinari reviewed Dec 11, 2022
View reviewed changes
@oliviertassinari oliviertassinari changed the title [website] Survey 2022 [website] Developer survey 2022 Dec 11, 2022
joserodolfofreitas and others added 3 commits December 12, 2022 12:45
@joserodolfofreitas @oliviertassinari
Oliviers suggestions from code review
6ef4157 
Co-authored-by: Olivier Tassinari <olivier.tassinari@gmail.com>
Signed-off-by: José Rodolfo Freitas <joserodolfo.freitas@gmail.com>
@joserodolfofreitas
Update side banner message with olivier+danilo suggestion
fcbf8bb
@joserodolfofreitas
Prettier
86762be
@joserodolfofreitas @danilo-leal
Danilo's review on AppHeaderBanner.tsx
ba74077 
Co-authored-by: Danilo Leal <67129314+danilo-leal@users.noreply.github.com>
Signed-off-by: José Rodolfo Freitas <joserodolfo.freitas@gmail.com>
danilo-leal
danilo-leal approved these changes Dec 12, 2022
View reviewed changes
Copy link
Contributor

@danilo-leal danilo-leal left a comment

Choose a reason for hiding this comment

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

Looking good ⎯ excited for this year's iteration 🙌

@joserodolfofreitas joserodolfofreitas merged commit c5ec6f6 into mui:master Dec 12, 2022
@oliviertassinari
Copy link
Member

oliviertassinari commented Dec 12, 2022 •
edited
Loading

Nice, very clean.

Edit: survey closed with 612db95.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@danilo-leal danilo-leal

@siriwatknp siriwatknp

@mnajdova mnajdova

@oliviertassinari oliviertassinari

Assignees
No one assigned
Labels
docs Improvements or additions to the documentation website Pages that are not documentation-related, marketing-focused.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@joserodolfofreitas @mui-bot @oliviertassinari @mnajdova @danilo-leal