Skip to content

Commit

Permalink
perf(module:resizable): mark mousedown and touchstart listeners a…
Browse files Browse the repository at this point in the history
…s passive (#7331)
  • Loading branch information
arturovt committed Apr 21, 2022
1 parent 106d346 commit 518997b
Showing 1 changed file with 8 additions and 2 deletions.
10 changes: 8 additions & 2 deletions components/resizable/resize-handle.component.ts
Expand Up @@ -3,6 +3,7 @@
* found in the LICENSE file at https://github.com/NG-ZORRO/ng-zorro-antd/blob/master/LICENSE
*/

import { normalizePassiveListenerOptions } from '@angular/cdk/platform';
import {
ChangeDetectionStrategy,
Component,
Expand Down Expand Up @@ -35,6 +36,8 @@ export class NzResizeHandleMouseDownEvent {
constructor(public direction: NzResizeDirection, public mouseEvent: MouseEvent | TouchEvent) {}
}

const passiveEventListenerOptions = <AddEventListenerOptions>normalizePassiveListenerOptions({ passive: true });

@Component({
selector: 'nz-resize-handle, [nz-resize-handle]',
exportAs: 'nzResizeHandle',
Expand Down Expand Up @@ -75,9 +78,12 @@ export class NzResizeHandleComponent implements OnInit {
});

this.ngZone.runOutsideAngular(() => {
// Note: since Chrome 56 defaults document level `touchstart` listener to passive.
// The element `touchstart` listener is not passive by default
// We never call `preventDefault()` on it, so we're safe making it passive too.
merge(
fromEvent<MouseEvent>(this.host.nativeElement, 'mousedown'),
fromEvent<TouchEvent>(this.host.nativeElement, 'touchstart')
fromEvent<MouseEvent>(this.host.nativeElement, 'mousedown', passiveEventListenerOptions),
fromEvent<TouchEvent>(this.host.nativeElement, 'touchstart', passiveEventListenerOptions)
)
.pipe(takeUntil(this.destroy$))
.subscribe((event: MouseEvent | TouchEvent) => {
Expand Down

0 comments on commit 518997b

Please sign in to comment.