#8978: 'Clicks' not set in MouseDown messages on Thinkpad TrackPoint buttons ---------------------------------+---------------------------- Reporter: mks | Owner: siarzhuk Type: bug | Status: new Priority: normal | Milestone: R1 Component: Drivers/Mouse/PS2 | Version: R1/Development Resolution: | Keywords: Blocked By: | Blocking: Has a Patch: 0 | Platform: All ---------------------------------+---------------------------- Comment (by korli): Since the device is detected as alps, handling happens in src/add- ons/kernel/bus_managers/ps2/ps2_alps.cpp. movement->clicks isn't set at all at in [http://cgit.haiku- os.org/haiku/tree/src/add-ons/kernel/bus_managers/ps2/ps2_alps.cpp#n243 this case]. See [http://cgit.haiku-os.org/haiku/tree/src/add- ons/kernel/bus_managers/ps2/movement_maker.cpp#n486 here] for how it should be done. Another problem though is that the state of cookie->movementMaker is inconsistent (TouchpadMovement::fButtonsState for instance). A simple solution would be to change TouchpadMovement::_UpdateButtons to public scope, and call it after [http://cgit.haiku-os.org/haiku/tree/src/add- ons/kernel/bus_managers/ps2/ps2_alps.cpp#n250 settings clicks]. -- Ticket URL: <http://dev.haiku-os.org/ticket/8978#comment:13> Haiku <http://dev.haiku-os.org> Haiku - the operating system.