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

web: window cursor position is set on mouse click #7513

Closed
rparrett opened this issue Feb 5, 2023 · 4 comments
Closed

web: window cursor position is set on mouse click #7513

rparrett opened this issue Feb 5, 2023 · 4 comments
Labels
A-Input Player input via keyboard, mouse, gamepad, and more C-Bug An unexpected or incorrect behavior O-Web Specific to web (WASM) builds

Comments

@rparrett
Copy link
Contributor

rparrett commented Feb 5, 2023

Bevy version

main

Relevant system information

AdapterInfo { name: "ANGLE (Apple, Apple M1 Max, OpenGL 4.1)", vendor: 4203, device: 0, device_type: IntegratedGpu, driver: "", driver_info: "", backend: Gl }

chrome, firefox, safari

What you did

(with wasm-server-runner)
cargo run --example 3d_scene --target=wasm32-unknown-unknown

Open javascript console, click canvas, observe error message

What went wrong

ERROR crates/bevy_winit/src/system.rs:182 could not set cursor position: NotSupported(NotSupportedError)

Additional information

This seems sort of harmless, but is very strange.

Doesn't happen in native builds on MacOS, but it seems worth testing other platforms.

@rparrett rparrett added C-Bug An unexpected or incorrect behavior S-Needs-Triage This issue needs to be labelled labels Feb 5, 2023
@tim-blackbird
Copy link
Contributor

I think it's the same issue as #7377. Clicking triggers change detection just like the scale_factor_override example does.

I'll get back to work on my PR to fix it. Lost some motivation as it ended up needing more changes than I thought :v

@hankjordan
Copy link
Contributor

Closing as duplicate, feel free to re-open if it isn't fixed by #7517

@hankjordan hankjordan closed this as not planned Won't fix, can't repro, duplicate, stale Feb 6, 2023
@alice-i-cecile alice-i-cecile added A-Input Player input via keyboard, mouse, gamepad, and more O-Web Specific to web (WASM) builds and removed S-Needs-Triage This issue needs to be labelled labels Feb 6, 2023
@rparrett
Copy link
Contributor Author

rparrett commented Feb 6, 2023

I am not really seeing the harm in leaving this open until closed automatically by that PR, as it's tracking a unique platform-specific behavior not really covered by the other issue text.

But I guess it can stay closed because this is definitely fixed by that PR and it seems likely to be merged.

@hankjordan
Copy link
Contributor

I am not really seeing the harm in leaving this open until closed automatically by that PR, as it's tracking a unique platform-specific behavior not really covered by the other issue text.

But I guess it can stay closed because this is definitely fixed by that PR and it seems likely to be merged.

Ah, my mistake - reopening.

@hankjordan hankjordan reopened this Feb 6, 2023
@bors bors bot closed this as completed in f69f132 Feb 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Input Player input via keyboard, mouse, gamepad, and more C-Bug An unexpected or incorrect behavior O-Web Specific to web (WASM) builds
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants