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

fonts: add per font sizing for themes #1393

Merged
merged 1 commit into from
Dec 2, 2024
Merged

Conversation

KrahJohlito
Copy link
Member

Pull Request checklist

Note: these are not necessarily requirements

  • I reformatted the code with clang-format
  • I checked to make sure my submission worked
  • I am the author of submission or have permission from the original author
  • Requires update of the PS2SDK or other dependencies
  • Others (please specify below)

Pull Request description

This adds per font sizing for custom themes.. I wanted per element but might cause performance issues so I think this is a fair compromise.

  • cfg entry example:
default_font=myDefaultFont.ttf
default_font_size=20

font1=myOtherFont.ttf
font1_size=12

main3:
 type=ItemsList
 font=0
main4:
 type=ItemText
 font=1

so in this example the games list has a font size of 20 using myDefaultFont.ttf and the game id uses a font size of 12 using myOtherFont.ttf.

- cfg entry example:
default_font=myDefaultFont.ttf
default_font_size=20

font1=mySmallerFont.ttf
font1_size=12

main3:
 type=ItemsList
 font=0
main4:
 type=ItemText
 font=1
@AKuHAK
Copy link
Member

AKuHAK commented Dec 2, 2024

doh, we dont have readme for themes inside repo?

@KrahJohlito
Copy link
Member Author

KrahJohlito commented Dec 2, 2024

doh, we dont have readme for themes inside repo?

Nope, would be a lot of information.. might be better off adding it to a wiki and having a link to the wiki in the readme

@AKuHAK AKuHAK merged commit 0f736b9 into ps2homebrew:master Dec 2, 2024
17 checks passed
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.

2 participants