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

[docs] Fix type vs. mode and capitalization of createMuiTheme #22844

Merged
merged 2 commits into from Oct 4, 2020

Conversation

joshwooding
Copy link
Member

@joshwooding joshwooding commented Oct 2, 2020

Noticed these.

@joshwooding joshwooding added docs Improvements or additions to the documentation component: slider This is the name of the generic UI component, not the React module! labels Oct 2, 2020
-const theme = createMuitheme({palette: { type: 'dark' }}),
+const theme = createMuitheme({
-const theme = createMuiTheme({palette: { type: 'dark' }}),
+const theme = createMuiTheme({
+ palette: { type: 'dark', text: { hint: 'rgba(0, 0, 0, 0.38)' } },
Copy link
Member Author

Choose a reason for hiding this comment

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

Wasn't sure whether to change this from type to mode.

Copy link
Contributor

Choose a reason for hiding this comment

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

This should be changed, I think.

Copy link
Member

Choose a reason for hiding this comment

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

Agree, we could base the diff based the most accurate state of v5. So use mode: dark in both cases. @hubgit do you want to send a pull request? :)

Copy link
Contributor

Choose a reason for hiding this comment

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

Ok, I'll send a PR. It seems that this diff is the wrong way round, too, if I'm understanding the changes correctly?

Copy link
Member

Choose a reason for hiding this comment

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

The diff is correct

Copy link
Contributor

Choose a reason for hiding this comment

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

Created #23416

@mui-pr-bot
Copy link

Details of bundle changes

Generated by 🚫 dangerJS against c4d5480

@oliviertassinari oliviertassinari changed the title [docs] More type to dark changes and correct capitalization of createMuiTheme [docs] Fix type vs. mode and capitalization of createMuiTheme Oct 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: slider This is the name of the generic UI component, not the React module! docs Improvements or additions to the documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants