refactor(cdk/dialog): switch to tree shakeable overlay APIs

Reworks the module to use the new tree-shakeable APIs for creating overlays.

Author: crisbeto

src/cdk/dialog/dialog-injectors.ts

@@ -6,8 +6,8 @@
  * found in the LICENSE file at https://angular.dev/license
  */
 
-import {InjectionToken, inject} from '@angular/core';
-import {Overlay, ScrollStrategy} from '../overlay';
+import {InjectionToken, Injector, inject} from '@angular/core';
+import {createBlockScrollStrategy, ScrollStrategy} from '../overlay';
 import {DialogConfig} from './dialog-config';
 
 /** Injection token for the Dialog's ScrollStrategy. */
@@ -16,8 +16,8 @@ export const DIALOG_SCROLL_STRATEGY = new InjectionToken<() => ScrollStrategy>(
   {
     providedIn: 'root',
     factory: () => {
-      const overlay = inject(Overlay);
-      return () => overlay.scrollStrategies.block();
+      const injector = inject(Injector);
+      return () => createBlockScrollStrategy(injector);
     },
   },
 );

src/cdk/dialog/dialog.spec.ts

@@ -1,6 +1,11 @@
 import {Directionality} from '../bidi';
 import {A, ESCAPE} from '../keycodes';
-import {Overlay, OverlayContainer, ScrollDispatcher} from '../overlay';
+import {
+  createCloseScrollStrategy,
+  createGlobalPositionStrategy,
+  OverlayContainer,
+  ScrollDispatcher,
+} from '../overlay';
 import {_supportsShadowDom} from '../platform';
 import {createKeyboardEvent, dispatchEvent, dispatchKeyboardEvent} from '../testing/private';
 import {Location} from '@angular/common';
@@ -37,7 +42,6 @@ describe('Dialog', () => {
   let testViewContainerRef: ViewContainerRef;
   let viewContainerFixture: ComponentFixture<ComponentWithChildViewContainer>;
   let mockLocation: SpyLocation;
-  let overlay: Overlay;
   let scrolledSubject = new Subject();
 
   beforeEach(fakeAsync(() => {
@@ -66,7 +70,6 @@ describe('Dialog', () => {
 
     dialog = TestBed.inject(Dialog);
     mockLocation = TestBed.inject(Location) as SpyLocation;
-    overlay = TestBed.inject(Overlay);
     overlayContainerElement = TestBed.inject(OverlayContainer).getContainerElement();
 
     viewContainerFixture = TestBed.createComponent(ComponentWithChildViewContainer);
@@ -452,7 +455,7 @@ describe('Dialog', () => {
 
   it('should be able to customize the position strategy', () => {
     dialog.open(PizzaMsg, {
-      positionStrategy: overlay.position().global().top('100px'),
+      positionStrategy: createGlobalPositionStrategy(TestBed.inject(Injector)).top('100px'),
     });
 
     viewContainerFixture.detectChanges();
@@ -555,12 +558,13 @@ describe('Dialog', () => {
 
   it('should close the dialog when detached externally', fakeAsync(() => {
     const closeSpy = jasmine.createSpy('closed');
+    const injector = TestBed.inject(Injector);
     dialog
-      .open(PizzaMsg, {scrollStrategy: overlay.scrollStrategies.close()})
+      .open(PizzaMsg, {scrollStrategy: createCloseScrollStrategy(injector)})
       .closed.subscribe(closeSpy);
     viewContainerFixture.detectChanges();
     dialog
-      .open(PizzaMsg, {scrollStrategy: overlay.scrollStrategies.close()})
+      .open(PizzaMsg, {scrollStrategy: createCloseScrollStrategy(injector)})
       .closed.subscribe(closeSpy);
     viewContainerFixture.detectChanges();
 

src/cdk/dialog/dialog.ts

@@ -22,7 +22,14 @@ import {Observable, Subject, defer} from 'rxjs';
 import {startWith} from 'rxjs/operators';
 import {_IdGenerator} from '../a11y';
 import {Direction, Directionality} from '../bidi';
-import {ComponentType, Overlay, OverlayConfig, OverlayContainer, OverlayRef} from '../overlay';
+import {
+  ComponentType,
+  createGlobalPositionStrategy,
+  createOverlayRef,
+  OverlayConfig,
+  OverlayContainer,
+  OverlayRef,
+} from '../overlay';
 import {BasePortalOutlet, ComponentPortal, TemplatePortal} from '../portal';
 import {DialogConfig} from './dialog-config';
 import {DialogRef} from './dialog-ref';
@@ -47,7 +54,6 @@ function getDirectionality(value: Direction): Directionality {
 
 @Injectable({providedIn: 'root'})
 export class Dialog implements OnDestroy {
-  private _overlay = inject(Overlay);
   private _injector = inject(Injector);
   private _defaultOptions = inject<DialogConfig>(DEFAULT_DIALOG_CONFIG, {optional: true});
   private _parentDialog = inject(Dialog, {optional: true, skipSelf: true});
@@ -131,7 +137,7 @@ export class Dialog implements OnDestroy {
     }
 
     const overlayConfig = this._getOverlayConfig(config);
-    const overlayRef = this._overlay.create(overlayConfig);
+    const overlayRef = createOverlayRef(this._injector, overlayConfig);
     const dialogRef = new DialogRef(overlayRef, config);
     const dialogContainer = this._attachContainer(overlayRef, dialogRef, config);
 
@@ -195,7 +201,7 @@ export class Dialog implements OnDestroy {
     const state = new OverlayConfig({
       positionStrategy:
         config.positionStrategy ||
-        this._overlay.position().global().centerHorizontally().centerVertically(),
+        createGlobalPositionStrategy(this._injector).centerHorizontally().centerVertically(),
       scrollStrategy: config.scrollStrategy || this._scrollStrategy(),
       panelClass: config.panelClass,
       hasBackdrop: config.hasBackdrop,