Skip to content

Conversation

@Inertia-Squared
Copy link

@Inertia-Squared Inertia-Squared commented Apr 19, 2025

  • Re-added @azealo 's font compatibility code after being overwritten by later commit
  • Changed fallback typeface to Franklin Gothic Medium Condensed - fits the UI better (vs. Arial) and is somewhat similar to the font used in the video
  • Added check to allow use of old font if it exists in typical location (also made it work regardless of user!)
  • Removed a redundant math import, might as well while I was there ¯\_(ツ)_/¯

- Changed typeface to Franklin Gothic Medium Condensed - fits the UI better and is somewhat similar to the font used in the video
- Added check to allow use of old font if it exists in typical location (also made it work regardless of user!)
@tsal
Copy link

tsal commented Apr 19, 2025

seems like for usability, custom font should be a getopts / option, and it should default to ./visuals/whatever.ttf; just using a font from the local folder makes it so this runs on macos / linux.

@Inertia-Squared
Copy link
Author

Just trying to keep the default behaviour but add a fallback for those that don't have the font like myself, the intent was to make it non-invasive to the original functionality while improving compatibility.

@Inertia-Squared
Copy link
Author

@carykh a lot of people are opening issues due to this error, would it be ok to merge?

@azealo
Copy link
Contributor

azealo commented May 2, 2025

Looks good to me

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants