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

Emscripten support #52

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Emscripten support #52

wants to merge 3 commits into from

Conversation

dxdxdt
Copy link

@dxdxdt dxdxdt commented Aug 14, 2024

Sharing my experimental Emscripten target support. Try it out:

https://snart.me/demos/emscripten/ft2-clone/

vær så god!

- Fix stuttering in background: use emscripten_sleep() only to yield
@8bitbubsy
Copy link
Owner

I mean, cool and all, but the mouse pointer has coordinate mapping issues for me on your web test link. It makes it quite unusable so far...

@dxdxdt
Copy link
Author

dxdxdt commented Aug 14, 2024

@8bitbubsy well, that's not the only problem(as per the notes). The mapping issue must be both from the scaling factor and canvas event translation. I know I'm still taking baby steps. Just wanted to get your attention and see if the Emscripten support is something you'd be interested. I don't mind maintaining a separate fork. Close the PR if you think it's not worth it. Leave it open if it looks promising.

Well, I've never used the original FT myself, but I quite don't understand why ft2-clone is so reliant on mouse when the original is from the 90's. Just a thought, not trying to troll or anything.

@8bitbubsy
Copy link
Owner

8bitbubsy commented Aug 14, 2024

As long as it's a nice and simple patch that doesn't change the behavior of the FT2 clone on other platforms, I'll merge it eventually if you manage to iron out the bugs. :)

The FT2 clone is reliant on the mouse just like original FT2, simply because it tries to be a copy of FT2. If you want less mouse-driven trackers, there are probably some alternatives out there.

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

Successfully merging this pull request may close these issues.

2 participants