-
-
Notifications
You must be signed in to change notification settings - Fork 961
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
feat: Hide default font name for SystemFontProvider, change default font #2385
Conversation
1d5f23f
to
7df5ca8
Compare
7df5ca8
to
3b9e83f
Compare
So, this PR is mostly to ensure that by default samples use a font that is compatible with the system it was created on, right ?
You meant to say that |
@Eideren Correct 😅 On most systems it will be the Arial font but on Linux does not. On most distros most common one would be
Both |
Right, so not specifying any font would automatically pick one of the fonts available on the current OS ? |
@Eideren If I understand correctly not specifying the parameters will result in not overwriting the |
I tested it by running Gamestudio and indeed it sets to |
Thanks for the clarifications |
PR Details
My PR tries to hide default default font name for SystemFontProvider. I also changed default font name from
Courier New
toArial
for sake of consistency - (Almost) all samples are usingArial
font anyway.In addition,
Arial
is a sans-serif font - these types of fonts are more popular and easier to read.Types of changes
Checklist