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

fix(material/sidenav): switch away from animations module #30235

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

crisbeto
Copy link
Member

Reworks the sidenav to animate using CSS, rather than the animations module. This requires less JavaScript, is simpler to maintain and avoids some memory leaks caused by the animations module.

@crisbeto crisbeto added the target: patch This PR is targeted for the next patch release label Dec 26, 2024
@crisbeto crisbeto requested a review from a team as a code owner December 26, 2024 10:59
@crisbeto crisbeto requested review from mmalerba and wagnermaciel and removed request for a team December 26, 2024 10:59
@crisbeto crisbeto force-pushed the sidenav-no-animations branch 10 times, most recently from 9fb285d to 9db3d12 Compare January 2, 2025 13:07
@crisbeto crisbeto force-pushed the sidenav-no-animations branch from 9db3d12 to 60555a9 Compare January 2, 2025 17:20
src/dev-app/sidenav/sidenav-demo.ts Outdated Show resolved Hide resolved
@crisbeto crisbeto removed the request for review from wagnermaciel January 3, 2025 09:34
@crisbeto crisbeto added the action: merge The PR is ready for merge by the caretaker label Jan 3, 2025
Reworks the sidenav to animate using CSS, rather than the animations module. This requires less JavaScript, is simpler to maintain and avoids some memory leaks caused by the animations module.
@crisbeto crisbeto force-pushed the sidenav-no-animations branch from 60555a9 to c509810 Compare January 3, 2025 09:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
action: merge The PR is ready for merge by the caretaker area: material/sidenav target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants