-
Notifications
You must be signed in to change notification settings - Fork 6.8k
docs: switch remaining docs to @use #22527
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
Conversation
This comment has been minimized.
This comment has been minimized.
src/cdk/overlay/overlay.md
Outdated
@@ -7,7 +7,7 @@ using the CDK on its own, you'll have to include the styles yourself. You can do | |||
the prebuilt styles in your global stylesheet: | |||
|
|||
```scss | |||
@import '~@angular/cdk/overlay-prebuilt.css'; | |||
@use '~@angular/cdk/overlay-prebuilt.css'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This one was actually a native CSS @import
rather than a Sass import (because of the .css
extension).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reverted, although I think that the result will be the same for Webpack projects where @import
statements inline their result.
Updates all of the remaining docs to go through the new `@use` API instead of `@import`. **Note:** does not include the `mdc-button` readme, because the necessary mixins aren't exported yet. See angular#22449.
d0bcd1a
to
39979b7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Updates all of the remaining docs to go through the new
@use
API instead of@import
.Note: does not include the
mdc-button
readme, because the necessary mixins aren't exported yet. See #22449.