Skip to content

fix(fonts): Remove missing cozette.bdf font reference#160

Merged
ChuckBuilds merged 1 commit intomainfrom
fix/missing-cozette-font-warning
Dec 28, 2025
Merged

fix(fonts): Remove missing cozette.bdf font reference#160
ChuckBuilds merged 1 commit intomainfrom
fix/missing-cozette-font-warning

Conversation

@ChuckBuilds
Copy link
Copy Markdown
Owner

@ChuckBuilds ChuckBuilds commented Dec 28, 2025

  • Removed cozette_bdf from common_fonts dictionary in font_manager.py
  • Removed cozette_bdf option from web interface font selectors
  • Resolves warning: 'Common font file not found: assets/fonts/cozette.bdf'
  • Font can be re-enabled by adding the font file and re-adding to common_fonts

Summary by CodeRabbit

  • Bug Fixes
    • Removed Cozette BDF font option from the application due to unavailable font file. The font is no longer available in font selection menus and font preview settings.

✏️ Tip: You can customize this high-level summary in your review settings.

- Removed cozette_bdf from common_fonts dictionary in font_manager.py
- Removed cozette_bdf option from web interface font selectors
- Resolves warning: 'Common font file not found: assets/fonts/cozette.bdf'
- Font can be re-enabled by adding the font file and re-adding to common_fonts
@ChuckBuilds ChuckBuilds merged commit 1815a5b into main Dec 28, 2025
1 check was pending
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Dec 28, 2025

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

The pull request removes the Cozette BDF font option from the application. This involves deleting the cozette_bdf entry from the FontManager's common fonts mapping and removing corresponding UI options from font selection dropdowns and display mappings.

Changes

Cohort / File(s) Change Summary
Font Provider Removal
src/font_manager.py
Removed cozette_bdf entry from the common_fonts dictionary in FontManager.init with explanatory comment noting font file unavailability
Font UI Options
web_interface/templates/v3/partials/fonts.html
Removed cozette_bdf option from three select elements (override-family, preview-family) and deleted corresponding entry from getFontDisplayName mapping

Estimated Code Review Effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 The Cozette font has bid adieu,
No longer here, we bid it through,
The mapping cleaned, the templates bright,
One unavailable font takes flight! ✨

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/missing-cozette-font-warning

📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 947a0fb and f4349fd.

📒 Files selected for processing (2)
  • src/font_manager.py
  • web_interface/templates/v3/partials/fonts.html

Comment @coderabbitai help to get the list of available commands and usage tips.

@ChuckBuilds ChuckBuilds deleted the fix/missing-cozette-font-warning branch January 5, 2026 17:59
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.

1 participant