<div dir="ltr"><div class="gmail_default" style="font-size:large"></div><div class="gmail_default">Hmmm, after using udev_device_has_tag(udev_device, "ID_INPUT_TOUCHPAD"),</div><div class="gmail_default">I'm getting linking errors. Quick google of this leads to: libraries on the command line should be after the object files being compiled.</div><div class="gmail_default">Is this a bug?</div><div class="gmail_extra"> <div class="gmail_quote">On Sun, Feb 5, 2017 at 7:20 PM, Peter Hutterer <<a href="mailto:peter.hutterer@who-t.net" target="_blank">peter.hutterer@who-t.net</a>> wrote: <blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Sun, Feb 05, 2017 at 07:13:04PM -0500, Jiayi Zhao wrote: > How would ID_INPUT_TOUCHPAD work? > I have something like udev_device_has_tag(udev_device, ID_INPUT_TOUCHPAD) > Unfortunately, I'm not sure where this constant is declared and its unable > to find the tag. :/ it's a udev property name, so use it as a string, not a #define. value is either 1 or 0, so you do need to check for validity. Cheers, Peter <div class="gmail-HOEnZb"><div class="gmail-h5"> > On Jan 29, 2017 4:42 PM, "Peter Hutterer" <<a href="mailto:peter.hutterer@who-t.net">peter.hutterer@who-t.net</a>> wrote: > > > On Fri, Jan 27, 2017 at 09:46:49PM -0500, Jiayi Zhao wrote: > > > [libinput] > > > natural_scroll=true > > > > > > CHANGES: > > > - libinput_device_config_scroll_has_natural_scroll() is no longer > > compared to != 0 > > > - added configuration option to weston.ini man page > > > > > > Signed-off-by: Jiayi Zhao <<a href="mailto:jeff.no.zhao@gmail.com">jeff.no.zhao@gmail.com</a>> > > > --- > > > compositor/main.c | 13 +++++++++++++ > > > man/weston.ini.man | 3 +++ > > > <a href="http://weston.ini.in" rel="noreferrer" target="_blank">weston.ini.in</a> | 1 + > > > 3 files changed, 17 insertions(+) > > > > > > diff --git a/compositor/main.c b/compositor/main.c > > > index 4bd6e68..84bb8dd 100644 > > > --- a/compositor/main.c > > > +++ b/compositor/main.c > > > @@ -1133,6 +1133,8 @@ configure_input_device(struct weston_compositor > > *compositor, > > > struct weston_config *config = wet_get_config(compositor); > > > int enable_tap; > > > int enable_tap_default; > > > + int natural_scroll; > > > + int natural_scroll_default; > > > > > > s = weston_config_get_section(config, > > > "libinput", NULL, NULL); > > > @@ -1147,6 +1149,17 @@ configure_input_device(struct weston_compositor > > *compositor, > > > libinput_device_config_tap_set_enabled(device, > > > enable_tap); > > > } > > > + > > > + if (libinput_device_config_scroll_has_natural_scroll(device)) { > > > + natural_scroll_default = > > > + libinput_device_config_scroll_get_default_natural_ > > scroll_enabled( > > > + device); > > > + weston_config_section_get_bool(s, "natural_scroll", > > > + &natural_scroll, > > > + natural_scroll_default); > > > + libinput_device_config_scroll_set_natural_scroll_ > > enabled(device, > > > + natural_scroll); > > > + } > > > > libinput exposes natural scrolling on mice as well and with this code > > you're enabling it on any device that has the option. so your mouse > > wheels will be upside down too when you enable this. That's probably not > > what you intended, but if so, prepare for angry emails :) > > > > there are two options to work around this: > > * check the udev device for ID_INPUT_TOUCHPAD > > * assume anything that has tapping is a touchpad > > > > Cheers, > > Peter > > > > > } > > > > > > static void > > > diff --git a/man/weston.ini.man b/man/weston.ini.man > > > index 2edb085..62e0f92 100644 > > > --- a/man/weston.ini.man > > > +++ b/man/weston.ini.man > > > @@ -187,6 +187,9 @@ Available configuration are: > > > .TP 7 > > > .BI "enable_tap=" true > > > enables tap to click on touchpad devices > > > +.TP 7 > > > +.BI "natural_scroll=" true > > > +enables natural scrolling on touchpad devices > > > .RS > > > .PP > > > > > > diff --git a/<a href="http://weston.ini.in" rel="noreferrer" target="_blank">weston.ini.in</a> b/<a href="http://weston.ini.in" rel="noreferrer" target="_blank">weston.ini.in</a> > > > index 257c4ec..f957ec6 100644 > > > --- a/<a href="http://weston.ini.in" rel="noreferrer" target="_blank">weston.ini.in</a> > > > +++ b/<a href="http://weston.ini.in" rel="noreferrer" target="_blank">weston.ini.in</a> > > > @@ -61,6 +61,7 @@ path=@libexecdir@/weston-keyboard > > > > > > #[libinput] > > > #enable_tap=true > > > +#natural_scroll=false > > > > > > #[touchpad] > > > #constant_accel_factor = 50 > > > -- > > > 2.10.2 > > > > > > _______________________________________________ > > > wayland-devel mailing list > > > <a href="mailto:wayland-devel@lists.freedesktop.org">wayland-devel@lists.freedesktop.org</a> > > > <a href="https://lists.freedesktop.org/mailman/listinfo/wayland-devel" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/wayland-devel</a> > > > > > </div></div></blockquote></div> </div></div>