Skip to content

Commit e095224

Browse files
committed
reduce scope of mousemove listener
1 parent 9fe0973 commit e095224

File tree

1 file changed

+6
-4
lines changed
  • packages/react/src/VideoPlayer/components/Range

1 file changed

+6
-4
lines changed

packages/react/src/VideoPlayer/components/Range/Range.tsx

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -37,12 +37,14 @@ export const Range = ({
3737
}, [startValue])
3838

3939
useEffect(() => {
40-
if (!max || !tooltip || !inputRef.current) {
40+
const input = inputRef.current
41+
42+
if (!max || !tooltip || !input) {
4143
return
4244
}
4345

4446
const handleMouseMove = event => {
45-
if (event.target !== inputRef.current) {
47+
if (event.target !== input) {
4648
setHoverValue(0)
4749
setMousePos(0)
4850
return
@@ -53,10 +55,10 @@ export const Range = ({
5355
setHoverValue((event.offsetX / event.target.clientWidth) * max)
5456
}
5557

56-
window.addEventListener('mousemove', handleMouseMove)
58+
input.addEventListener('mousemove', handleMouseMove)
5759

5860
return () => {
59-
window.removeEventListener('mousemove', handleMouseMove)
61+
input.removeEventListener('mousemove', handleMouseMove)
6062
}
6163
}, [max, tooltip, inputRef])
6264

0 commit comments

Comments
 (0)