search

vuetifyjs / vuetify

🐉 Vue Component Framework
https://vuetifyjs.com
MIT License
39.87k stars 6.96k forks source link

[Bug Report][3.4.10] v-slider / v-range-slider mousedown (not left button) #19021

Open xmorell opened 10 months ago

xmorell commented 10 months ago

Environment

Vuetify Version: 3.4.10 Vue Version: 3.4.7 Browsers: Chrome 120.0.0.0 OS: Mac OS 10.15.7

Steps to reproduce

Click over the slider with the right button of the mouse or with the mouse wheel.

Expected Behavior

The slider value should not change.

Actual Behavior

The slider value will change.

Reproduction Link

https://play.vuetifyjs.com/#...

mirocklez commented 9 months ago

The right button mouse, by default, shows the context menu - so you suggest to disable context menu and no matter which button is pressed - to update the slider?

I disagree this is a good approach - to prevent the default browser functionality just to ensure there is no matter which button user clicks.

xmorell commented 9 months ago

I suggest just the opposite. That the value is only changed with the left button.

now they have something similar to @mousedown="action" I suggest it should be @mousedown.left="action"

mirocklez commented 9 months ago

I suggest just the opposite. That the value is only changed with the left button.

now they have something similar to @mousedown="action" I suggest it should be @mousedown.left="action"

Yep - my mistake :) I totally agree with you 👍