Skip to content

Conversation

mhdsbq
Copy link
Contributor

@mhdsbq mhdsbq commented Aug 23, 2025

When a form becomes invalid and displays an error, the error message was no longer animating. This regression was introduced in #30678 (accessibility changes), because error elements were no longer destroyed/re-rendered, preventing the CSS animation from running.

This fix ensures that the animation works again without impacting accessibility.

Fixes #31712

When a form becomes invalid and displays an error, the error message was no longer animating. This regression was introduced in angular#30678 (accessibility changes), because error elements were no longer destroyed/re-rendered, preventing the CSS animation from running.

This fix ensures that the animation works again without impacting accessibility.

Fixes angular#31712
@mhdsbq mhdsbq requested a review from a team as a code owner August 23, 2025 17:10
@mhdsbq mhdsbq requested review from crisbeto and ok7sai and removed request for a team August 23, 2025 17:10
@crisbeto crisbeto requested a review from adolgachev August 23, 2025 20:38
@crisbeto crisbeto added action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release labels Sep 19, 2025
@crisbeto crisbeto merged commit 977f46f into angular:main Sep 19, 2025
25 checks passed
crisbeto pushed a commit that referenced this pull request Sep 19, 2025
When a form becomes invalid and displays an error, the error message was no longer animating. This regression was introduced in #30678 (accessibility changes), because error elements were no longer destroyed/re-rendered, preventing the CSS animation from running.

This fix ensures that the animation works again without impacting accessibility.

Fixes #31712

(cherry picked from commit 977f46f)
@crisbeto
Copy link
Member

This PR was merged into the repository. The changes were merged into the following branches:

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/form-field target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bug(mat-form-field): No animation for error message.
2 participants