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

Update Integration to work on WASM fully. #2039

Closed
wants to merge 4 commits into from
Closed

Update Integration to work on WASM fully. #2039

wants to merge 4 commits into from

Conversation

genusistimelord
Copy link

This will add and load a Font for the Integration project which will allow Text to render when running as WASM. This has to be shown since WASM cant load system fonts.

@hecrj
Copy link
Member

hecrj commented Aug 20, 2023

This doesn't seem like the proper fix. If Wasm is truly broken by default, the fix should happen at a different abstraction level.

Discussions before code and all that!

@hecrj hecrj closed this Aug 20, 2023
@hecrj hecrj added this to the 0.11.0 milestone Aug 20, 2023
@hecrj hecrj added improvement An internal improvement wgpu labels Aug 20, 2023
@genusistimelord
Copy link
Author

genusistimelord commented Aug 20, 2023

generally I did discuss this with @jackpot51 (jermey) and dfrg. https://discord.com/channels/628993209984614400/1027584473631555604/1142122898140696698 The browser doesn't have access to fonts so they can not be loaded the normal way. This is the only way to fix this issue. So adding it to the example is a way to show them how to load the fonts. unless you are thinking of another internal way of preloading select fonts inside of iced. but this would mean iced would have to store these inside the library for WASM.

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

Successfully merging this pull request may close these issues.

2 participants