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

Make sure to call raw_input_hook on web #4646

Merged
merged 1 commit into from
Jun 18, 2024

Conversation

owen-d
Copy link
Contributor

@owen-d owen-d commented Jun 10, 2024

What's new

Details

Debugging this locally after my app's raw_input_hook wasn't being called, I realized it's not in the code path of eframe's web runner, only the native integration. Below is a toy example running on the web.

Screen.Recording.2024-06-09.at.7.57.07.PM.mov

Signed-off-by: Owen Diehl <ow.diehl@gmail.com>
@emilk emilk added web Related to running Egui on the web eframe Relates to epi and eframe labels Jun 18, 2024
@emilk emilk changed the title [eframe][web] Call raw_input_hook on web app_runner Make sure to call raw_input_hook on web Jun 18, 2024
@emilk emilk merged commit 413843d into emilk:master Jun 18, 2024
20 of 21 checks passed
@emilk emilk added the feature New feature or request label Jun 27, 2024
hacknus pushed a commit to hacknus/egui that referenced this pull request Oct 30, 2024
## What's new
* Extends @varphone's excellent `raw_input_hook` idea from
emilk#4008 to `web/app_runner`.

## Details
Debugging this locally after my app's `raw_input_hook` wasn't being
called, I realized it's not in the code path of eframe's web runner,
only the native integration. Below is a toy example running on the web.


https://github.com/emilk/egui/assets/8173478/d470b7e6-d393-4ead-9745-3aafc72ae6bc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
eframe Relates to epi and eframe feature New feature or request web Related to running Egui on the web
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants