Skip to content

fix(material/expansion): switch away from animations module - #30119

Merged
crisbeto merged 1 commit into
angular:mainfrom
crisbeto:expansion-no-animations
Dec 5, 2024
Merged

fix(material/expansion): switch away from animations module#30119
crisbeto merged 1 commit into
angular:mainfrom
crisbeto:expansion-no-animations

Conversation

@crisbeto

@crisbetocrisbeto commented Dec 2, 2024

Copy link
Copy Markdown
Member

Reworks the expansion panel to animate purely with CSS, rather than going through the @angular/animations module. This simplifies the setup and allows us to resolve several long-standing bug reports.

Fixes#27946.
Fixes#22715.
Fixes#21434.
Fixes#20517.
Fixes#17177.
Fixes#16534.
Fixes#16503.
Fixes#14952.
Fixes#14759.
Fixes#14075.
Fixes#11765.

@crisbetocrisbeto added target: patch This PR is targeted for the next patch release dev-app preview When applied, previews of the dev-app are deployed to Firebase labels Dec 2, 2024
@crisbeto
crisbetoforce-pushed the expansion-no-animations branch from 0c11393 to 1b0a288CompareDecember 2, 2024 16:54
@github-actions

github-actionsBot commented Dec 2, 2024

Copy link
Copy Markdown

Deployed dev-app for 99c87e8 to: https://ng-dev-previews-comp--pr-angular-components-30119-dev-wmtwncsw.web.app

Note: As new commits are pushed to this pull request, this link is updated after the preview is rebuilt.

@crisbeto
crisbetoforce-pushed the expansion-no-animations branch 4 times, most recently from f1deed4 to 01750c7CompareDecember 3, 2024 10:26
Comment threadsrc/material/expansion/expansion-panel.ts Outdated
Reworks the expansion panel to animate purely with CSS, rather than going through the `@angular/animations` module. This simplifies the setup and allows us to resolve several long-standing bug reports.
Fixesangular#27946.
Fixesangular#22715.
Fixesangular#21434.
Fixesangular#20517.
Fixesangular#17177.
Fixesangular#16534.
Fixesangular#16503.
Fixesangular#14952.
Fixesangular#14759.
Fixesangular#14075.
Fixesangular#11765.
@crisbeto
crisbetoforce-pushed the expansion-no-animations branch from bcfd6c3 to b35c757CompareDecember 5, 2024 12:02
@crisbetocrisbeto removed the dev-app preview When applied, previews of the dev-app are deployed to Firebase label Dec 5, 2024
@crisbetocrisbeto added the action: merge The PR is ready for merge by the caretaker label Dec 5, 2024
@crisbeto
crisbeto removed the request for review from amysortoDecember 5, 2024 12:43
@crisbeto
crisbeto merged commit aafa151 into angular:mainDec 5, 2024
crisbeto added a commit that referenced this pull request Dec 5, 2024
Reworks the expansion panel to animate purely with CSS, rather than going through the `@angular/animations` module. This simplifies the setup and allows us to resolve several long-standing bug reports.
Fixes#27946.
Fixes#22715.
Fixes#21434.
Fixes#20517.
Fixes#17177.
Fixes#16534.
Fixes#16503.
Fixes#14952.
Fixes#14759.
Fixes#14075.
Fixes#11765.
(cherry picked from commit aafa151)
@angular-automatic-lock-bot

Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-botangular-automatic-lock-botBot locked and limited conversation to collaborators Jan 5, 2025
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.