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 support #63

Draft
wants to merge 8 commits into
base: master
Choose a base branch
from
Draft

Web support #63

wants to merge 8 commits into from

Conversation

hecrj
Copy link
Owner

@hecrj hecrj commented Jul 2, 2019

Depends on #62 and #60.

This PR allows Coffee to compile to Wasm! 🎉

Examples

I would greatly appreciate it if you report any issues you find!

Known issues

  • Text does not render on some environments.

Implementation details

The changes basically use the new gl backend coming to wgpu to run graphics on the web using WebGL2.

The PR uses my own forks of:

This PR does not contain any additional breaking changes to the ones found in #62. The API remains unchanged, meaning web support will be mostly free for projects that are already using Coffee.

Pending work

hecrj added 7 commits July 2, 2019 04:34
Apparently, `gfx-backend-gl` cannot easily distinguish between
single-layered texture arrays and normal textures.
Apparently, `BGRA` does not work properly. We should investigate why.
@hecrj hecrj added the feature New feature or request label Jul 2, 2019
@hecrj hecrj self-assigned this Jul 2, 2019
@hecrj hecrj added the help wanted Extra attention is needed label Jul 3, 2019
@hecrj hecrj added this to the 0.4.0 milestone Jul 7, 2019
@hecrj hecrj mentioned this pull request Oct 21, 2019
@hecrj hecrj modified the milestones: 0.4.0, 0.5.0 May 10, 2020
@dlight
Copy link
Contributor

dlight commented May 14, 2020

It seems that it was closed by rust-windowing/winit#1221, so it should be checked.

I suspect that many other issues are solved by now too!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants