Skip to content

Conversation

@tronical
Copy link
Member

cc #4352

Copy link
Member

@burhankhanzada burhankhanzada left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need touch_id and remove button

touch_down_event.y_transformed(screen_size.height as u32) as _,
);
Some(WindowEvent::PointerPressed {
Some(WindowEvent::TouchPressed {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If its TouchPressed then need touch_id and position, button is not required

})
}
input::event::TouchEvent::Up(..) => Some(WindowEvent::PointerReleased {
input::event::TouchEvent::Up(..) => Some(WindowEvent::TouchReleased {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also here TouchReleased need touch_id and position

touch_motion_event.y_transformed(screen_size.height as u32) as _,
);
Some(WindowEvent::PointerMoved { position: self.last_touch_pos })
Some(WindowEvent::TouchMoved { position: self.last_touch_pos })
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

touch_id required here also

@tronical tronical force-pushed the simon/linuxkms-touch branch from 74d9e5b to 040a7d7 Compare October 28, 2025 10:42
Copy link
Member

@ogoffart ogoffart left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks alright

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants