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

Certain font sizes nuke the text rendering altogether! :-o #196

Open
xparq opened this issue Mar 13, 2023 · 5 comments
Open

Certain font sizes nuke the text rendering altogether! :-o #196

xparq opened this issue Mar 13, 2023 · 5 comments
Labels
appearance | UX | design Affects usability/experience in an important way bug (Including appearance problems etc.) regression

Comments

@xparq
Copy link
Owner

xparq commented Mar 13, 2023

In a DEBUG-built demo, with the "sfml-widgets' default" theme, second but last slider stop... (So, curiously: not the largest size! :-o)
Wow, in every theme, actually... In "Default" & "Classic", also the previous stop, too: font size 16, 17...

image

BUT! Doesn't happen in the test app!!! :-o :-/


Some specific SFML versions I've had this issue with (unfortunately I didn't record any of the (many!) others):

  • vc17-64-90f05ac4e8cc4a5c445a125f6e8ac661c8e95130, vc16-64-90f05ac4e8cc4a5c445a125f6e8ac661c8e95130
@xparq xparq added bug (Including appearance problems etc.) appearance | UX | design Affects usability/experience in an important way regression labels Mar 13, 2023
@xparq xparq pinned this issue Mar 13, 2023
@xparq
Copy link
Owner Author

xparq commented Mar 13, 2023

It's pretty crazy stable too, despite the mysterious difference between the demo app and the test app...

Note: the b. bar labels and the img. button text survive it, because they don't track font size changes. However, not at all unscaved even that, e.g. notice how "0" and/or the "%" sign still degraded in this other run (apart from the weird overwrites only on "w" for some reason; and again no errors on the console):

image

Rebuilt from scratch in release mode, and linked to a newly downloaded SFML snapshot, and if fails the exact same way! (And also doesn't fail with the test app!)


And now, after (quite) a few updates, it has changed so that only one stop of the font-size slider exhibits the problem.

@xparq
Copy link
Owner Author

xparq commented Mar 16, 2023

OK, it has finally shown its true nature:

It's now crashing with the non-debug build of the new threaded demo!

Still nothing with the test app! :-o

Can crash in the DEBUG demo build, too, fortunately! :)

(But didn't seem to do it so readily as in non-debug, when I tried.)

@xparq
Copy link
Owner Author

xparq commented Mar 17, 2023

Demo run (DEBUG build) in the VS debugger: "Access violation" in SFML, deep down below getLocalBounds:

image

Then a plain run, outside of VS (telling this to note that this failure has manifested in 3-4 different ways already):

image

Also, then, back in the VS debugger, for once, at last, an error message from SFML made it through (or was it also just another effect, not the cause?! looks very damn closely related nonetheless):

Failed to add a new character to the font: the maximum texture size has been reached

And this too, in the "Diagnostics Hub" VS pane -- probably unrelated:

Failed to validate the Controller process being connected to.

@xparq
Copy link
Owner Author

xparq commented Mar 21, 2023

In a new build, with my latest MSVC portable + recent SFML + my new Jumpstart MSVC makefile now it crashes even if I just decrease the font size a step or two, right from all else being defaults...
-> UPDATE: This seems to be unrelated now, and more likely due to the simplistic (unguarded) threading of the "toy animation"!

Still no issues with the test app!


(Posted to the SFML forum -> https://en.sfml-dev.org/forums/index.php?topic=28983.msg179496)

@xparq
Copy link
Owner Author

xparq commented Mar 22, 2023

Important: If I start the demo with the problematic font size in the default theme config, that "priming" removes the problem entirely! :-o

@xparq xparq unpinned this issue Mar 24, 2023
@xparq xparq pinned this issue Apr 5, 2023
@xparq xparq unpinned this issue Apr 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
appearance | UX | design Affects usability/experience in an important way bug (Including appearance problems etc.) regression
Projects
None yet
Development

No branches or pull requests

1 participant