search

askmrsinh / touchpad-indicator

Touchpad management GNOME Shell Extension.
https://extensions.gnome.org/extension/131/touchpad-indicator/
GNU General Public License v2.0
83 stars 20 forks source link

Lenovo X1 2018 touch screen support #30

Closed tec closed 5 years ago

tec commented 5 years ago

Please add support for Lenovo X1 2018 touch screen:

XInput identifies touch screen as "Raydium Corporation Raydium Touch System".

I suggest to add 'touch system' to lib.js const TOUCHSCREENS: https://github.com/user501254/TouchpadIndicator/blob/de79bd556b2ea520bdbb6d032a26f916987f9c3e/lib.js#L60

const TOUCHSCREENS = Array('touchscreen', 'maxtouch', 'touch digitizer', 'touch system');

Thanks

askmrsinh commented 5 years ago

Hi @tec Have you tested this on your system? I would be happy to merge a PR if you can make one.

tec commented 5 years ago

Yes, I tested it on my X1 Thinkpad. Before the fix the third option was missing in the indicator menu:

touchpadindicator with touchscreen

I'll try to create a PR.

askmrsinh commented 5 years ago

Merged 7e98267dfef5c75e5f429fe69dc17a91829aa7f1.

tec commented 5 years ago

Yay, thank you!