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

When trying to drag window with touch interface (WASM on mobile) get stuck in moving window state #7

Closed
Xadiant opened this issue Dec 12, 2022 · 5 comments

Comments

@Xadiant
Copy link

Xadiant commented Dec 12, 2022

I compiled using emscripten (latest pre-compiled release as of today) with the example app in the main readme for imgui-app. I then hosted the resulting html/wasm/js and tested on my mobile phone.

When I give the top of an imgui window a series of taps, it then glues to my finger and I can move it, however, all subsequent taps move the window to the new tap's location and there is no way to stop tat from happening save from restarting the application (refreshing the page).

This does not happen on the live imgui emscripten example: https://www.dearimgui.org/webdemo/master/example_emscripten.html

Any tips on how to fix this? I don't mind working on a fix and submitting a pull request if I can figure this out.

@pplux
Copy link
Owner

pplux commented Dec 13, 2022

this must be a sokol issue somehow, I haven't test it so I can't provide that much info. Have you looked into sokol repo to see if the issue comes from event handling there?

@Xadiant
Copy link
Author

Xadiant commented Dec 13, 2022

Ah I see, I initially thought thought the events were handled in this repo for some reason. Ill close this for now, and update here if I find a workaround.

@Xadiant Xadiant closed this as completed Dec 13, 2022
@Xadiant
Copy link
Author

Xadiant commented Dec 13, 2022

Here is the issue as observed highly detailed as an issue on the sokol page: floooh/sokol#759

Includes video of problem, and my attempted work-arounds so far.

@Xadiant
Copy link
Author

Xadiant commented Dec 13, 2022

See the pull request to fix in sokol, this is a simple change anybody can make themselves int he meantime: floooh/sokol#760

@floooh
Copy link

floooh commented Dec 15, 2022

I just merged the fix. Many thanks again for the investigation and PR, @Xadiant!

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

No branches or pull requests

3 participants