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

Update jss/jss-theme-reactor #5772

Merged
merged 1 commit into from
Dec 13, 2016
Merged

Update jss/jss-theme-reactor #5772

merged 1 commit into from
Dec 13, 2016

Conversation

nathanmarks
Copy link
Member

@nathanmarks nathanmarks commented Dec 12, 2016 •
edited
Loading

nathanmarks
nathanmarks commented Dec 12, 2016
View reviewed changes
docs/site/src/components/AppFrame.js
color: theme.palette.accent.A400,
textDecoration: 'none',
},
'a:hover': {
Copy link
Member Author

@nathanmarks nathanmarks Dec 12, 2016 •
edited
Loading

Choose a reason for hiding this comment

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

Temporary for a bug: cssinjs/jss#380

@muibot muibot added PR: needs revision The pull request can't be merged. More details is available on the code review or fails in the CI and removed PR: Needs Review labels Dec 12, 2016
@nathanmarks nathanmarks removed the PR: needs revision The pull request can't be merged. More details is available on the code review or fails in the CI label Dec 12, 2016
@muibot muibot added PR: needs revision The pull request can't be merged. More details is available on the code review or fails in the CI and removed PR: Needs Review labels Dec 12, 2016
@nathanmarks nathanmarks removed the PR: needs revision The pull request can't be merged. More details is available on the code review or fails in the CI label Dec 12, 2016
@muibot muibot added PR: needs revision The pull request can't be merged. More details is available on the code review or fails in the CI and removed PR: Needs Review labels Dec 13, 2016
@nathanmarks nathanmarks changed the title Update jss/jss-theme-reactor [wip] Update jss/jss-theme-reactor Dec 13, 2016
@nathanmarks nathanmarks removed in progress PR: needs revision The pull request can't be merged. More details is available on the code review or fails in the CI labels Dec 13, 2016
@nathanmarks nathanmarks changed the title [wip] Update jss/jss-theme-reactor Update jss/jss-theme-reactor Dec 13, 2016
@nathanmarks nathanmarks merged commit 898d1d7 into mui:next Dec 13, 2016
@nathanmarks nathanmarks deleted the update-jss branch December 13, 2016 02:02
oliviertassinari
oliviertassinari reviewed Dec 13, 2016
View reviewed changes
docs/site/src/components/AppFrame.js
width: 'auto',
},
}), {
named: false, // global CSS.
Copy link
Member

Choose a reason for hiding this comment

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

The new API looks better 👍.

@zannager zannager added the package: system Specific to @mui/system label Mar 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@oliviertassinari oliviertassinari oliviertassinari left review comments

Assignees
No one assigned
Labels
package: system Specific to @mui/system
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@nathanmarks @oliviertassinari @muibot @zannager