Skip to content

i18n: Update Norwegian translations - #13781

Merged
trag1c merged 1 commit into
ghostty-org:mainfrom
Uzaaft:uzaaft/i18n
Aug 14, 2026
Merged

i18n: Update Norwegian translations#13781
trag1c merged 1 commit into
ghostty-org:mainfrom
Uzaaft:uzaaft/i18n

Conversation

@Uzaaft

@Uzaaft Uzaaft commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

@jakeriksen Feel free to slaughter my translation 😂

@ghostty-bot ghostty-bot Bot added the translations Translations of the Ghostty app label Aug 13, 2026

@jakeriksen jakeriksen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

There are mainly 2 places where I would suggest edits due to potential mistakes (the places I tagged you in).

Otherwise I've suggested some shorter alternatives.

Also, why are there a lot of similar commands? For example:

#: src/input/command.zig:636
msgid "Close All Windows"
msgstr "Lukk alle vinduer"

#: src/input/command.zig:637
msgid "Close all windows."
msgstr "Lukk alle vinduer."

EDIT: If it is due to Gnome HIG, perhaps use capital letters in Norwegian As Well.?

Comment thread po/nb.po Outdated
Comment thread po/nb.po Outdated
Comment thread po/nb.po Outdated
Comment thread po/nb.po Outdated
Comment thread po/nb.po
Comment thread po/nb.po Outdated
Comment thread po/nb.po Outdated
Comment thread po/nb.po Outdated
Comment thread po/nb.po Outdated
Comment thread po/nb.po Outdated
Comment thread po/nb.po Outdated
Comment thread po/nb.po Outdated
@trag1c

trag1c commented Aug 13, 2026

Copy link
Copy Markdown
Member

EDIT: If it is due to Gnome HIG, perhaps use capital letters in Norwegian As Well.?

@jakeriksen FYI this exclusively applies to English, all the other languages do sentence case rather than title case.

@jakeriksen

Copy link
Copy Markdown
Contributor

EDIT: If it is due to Gnome HIG, perhaps use capital letters in Norwegian As Well.?

@jakeriksen FYI this exclusively applies to English, all the other languages do sentence case rather than title case.

Ok, nice to know, thanks @trag1c

@00-kat

00-kat commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Also, why are there a lot of similar commands? For example:

There's a couple big reasons; that one's a title vs a description:

.close_all_windows => comptime &.{.{
.action = .close_all_windows,
.title = i18n.N_("Close All Windows"),
.description = i18n.N_("Close all windows."),
}},

The line and file the string is found in is always present above it, and even if you don't know how to code or anything about GTK you can usually guess where it goes. Or you could look for it in the UI (but finding it may be a bit hard: that one isn't implemented on GTK yet, so it's not actually in the UI, for example).

Here's another example: https://github.com/ghostty-org/ghostty/blob/47c71d062a7611791e305ae178760d754d84acfe/po/nb.po#L835-L837 https://github.com/ghostty-org/ghostty/blob/47c71d062a7611791e305ae178760d754d84acfe/po/nb.po#L369-L371

The first's in the command palette: https://github.com/ghostty-org/ghostty/blob/47c71d062a7611791e305ae178760d754d84acfe/src/input/command.zig#L456-L460
Screenshot.

While the second one's in the popup that appears when you press the one in the command palette: https://github.com/ghostty-org/ghostty/blob/47c71d062a7611791e305ae178760d754d84acfe/src/apprt/gtk/class/title_dialog.zig#L225-L231
Screenshot.

EDIT: If it is due to Gnome HIG, perhaps use capital letters in Norwegian As Well.?

As for the capital letters, Does Norwegian Use Title Case in Titles? If it does, then yes; don't do it simply to copy the English source.

@jakeriksen

jakeriksen commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Got it, thanks @00-kat. Didn't trace it to the usage, just focused on the language.

As for the capital letters, Does Norwegian Use Title Case in Titles? If it does, then yes; don't do it simply to copy the English source.

Norwegian doesn't use Title Case, only sentence case, as @trag1c confirmed.

@Uzaaft should have enough feedback now for finishing this up.

@00-kat

00-kat commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Also, @jakeriksen: would you like to join the nb_NO localization team to help out with and be notified of future updates or requests thereof to the Norwegian translations?

@jakeriksen

Copy link
Copy Markdown
Contributor

Yeah, sure, I can help out there @00-kat

@jakeriksen jakeriksen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM @Uzaaft Enough slaughtering now 🗡️

@Uzaaft
Uzaaft force-pushed the uzaaft/i18n branch 2 times, most recently from eeaff46 to d474a52 Compare August 14, 2026 13:01
Co-authored-by: Aleksander Eriksen <jakeriksen@gmail.com>
@Uzaaft
Uzaaft marked this pull request as ready for review August 14, 2026 13:19

@jakeriksen jakeriksen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No more slaughtering from me

@jakeriksen

jakeriksen commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

@Uzaaft I don't have write access, so someone with write-access need to approve. @00-kat or @trag1c?

@trag1c
trag1c enabled auto-merge August 14, 2026 13:27
@trag1c
trag1c merged commit 6584450 into ghostty-org:main Aug 14, 2026
100 checks passed
@github-actions github-actions Bot added this to the 1.4.0 milestone Aug 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

translations Translations of the Ghostty app

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants