Emscripten build runtime error: Unable to preventDefault inside passive event listener about imgui

[ypujante]

@CallumFerguson I am currently working on a PR to use a different implementation of glfw with emscripten. One of the changes is to get rid of the mouse wheel callback in ImGui since the new implementation takes care of it.

I definitely can reproduce the problem with the build BEFORE my PR (https://pongasoft.github.io/imgui/pr-7520/imgui-master-b720c0f/). But I can see that the build AFTER my PR does not have this issue (https://pongasoft.github.io/imgui/pr-7520/ypujante-master-fd249f9/). So it looks like my new glfw implementation does the "right" thing in regards to mousewheel.

Could you confirm on your end that the problem goes away with the PR (by trying this demo)? (note that on macOS Firefox does not support webgpu yet so it requires Chrome, looks like you are on Windows and I believe it works fine there).

@ocornut I apologize for the delay in submitting the final PR: emscripten has been slow in releasing the needed version and then I am currently away on vacation and prefer to wait until I return (in a couple weeks) instead of trying to do a push from my laptop thousands of miles away...

[CallumFerguson]

I tried your demos. I can see the warning on the build before the PR, and I do not see the warning on the build after the PR.

[ypujante]

As promised, I did fix this bug part of the PR. See #7647.

Turns out that using the document as the target of the event was the cause of the issue. Using the canvas selector instead addresses the issue.

[ocornut]

Maybe @ypujante would know?

[ocornut]

As discussed in said PR, it is likely we'd want to keep support for both, so it would be good to fix that particular warning either way.

Enjoy your holidays!

[ypujante]

@ocornut Got it. I will make sure that the old path does not have the warning anymore when I submit the PR. I still would like to confirm that @CallumFerguson does not see the warning with the new code.