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

Consider switching to winit for WASM target #243

Open
3 of 8 tasks
alvinhochun opened this issue Aug 18, 2020 · 2 comments
Open
3 of 8 tasks

Consider switching to winit for WASM target #243

alvinhochun opened this issue Aug 18, 2020 · 2 comments

Comments

@alvinhochun
Copy link
Collaborator

alvinhochun commented Aug 18, 2020

winit supports both stdweb and web-sys by using a <canvas> element for the "window". Using winit for WASM may simplify some of the canvas handling code as glutin also uses winit behind the the scene (however glutin itself does not work on WASM so only the winit parts can be shared).

Potential blockers:

@sebcrozet
Copy link
Owner

sebcrozet commented Nov 22, 2020

Sounds like a great idea.

Update winit and change kiss3d code flow and API to use event loop v2

As you suggested in #252 (comment) we should probably just deprecate Window::render and always use Window::render_loop.

@alvinhochun
Copy link
Collaborator Author

It's not absolutely necessary to deprecate Window::render, but it does mean there will be two separate code paths for handling the event loop since Window::render_loop will no longer use poll_events.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants