Code review comment for lp:~oif-team/frame/semi-mt-add-touches

Revision history for this message
Chase Douglas (chasedouglas) wrote :

> > I believe they are handled appropriately now.
>
> Right, it does look right for one finger too now. It would not hurt to
> simplify the logic a bit, though.

I had one idea on how the logic should work, and it was needed because I originally didn't handle BTN_TOUCH. The simplified logic in this last patch is a little easier to comprehend.

I hope this is good enough to merge now. Stephen noted a lot of people asking this past weekend for higher touch gestures in synaptics trackpads. With it merged we can at least give them the daily ppa to test with, and we can work on releasing the work and pushing it to the utouch stable ppa.

« Back to merge proposal