Skip to content

core/gtk: add language config entry to override GUI localization - #10428

Merged
pluiedev merged 1 commit into
ghostty-org:mainfrom
jcollie:language-override
Feb 14, 2026
Merged

pluiedev merged 1 commit into
ghostty-org:mainfrom
jcollie:language-override

Conversation

@jcollie

@jcollie jcollie commented Jan 23, 2026

Copy link
Copy Markdown
Member

Fixes #10276

Screenshot From 2026-01-23 16-45-11

@jcollie
jcollie requested a review from a team as a code owner January 23, 2026 23:24
@jcollie

jcollie commented Jan 23, 2026

Copy link
Copy Markdown
Member Author

I'm not 100% sure that this should be merged as it basically prevents us from localizing any error messages that originate from config parsing. Basically because we have to parse the config file to see if the language has been overridden before we can initialize the localization engine.

Also not sure if the original LANG environment should be saved/restored. If we don't then any programs run within Ghostty also change languages.

@pluiedev pluiedev left a comment •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think it's worth to have this setting now as we don't really have parsing errors localized yet. For the long run we should probably use some sort of internal error enum that represents all failure cases instead of pushing raw strings into the config diagnostics

We should definitely preserve LANG for the child environment, since language is only supposed to override the interface language independent from the apps within.

@pluiedev
pluiedev merged commit 61e347a into ghostty-org:main Feb 14, 2026
54 checks passed
@github-actions github-actions Bot added this to the 1.3.0 milestone Feb 14, 2026
@jcollie
jcollie deleted the language-override branch February 14, 2026 16:31
@Filip7 Filip7 mentioned this pull request Feb 26, 2026
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.

GTK: Add the locale as a config setting

2 participants