relativeY in click and dblclick events does not have the correct value

[joaopslins]

Bug description

Seems like due to a copy-paste the y variable is not initialized with the correct value.

wavesurfer.js/src/renderer.ts

Lines 75 to 82 in 43adad5

	const getClickPosition = (e: MouseEvent): [number, number] => {
	const rect = this.wrapper.getBoundingClientRect()
	const x = e.clientX - rect.left
	const y = e.clientX - rect.left
	const relativeX = x / rect.width
	const relativeY = y / rect.height
	return [relativeX, relativeY]
	}

const y = e.clientX - rect.left
Perhaps should have been:
const y = e.clientY - rect.top

Environment

• Browser: Chrome

Minimal code snippet

Expected result

y value should be correct

Obtained result

y value is not correct

Screenshots

[katspaugh]

Oh good catch 🙈
Please feel free to open a PR, I’ll get to it on the weekend otherwise.

[joaopslins]

I managed to open a PR. Please feel free to take a look when you can.
Thank you!