[patch] Don't register mouse click when Wacom pen slightly touches tablet

[idefix]

I often accidentally touches the Wacom tablet slightly with the pen, causing Haiku to register a mouse click. Therefore I've created a patch​ that implements a minimum click pressure.

[idefix]

(patch) set minimum click pressure for Wacom pen

[stippi]

The code already implements that. Cutting of a range of the pressure is not the way to go. You need to find the place in the code where it uses the proximity as a threshold for the click. This value seems to be too low for your tablet. Since this is obviously tablet specific, and I even remember this value being different depending on where you touched the tablet, this should definitely go into a future preference panel.

[idefix]

Replying to stippi:

The code already implements that.

You mean (in case of a Wacom Graphire) line 288 in TabletDevice.cpp?

Cutting of a range of the pressure is not the way to go.

No, and it wasn't my first intention to do that. I was going to use only this line:

if (pressure <= MIN_CLICK_PRESSURE && mode == MODE_PEN) 
firstButton = false; 

But this caused Wonderbrush to ignore the pressure-levels below MIN_CLICK_PRESSURE, so drawing thin lines didn't work anymore.

You need to find the place in the code where it uses the proximity as a threshold for the click. This value seems to be too low for your tablet. Since this is obviously tablet specific, and I even remember this value being different depending on where you touched the tablet, this should definitely go into a future preference panel.

Well, I had a more generic solution in mind. But a preference panel does sound good. :)

[stippi]

Hm, maybe line 288 should be

firstButton = (data[1] & 0x01) != 0;

If that doesn't work, you can cut off the pressure, but do it like this:

pressure = max_c(0.0, (pressure - 0.1)) / 0.9;

which will make it range from 0.0 to 1.0 again.

[idefix]

Replying to stippi:

Hm, maybe line 288 should be

firstButton = (data[1] & 0x01) != 0;

This causes firstButton to be true at a pressure of approximately 0.12. But this also causes WonderBrush to ignore the pressure-levels below that. Maybe WonderBrush should only act on the pressure-level and not on the button-data when one of the parameters is configured for pressure-sensitivity and the user is using a Wacom pen? That way you can use the full pressure-range when drawing and still have a trigger-point for firstButton when using any other application.

If that doesn't work, you can cut off the pressure, but do it like this:

pressure = max_c(0.0, (pressure - 0.1)) / 0.9;

which will make it range from 0.0 to 1.0 again.

This works, but as you said, it will cut a range of the pressure which the driver shouldn't do at default. Maybe one day this can be implemented in a preference panel.

[mmlr]

Reassigning as the wacom driver's not really my area.

[waddlesplash]

Reverting earlier milestone change