feat!: upgrade to Angular 21#4442
Merged
damienwebdev merged 13 commits intoJun 4, 2026
Merged
Conversation
griest024
force-pushed
the
feat/all/upgrade/angular/21
branch
from
5f72ad8 to
4c0f937
Compare
Member
|
@griest024 your build is now running against Angular 21. |
griest024
force-pushed
the
feat/all/upgrade/angular/21
branch
from
d343777 to
db7248f
Compare
griest024
force-pushed
the
feat/all/upgrade/angular/21
branch
3 times, most recently
from
097f294 to
7d37927
Compare
damienwebdev
previously requested changes
Jun 3, 2026
Member
|
@griest024 a few small comments, generally LGTM from playing around with it for a few hours. tyvm! |
damienwebdev
force-pushed
the
feat/all/upgrade/angular/21
branch
from
7d37927 to
1ea1bfb
Compare
Co-authored-by: Peter Ashwood <griest024@gmail.com>
… that angular host can access them (graycoreio#4442) Co-authored-by: Peter Ashwood <griest024@gmail.com>
Convert the WrapperComponent's fields to signals so template bindings update without relying on zone-based change detection
…ntFocus (graycoreio#4442) Patch native focus/blur in specs so focus-driven tests don't depend on the document actually having focus, which makes them order-dependent (at least in Headless Chrome).
…ycoreio#4442) Convert the WrapperComponent's view field to a signal so the template binding updates without relying on zone-based change detection, fixing NG0100 errors when testing without zone CD.
…#4442) Co-authored-by: Peter Ashwood <griest024@gmail.com>
…int 21 (graycoreio#4442) angular-eslint 21 changed the component-selector and directive-selector rules to enforce an "app" prefix by default when the prefix option is omitted. Set prefix to an empty string wherever these rules are configured to preserve the existing unprefixed selectors, and drop the eslint-disable comments this makes unnecessary. Co-authored-by: Peter Ashwood <griest024@gmail.com>
This was previously shadowed by the global document when the root `tsconfig.json` contained es2022 and dom.
Co-authored-by: Peter Ashwood <griest024@gmail.com>
Co-authored-by: Peter Ashwood <griest024@gmail.com>
…coreio#4442) Co-authored-by: Peter Ashwood <griest024@gmail.com>
damienwebdev
force-pushed
the
feat/all/upgrade/angular/21
branch
from
1ea1bfb to
cb300c8
Compare
Closed
damienwebdev
added a commit
that referenced
this pull request
Jun 4, 2026
damienwebdev
added a commit
that referenced
this pull request
Jun 4, 2026
Co-authored-by: Peter Ashwood <griest024@gmail.com>
damienwebdev
added a commit
that referenced
this pull request
Jun 4, 2026
damienwebdev
added a commit
that referenced
this pull request
Jun 4, 2026
… that angular host can access them (#4442) Co-authored-by: Peter Ashwood <griest024@gmail.com>
damienwebdev
added a commit
that referenced
this pull request
Jun 4, 2026
Convert the WrapperComponent's fields to signals so template bindings update without relying on zone-based change detection
damienwebdev
added a commit
that referenced
this pull request
Jun 4, 2026
…ntFocus (#4442) Patch native focus/blur in specs so focus-driven tests don't depend on the document actually having focus, which makes them order-dependent (at least in Headless Chrome).
damienwebdev
added a commit
that referenced
this pull request
Jun 4, 2026
Convert the WrapperComponent's view field to a signal so the template binding updates without relying on zone-based change detection, fixing NG0100 errors when testing without zone CD.
damienwebdev
added a commit
that referenced
this pull request
Jun 4, 2026
Co-authored-by: Peter Ashwood <griest024@gmail.com>
damienwebdev
added a commit
that referenced
this pull request
Jun 4, 2026
…int 21 (#4442) angular-eslint 21 changed the component-selector and directive-selector rules to enforce an "app" prefix by default when the prefix option is omitted. Set prefix to an empty string wherever these rules are configured to preserve the existing unprefixed selectors, and drop the eslint-disable comments this makes unnecessary. Co-authored-by: Peter Ashwood <griest024@gmail.com>
damienwebdev
added a commit
that referenced
this pull request
Jun 4, 2026
This was previously shadowed by the global document when the root `tsconfig.json` contained es2022 and dom.
damienwebdev
added a commit
that referenced
this pull request
Jun 4, 2026
Co-authored-by: Peter Ashwood <griest024@gmail.com>
damienwebdev
added a commit
that referenced
this pull request
Jun 4, 2026
Co-authored-by: Peter Ashwood <griest024@gmail.com>
damienwebdev
added a commit
that referenced
this pull request
Jun 4, 2026
Co-authored-by: Peter Ashwood <griest024@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR Checklist
PR Type
Current behavior
Fixes: #4294 fixes: #4292
Part of: #
New behavior
Breaking change?
Additional context