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

Reproduced texture size issue (WebGPU) using Material Design Icons font + Dear Imgui's wchar32 feature/setting. #29

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

andreamancuso
Copy link
Collaborator

Re-created issue based on discussion: ocornut/imgui#7649

image

@andreamancuso andreamancuso added the wontfix This will not be worked on label Jun 3, 2024
@andreamancuso
Copy link
Collaborator Author

Keeping this PR open to serve as a reminder re: we must stick to Font Awesome icons for now.

…e issue. Unfortunately there's a 15 second long period of very high (100%) CPU utilisation before the application gets rendered.
@andreamancuso
Copy link
Collaborator Author

andreamancuso commented Jun 3, 2024

Setting icons_config.OversampleH = 1; initially appeared to have fixex the texture size issue. This also caused a 15 second long period of very high (100%) CPU utilisation before the application gets rendered.

Setting io.Fonts->TexDesiredWidth = 4096; didn't seem to make a difference.

After a while, the texture size issue was back and I was unable to get the application to render anymore.

@andreamancuso
Copy link
Collaborator Author

The only concrete solution for this is to include web font files containing exclusively the icons actually being used within the application. This used to be a thing over a decade ago in the web world - I should hope this won't be overly difficult in 2024...

Copy link

sonarqubecloud bot commented Aug 3, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
E Maintainability Rating on New Code (required ≥ A)

See analysis details on SonarCloud

Catch issues before they fail your Quality Gate with our IDE extension SonarLint

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant