Skip to content

Conversation

@Tony133
Copy link
Contributor

@Tony133 Tony133 commented Jan 8, 2026

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Docs
  • Other... Please describe:

What is the current behavior?

Issue Number: N/A

What is the new behavior?

Proposals:

  • Removal of @angular/animations: This update removes the @angular/animations dependency (removal planned for v23, as shown in the screenshot). The animations have been replaced with a lightweight SCSS-based solution.

  • Performance Optimization: Introduced improvements to the application’s change detection cycle by enabling event coalescing, which reduces unnecessary re-renders.

angular-animations-deprecated

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

see preview here: https://deploy-preview-3376--docs-nestjs.netlify.app/ 🔥

@kamilmysliwiec
Copy link
Member

padding/margins are different now

image

@Tony133
Copy link
Contributor Author

Tony133 commented Jan 9, 2026

padding/margins are different now

image

Okay, I'll see if I can give it the same values as before, so that there aren't too many differences 😅

@Tony133
Copy link
Contributor Author

Tony133 commented Jan 9, 2026

mmmh It seems to be working fine on the Chrome browser at least 😅

Screenshot 2026-01-09 alle 12 07 29

@kamilmysliwiec
Copy link
Member

It's different on your screenshot too

@Tony133
Copy link
Contributor Author

Tony133 commented Jan 9, 2026

It's different on your screenshot too

Hmm, maybe it's a browser compatibility issue ? 🤔

@kamilmysliwiec
Copy link
Member

What I meant is that paddings/margins are actually incorrect on your screenshot too

@Tony133
Copy link
Contributor Author

Tony133 commented Jan 10, 2026

What I meant is that paddings/margins are actually incorrect on your screenshot too

I made a correction, yes, there is something slightly different from before, and I'm not sure if it's possible to restore it exactly as it was before 😅. I am attaching a screenshot of how I see it on localhost and on docs.nestjs.com

Screenshot 2026-01-10 alle 11 04 17

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants