-
Notifications
You must be signed in to change notification settings - Fork 6.8k
fix(card): disable all animations when using NoopAnimationsModule #11538
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
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
@crisbeto could you rebase this one? Just having an issue with the sync script since the sha is older |
Done. |
99f5c65
to
e9eabcf
Compare
Hi @crisbeto! This PR has merge conflicts due to recent upstream merges. |
1 similar comment
Hi @crisbeto! This PR has merge conflicts due to recent upstream merges. |
e9eabcf
to
bac12e6
Compare
Fixes the card `box-shadow` transition not being disabled when it's inside something with the `NoopAnimationsModule`. Relates to angular#10590.
bac12e6
to
f8bfe0f
Compare
…gular#11538) * fix(card): disable all animations when using NoopAnimationsModule Fixes the card `box-shadow` transition not being disabled when it's inside something with the `NoopAnimationsModule`. Relates to angular#10590. * fix lint * fix api test
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. |
Fixes the card
box-shadow
transition not being disabled when it's inside something with theNoopAnimationsModule
.Relates to #10590.