-
Notifications
You must be signed in to change notification settings - Fork 163
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
Run fonttools against TTF files to save 300KB~ #473
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I approve of this PR, as it will reduce the font sized in the repository. But I don't currently have an environment to test it in. We could really use some help from @LibretroAdmin or another top contributor to the main project.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems to work on Linux with XMB. I don't have any other devices to really test this on, but I assume it's okay?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like a conflict since your last commit?
c1c2df2
to
771e60b
Compare
Had to hard reset the fork, but it shouldn't conflict anymore. |
We'll try this. If it causes issues we will have to revert it. |
Conversation in #471
This needs testing before merge.