Skip to content

Conversation

@mnajdova
Copy link
Member

@mnajdova mnajdova commented Oct 2, 2020

Breaking changes

  • Renamed fade to alpha to better describe its functionality.
- import { fade } from '@material-ui/core/styles';
+ import { alpha } from '@material-ui/core/styles';
const classes = makeStyles(theme => ({
-  backgroundColor: fade(theme.palette.primary.main, theme.palette.action.selectedOpacity),
+  backgroundColor: alpha(theme.palette.primary.main, theme.palette.action.selectedOpacity),
}));

Part of #20012, the change is discussed in #13039 (comment).

@mui-pr-bot
Copy link

mui-pr-bot commented Oct 2, 2020

Details of bundle changes

Generated by 🚫 dangerJS against 9775213

@mnajdova mnajdova marked this pull request as ready for review October 2, 2020 09:30
@mnajdova mnajdova added the breaking change Introduces changes that are not backward compatible. label Oct 2, 2020
@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged. label Oct 2, 2020
@oliviertassinari oliviertassinari changed the title [theme] rename fade to alpha [theme] Rename fade to alpha Oct 3, 2020
@oliviertassinari oliviertassinari added this to the v5 milestone Oct 3, 2020
@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged. label Oct 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking change Introduces changes that are not backward compatible.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants