Fix: make the default font style configurable - #829
Closed
nborodikhin wants to merge 1 commit into
Closed
Conversation
MinUI added TTF_STYLE_BOLD while its bundled BPreplay face was already bold. NextUI briefly used a regular face, then replaced it with Rounded Mplus 1c Bold before exposing Normal and Bold as a setting. The style selection therefore had no visual effect with either stock font. Replace the default face with the matching Google Fonts Regular weight. Normal now uses the face as supplied, while Bold lets SDL_ttf synthesize the requested weight.
nborodikhin
marked this pull request as ready for review
September 6, 2026 05:33
2 tasks
Author
|
Closing in favor of #830 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
The issue:
Brief history of the default NextUI font:
There was no option to control the style of the font, it was dictated by the font file.
The setting is passed as
TTF_SetFontStyle(font.large, CFG_getFontStyle());with the caveat:This change replaces the default font
Rounded Mplus 1c Boldfont with its regular-weight variantRounded Mplus 1c Regular, which makes the setting effective. Since the default settingCFG_DEFAULT_FONT_STYLEisBold, the change has limited impact on users that use only supplied fonts.This is not ideal though - regular-made-bold font has irregular thickness, see screenshots below and #748
A possible solution would be for NextUI to ship both regular and bold font files per font family, and introduce a configuration function to resolve font file per style (not a single font style).
Change
skeleton/SYSTEM/res/font1.ttf: use Rounded Mplus 1c Regular as the default Next fontTesting