Andrew Scott

See individual commits

Deprecate class and `InjectionToken` guards and resolvers

26

Class and `InjectionToken`-based guards and resolvers are not as configurable, are less re-usable, require more boilerplate, cannot be defined inline with the route, and require more in-depth knowledge of Angular...

fix(router): Ensure anchor scrolling happens on ignored same URL navigations

1

The Router scroller only listens for NavigationEnd events. However, the default behavior of the Router is to ignore navigations to the same URL. This breaks the anchor scrolling when clicking...

feat(router): Add opt-in preview function for upcoming router changes

10

The `provideRouter` API introduced a way to configure providers without also including all implementations in the bundle, regardless of whether they were used. As a result, we can explore creating...

This commit introduces helper functions to easily convert `Injectable`s with functions compatible with `Route` guards to the corresponding guard functions. These functions will serve to aid in migrating off of...

The Router currently provides the ability to configure a custom `errorHandler` for when unhandled exceptions are thrown during navigation. This allows developers to handle navigation errors separately from other errors...

refactor(core): Update CD traversal to use 'modes'

3

Rather than maintaining separate traversal functions that act differently, this change updates the change detection traversal to share more code and use different modes to control the type of traversal...

This commit adds tests that can be turned on as change detection for signals is implemented.

I will not be available to review PRs for an extended period of time.

fix(material/dialog): `mat-dialog-title` should work under `OnPush` `viewContainerRef`

3

The `mat-dialog-title` directive updates state in a microtask and should call `ChangeDetectorRef.markForCheck`. Failing to do this will cause the component tree to not be checked if it lives under an...