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

refactor: fix up sass division deprecation warnings #7158

Merged

Conversation

crisbeto
Copy link
Collaborator

@crisbeto crisbeto commented Jun 3, 2021

The division operator is deprecated and it causes Sass to print a warning in the latest version. These changes migrate to the math.div helper.

See https://sass-lang.com/documentation/breaking-changes/slash-div.

@crisbeto crisbeto requested a review from abhiomkar June 3, 2021 14:22
@google-cla google-cla bot added the cla: yes label Jun 3, 2021
Copy link
Contributor

@abhiomkar abhiomkar left a comment

Choose a reason for hiding this comment

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

LGTM.

packages/mdc-radio/_radio-theme.scss Outdated Show resolved Hide resolved
The division operator is deprecated and it causes Sass to print a warning in the latest version. These changes migrate to the `math.div` helper.
@crisbeto
Copy link
Collaborator Author

crisbeto commented Jun 3, 2021

It's rebased @abhiomkar

@abhiomkar
Copy link
Contributor

Thanks @crisbeto ! Imported this change internally. Will merge this after the test results.

@copybara-service copybara-service bot merged commit 2daa49b into material-components:master Jun 4, 2021
ChadBurggraf added a commit to dispoteca/material-components-web that referenced this pull request Aug 24, 2021
This backports PR material-components#7158 to the v5.1 branch so that a new
5.x release can be made. Some popular libraries like
[React Material Web Components](https://github.com/jamesmfriedman/rmwc) are
based on 5.x. This removes SASS warnings while users migrate to
better-supported libraries.
ChadBurggraf added a commit to dispoteca/material-components-web that referenced this pull request Aug 24, 2021
This backports PR material-components#7158 to the v5.1 branch so that a new
5.x release can be made. Some popular libraries like
[React Material Web Components](https://github.com/jamesmfriedman/rmwc) are
based on 5.x. This removes SASS warnings while users migrate to
better-supported libraries.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants