Skip to content

Switch font to Inter Tight#39

Merged
nchapman merged 2 commits into
developfrom
feature/fonts
Nov 27, 2025
Merged

Switch font to Inter Tight#39
nchapman merged 2 commits into
developfrom
feature/fonts

Conversation

@nchapman
Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings November 27, 2025 16:55
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR switches the UI font from BPreplay Bold to Inter Tight Bold. The change involves updating the font file path, adjusting text baseline positioning to accommodate the new font's visual characteristics, and removing redundant bold styling since Inter Tight Bold is already a bold variant font. The new font file has been properly added to the resources directory.

  • Updates font file from BPreplayBold-unhinted.otf to InterTight-Bold.ttf
  • Adjusts text baseline offset from ~4dp to ~6dp for better visual alignment with Inter Tight's metrics
  • Removes unnecessary TTF_SetFontStyle bold styling calls

Reviewed changes

Copilot reviewed 2 out of 4 changed files in this pull request and generated no comments.

File Description
workspace/all/common/defines.h Updates FONT_PATH to reference the new InterTight-Bold.ttf font file
workspace/all/common/api.c Adjusts text baseline calculation for the new font's metrics and removes redundant TTF_SetFontStyle calls since Inter Tight Bold is already bold
skeleton/SYSTEM/res/BPreplayBold-unhinted.otf Removes the old BPreplay Bold font file (replaced by Inter Tight Bold)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@nchapman nchapman merged commit bf545e8 into develop Nov 27, 2025
4 checks passed
@nchapman nchapman deleted the feature/fonts branch November 27, 2025 17:02
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