Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

On Web, implement DeviceEvents #2871

Merged
merged 6 commits into from
Jun 14, 2023

Conversation

daxpedda
Copy link
Member

@daxpedda daxpedda commented Jun 11, 2023

Tracking

  • MouseMotion
  • Motion
  • MouseWheel
  • Button
  • Key
  • Text (not implemented on any backend)

Changes

This PR intends to implements all DeviceEvents. Additionally DeviceEvents are only emitted according to the setting in EventLoopWindowTarget::listen_device_events().


Fixes #2036.

@daxpedda daxpedda changed the title Moved DeviceEvent::MouseMotion to browser window On Web, implement DeviceEvents Jun 11, 2023
@daxpedda
Copy link
Member Author

Just hit this bug on Mozilla: https://bugzilla.mozilla.org/show_bug.cgi?id=1753724.
So basically I broke DeviceEvent::MouseMotion in Firefox in #2840 and didn't notice.

Will figure out a fix.

@daxpedda
Copy link
Member Author

I split off any pointer related fixes I originally implemented in this PR into #2878.

@daxpedda daxpedda force-pushed the web-device-event branch 3 times, most recently from fe19240 to cf940e8 Compare June 13, 2023 12:35
@daxpedda daxpedda marked this pull request as ready for review June 13, 2023 12:39
@daxpedda daxpedda force-pushed the web-device-event branch 2 times, most recently from 2aad007 to 414fad0 Compare June 13, 2023 22:09
@daxpedda daxpedda merged commit 6300cf9 into rust-windowing:master Jun 14, 2023
55 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

Most Device events do not work on the web.
1 participant