Skip to content

Commit 6cf89e8

Browse files
committed
fixup! fix(material/slider): handle contextmenu events
1 parent 3e83fb6 commit 6cf89e8

File tree

2 files changed

+5
-4
lines changed

2 files changed

+5
-4
lines changed

src/material/slider/slider-input.ts

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -269,15 +269,13 @@ export class MatSliderThumb implements _MatSliderThumb, OnDestroy, ControlValueA
269269
this._hostElement.addEventListener('pointerdown', this._onPointerDown.bind(this));
270270
this._hostElement.addEventListener('pointermove', this._onPointerMove.bind(this));
271271
this._hostElement.addEventListener('pointerup', this._onPointerUp.bind(this));
272-
this._hostElement.addEventListener('contextmenu', this._onPointerUp.bind(this));
273272
});
274273
}
275274

276275
ngOnDestroy(): void {
277276
this._hostElement.removeEventListener('pointerdown', this._onPointerDown);
278277
this._hostElement.removeEventListener('pointermove', this._onPointerMove);
279278
this._hostElement.removeEventListener('pointerup', this._onPointerUp);
280-
this._hostElement.removeEventListener('contextmenu', this._onPointerUp);
281279
this._destroyed.next();
282280
this._destroyed.complete();
283281
this.dragStart.complete();
@@ -639,7 +637,7 @@ export class MatSliderRangeThumb extends MatSliderThumb implements _MatSliderRan
639637
}
640638

641639
override _onPointerDown(event: PointerEvent): void {
642-
if (this.disabled) {
640+
if (this.disabled || event.button !== 0) {
643641
return;
644642
}
645643
if (this._sibling) {

src/material/slider/slider-thumb.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -174,7 +174,10 @@ export class MatSliderVisualThumb implements _MatSliderVisualThumb, AfterViewIni
174174
this._hostElement.classList.remove('mdc-slider__thumb--focused');
175175
};
176176

177-
private _onDragStart = (): void => {
177+
private _onDragStart = (event: PointerEvent): void => {
178+
if (event.button !== 0) {
179+
return;
180+
}
178181
this._isActive = true;
179182
this._showActiveRipple();
180183
};

0 commit comments

Comments
 (0)