Skip to content

[Joy] Batch a couple of documentation refinements#33158

Merged
danilo-leal merged 50 commits intomui:masterfrom
danilo-leal:batch-joy-refinements
Jul 4, 2022
Merged

[Joy] Batch a couple of documentation refinements#33158
danilo-leal merged 50 commits intomui:masterfrom
danilo-leal:batch-joy-refinements

Conversation

@danilo-leal
Copy link
Collaborator

@danilo-leal danilo-leal commented Jun 15, 2022

Using this one to do some overall cleanup of the docs we currently already have + demos design + some design iteration as well. Deploy preview: https://deploy-preview-33158--material-ui.netlify.app/joy-ui/getting-started/overview/

@danilo-leal danilo-leal added docs Improvements or additions to the documentation. package: joy-ui Specific to Joy UI. labels Jun 15, 2022
@danilo-leal danilo-leal marked this pull request as draft June 15, 2022 07:12
@mui-bot
Copy link

mui-bot commented Jun 15, 2022

Details of bundle changes

@mui/joy: parsed: -0.25% 😍, gzip: -0.01% 😍

Generated by 🚫 dangerJS against ef1ae81

@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged. label Jun 15, 2022
@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged. label Jun 15, 2022
@danilo-leal danilo-leal self-assigned this Jun 16, 2022
@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged. label Jul 1, 2022
@github-actions github-actions bot added PR: out-of-date The pull request has merge conflicts and can't be merged. and removed PR: out-of-date The pull request has merge conflicts and can't be merged. labels Jul 1, 2022
@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged. label Jul 4, 2022
@danilo-leal danilo-leal mentioned this pull request Jul 4, 2022
@danilo-leal
Copy link
Collaborator Author

cc @samuelsycamore

I'll merge this one for this week's release given the documentation organization and color palette updates! There are for sure many opportunities for improvement here so let's revise it further down the road. Didn't want to let it go past 20 days also 😅

@danilo-leal danilo-leal merged commit 461c04b into mui:master Jul 4, 2022
@danilo-leal danilo-leal deleted the batch-joy-refinements branch July 4, 2022 13:53
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.

https://www.notion.so/mui-org/Engineering-philosophy-ec1615511f594782a8a2adeec3e25143#8d36ec1163de460fbc7059747855e8e3. I get that we can skip the reviews for small PRs, but here, there are over 1,000 LOCs changed. I think that one approved review would have been great.

A couple of regressions:

daniel-rabe pushed a commit to daniel-rabe/material-ui that referenced this pull request Nov 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Improvements or additions to the documentation. package: joy-ui Specific to Joy UI.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants