On Wed, Mar 24, 2021 at 01:42:19PM GMT, Klemens Nanni wrote:
> On Tue, Mar 23, 2021 at 12:41:25PM +0000, Raf Czlonka wrote:
> > According to wsmouse(4), the orders is as follows:
> > 
> >     This list of three parameters configures these mappings,
> >     in the order:
> > 
> >                    one-finger,two-finger,three-finger
> > 
> >     Setting a parameter to a positive value enables that tap
> >     gesture and maps it to the given mouse button.  To disable
> >     all three tap gestures at once, provide the single value
> >     of 0.  Conversely, a single non-zero value will enable
> >     one-finger, two-finger, and three-finger tap gestures with
> >     their default mappings of left button, right button, and
> >     middle button, respectively.
> I had to let this be for a day and reread it now.

Hi Klemens,

No worries - I had to re-read your patch and the man page several
times just to be sure I got it right ;^)

> > Your updated patch reads:
> > 
> >     #mouse.tp.tapping=0     # 1,3,2=interpret one/three/two (simultaneous) 
> >                             # tap(s) as left/middle/right mouse button click
> > 
> > Whilst technically correct, if I'm reading the man page correctly,
> > it is a little bit confusing as the gestures are out of order and
> > the digits look like they represent the number of fingers when, in
> > reality, they represent the mouse *buttons* so, unless I got totally
> > confused by either of the above, or am missing something altogether,
> > the patch should probably read:
> > 
> >     #mouse.tp.tapping=1     # 1,3,2=interpret one/two/three (simultaneous) 
> >                             # tap(s) as left/right/middle mouse button click
> > 
> > As it is clear that digits do not map to numerals.
> So yes, you're right.

I suggested "1" instead of "0" on purpose, as it would match the
comment/description. As per the man page, "0" disable all tap gestures.

Regards,

Raf

> Anyone willing to OK this?
> 
> Index: wsconsctl.conf
> ===================================================================
> RCS file: /cvs/src/etc/examples/wsconsctl.conf,v
> retrieving revision 1.1
> diff -u -p -r1.1 wsconsctl.conf
> --- wsconsctl.conf    16 Jul 2014 13:21:33 -0000      1.1
> +++ wsconsctl.conf    24 Mar 2021 13:40:02 -0000
> @@ -9,3 +9,5 @@
>  #display.vblank=on           # enable vertical sync blank for screen burner
>  #display.screen_off=60000    # set screen burner timeout to 60 seconds
>  #display.msact=off           # disable screen unburn w/ mouse
> +#mouse.tp.tapping=0          # 1,3,2=interpret one-/two-/three-finger taps as
> +                             # 1st/3rd/2nd, i.e. left/right/middle button 
> click

Reply via email to