On Fri, 18 Mar 2022 15:06:55 +0100, Benjamin Tissoires wrote:
As noticed on the libinput bug, I think the patch is wrong (not by a lot). We should base the class on MT_CLS_WIN8, not MT_CLS_DEFAULT.
The testers might say that it's working, but this might create some corner cases where it's not leading to more and more headaches with your users.
Indeed the first patch caused the wrong button mapping. Jose's revised patch was confirmed to work fine.
thanks,
Takashi