From 09e199b730393ab6936c4dc1669e9efea523ef34 Mon Sep 17 00:00:00 2001
From: Richard Helm <richard.helm@vonage.com>
Date: Mon, 4 Nov 2024 19:04:54 +0000
Subject: [PATCH] Remove overridden function

---
 libs/components/src/lib/slider/slider.ts | 41 +++++-------------------
 1 file changed, 8 insertions(+), 33 deletions(-)

diff --git a/libs/components/src/lib/slider/slider.ts b/libs/components/src/lib/slider/slider.ts
index 9d7ff16787..8d87c6aa85 100644
--- a/libs/components/src/lib/slider/slider.ts
+++ b/libs/components/src/lib/slider/slider.ts
@@ -318,6 +318,8 @@ export class Slider extends FormAssociatedSlider {
 	}
 
 	protected keypressHandler = (e: KeyboardEvent) => {
+		this._focusVisible = true;
+
 		if (this.readOnly) {
 			return;
 		}
@@ -447,7 +449,9 @@ export class Slider extends FormAssociatedSlider {
 			if (this.readOnly || this.disabled || event.defaultPrevented) {
 				return;
 			}
+			this.#isNonVisibleFocus = true;
 			(event.target as HTMLElement).focus();
+			this.#isNonVisibleFocus = false;
 		}
 		const eventAction =
 			event !== null ? 'addEventListener' : 'removeEventListener';
@@ -533,10 +537,14 @@ export class Slider extends FormAssociatedSlider {
 			if (e) {
 				e.preventDefault();
 				this.setupTrackConstraints();
+				this.#isNonVisibleFocus = true;
 				(e.target as HTMLElement).focus();
+				this.#isNonVisibleFocus = false;
 				const value = this.#calculateValueFromMouseEvent(e);
 
 				this.value = `${this.#roundToNearestStep(value)}`;
+
+				this.isDragging = true;
 			}
 		}
 	};
@@ -597,39 +605,6 @@ export class Slider extends FormAssociatedSlider {
 
 	#isNonVisibleFocus = false;
 
-	constructor() {
-		super();
-
-		const fastSliderInternals = this as any;
-
-		const originalHandleMouseDown = fastSliderInternals.handleMouseDown;
-		// Called with e=null to clean up after releasing the mouse
-		fastSliderInternals.handleMouseDown = (e: MouseEvent | null) => {
-			this.#isNonVisibleFocus = true;
-			originalHandleMouseDown(e);
-			this.#isNonVisibleFocus = false;
-
-			if (e && !this.disabled && !this.readOnly) {
-				// Fix bug in FAST: isDragging is not set on mousedown
-				this.isDragging = true;
-			}
-		};
-
-		const originalHandleThumbMouseDown =
-			fastSliderInternals.handleThumbMouseDown;
-		fastSliderInternals.handleThumbMouseDown = (e: MouseEvent) => {
-			this.#isNonVisibleFocus = true;
-			originalHandleThumbMouseDown(e);
-			this.#isNonVisibleFocus = false;
-		};
-
-		const originalKeypressHandler = fastSliderInternals.keypressHandler;
-		fastSliderInternals.keypressHandler = (e: KeyboardEvent) => {
-			this._focusVisible = true;
-			originalKeypressHandler(e);
-		};
-	}
-
 	#registerThumbListeners() {
 		this.thumb.addEventListener('mouseover', this.#onMouseOver, {
 			passive: true,