skills/angular__angular
1
0
Fork 0
mirror of https://github.com/angular/angular.git synced 2026-09-14 13:54:52 +08:00
Code Issues Packages Projects Releases Wiki Activity

refactor(common): drop NullViewportScroller for client bundles (#59440)

Browse Source 
In this commit, we replace the `isPlatformBrowser` runtime call with the `ngServerMode` in order to drop the `NullViewportScroller` for client bundles.

PR Close #59440
This commit is contained in:
arturovt
2025-01-09 00:38:18 +02:00
committed by kirjs
parent 205330d46a
commit 83bd63fa7c
1 changed files with 4 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
packages/common/src/viewport_scroller.ts
+4 -5
View File
@@ -6,10 +6,9 @@
* found in the LICENSE file at https://angular.dev/license
*/
import {inject, PLATFORM_ID, ɵɵdefineInjectable} from '@angular/core';
import {inject, ɵɵdefineInjectable} from '@angular/core';
import {DOCUMENT} from './dom_tokens';
import {isPlatformBrowser} from './platform_id';
/**
* Defines a scroll position manager. Implemented by `BrowserViewportScroller`.
@@ -24,9 +23,9 @@ export abstract class ViewportScroller {
token: ViewportScroller,
providedIn: 'root',
factory: () =>
isPlatformBrowser(inject(PLATFORM_ID))
? new BrowserViewportScroller(inject(DOCUMENT), window)
: new NullViewportScroller(),
typeof ngServerMode !== 'undefined' && ngServerMode
? new NullViewportScroller()
: new BrowserViewportScroller(inject(DOCUMENT), window),
});
/**