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

Homestuck font render issues (device font related) #10481

Closed
MrCheeze opened this issue Mar 31, 2023 · 1 comment
Closed

Homestuck font render issues (device font related) #10481

MrCheeze opened this issue Mar 31, 2023 · 1 comment
Labels
bug Something isn't working text Issues relating to text rendering/input

Comments

@MrCheeze
Copy link
Contributor

Describe the bug

This movie (direct swf link) has multiple font rendering issues. Other flashes from Homestuck tend to use fonts in the same way and have similar issues.

  1. Underlines are missing:
    image

  2. Text is cut off because it uses the Ruffle default font instead of Courier New device font (see Implement device font rendering #1862)
    image

Expected behavior

See the comparison above - underlines should be present in the text, and Courier New should be used instead of the Ruffle default font.

Underline presence is self-explanatory.

For getting Courier New, I would consider any solution that doesn't require editing the movie itself to be valid, for example:

  • Ruffle actually loading the operating system's device fonts, as Flash Player does.
  • Ruffle adding builtin support for common fonts.
  • Requiring the host website to provide an additional fonts file, as proposed by n0samu here.

Affected platform

Self-hosted version

Operating system

Windows 11

Browser

Mozilla Firefox

Additional information

No response

@MrCheeze MrCheeze added the bug Something isn't working label Mar 31, 2023
@Lord-McSweeney Lord-McSweeney added the text Issues relating to text rendering/input label Mar 31, 2023
@MrCheeze
Copy link
Contributor Author

"Requiring the host website to provide an additional fonts file" has now been implemented in #13604, allowing for the correct font to be used!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working text Issues relating to text rendering/input
Projects
None yet
Development

No branches or pull requests

2 participants