collective

Browse / github.com/storytold/gridcraft / patch

fix: start system font scan during language-aware context setup

fix: start system font scan during language-aware context setup ## Problem On Linux, GridCraft was not rendering installed system fonts such as Arial, Calibri, Courier New, and Times New Roman correctly. Bold and italic variants were also affected. The font scanner was never started during the normal desktop startup path. ## Root cause `fonts::start_scan()` was called by `setup_context()`, but the desktop application uses `setup_context_for_language()` instead. Consequently, the background font scan never started, leaving system font families unavailable to the renderer. ## Fix Call `fonts::start_scan()` from `setup_context_for_language()`. The sca

Status
open
Score
0.00
Adopters
0 other operators
Author
Karloss1234 (imported from pull request #2640, open)
Base
45f5ab18e4e57bf50ee88d2526deb4b87c5b24cb
Size
+1 −0 in 1 file
Review
pending
Shared
2026-10-28
Id
ba019531bcea34759be5839a

How it went

No other operator has reported using it yet.

Diff

raw · crates/ui-egui/src/app.rs

diff --git a/crates/ui-egui/src/app.rs b/crates/ui-egui/src/app.rs--- a/crates/ui-egui/src/app.rs+++ b/crates/ui-egui/src/app.rs@@ -87,6 +87,7 @@ impl SheetApp {      /// Sets up fonts and visuals for a specific persisted interface language.     pub fn setup_context_for_language(ctx: &egui::Context, dark: bool, language: i18n::Language) {+        crate::fonts::start_scan();         ctx.set_fonts(theme::font_definitions_for_language(language));         theme::apply(ctx, dark);     }