[PR #6812] [MERGED] fix(i18n): respect browser preferred languages when supported #13808

Closed
opened 2026-04-10 22:03:33 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/6812
Author: @elevennprime
Created: 1/29/2026
Status: Merged
Merged: 2/4/2026
Merged by: @matt-fidd

Base: masterHead: fix-ignoring-browser-languages-when-set


📝 Commits (4)

  • c7b5543 fix(i18n): respect browser preferred languages when supported
  • a1a7bfe chore: add the release notes related file
  • 13a8ee5 fix(i18n): check region locales support before falling back to base locale
  • 98944e6 fix(i18n): make the unit test aligned with the region locales checking

📊 Changes

3 files changed (+49 additions, -32 deletions)

View changed files

📝 packages/desktop-client/src/i18n.test.ts (+1 -1)
📝 packages/desktop-client/src/i18n.ts (+42 -31)
upcoming-release-notes/6812.md (+6 -0)

📄 Description

Instead of relying on the first browser language, the application now iterates through navigator.languages and selects the first supported locale, with a fallback to English. (Fixes #4229)


Bundle Stats

Bundle Files count Total bundle size % Changed
desktop-client 28 14.47 MB → 14.47 MB (+413 B) +0.00%
loot-core 1 5.84 MB 0%
api 1 4.38 MB 0%
View detailed bundle stats

desktop-client

Total

Files count Total bundle size % Changed
28 14.47 MB → 14.47 MB (+413 B) +0.00%
Changeset
File Δ Size
src/i18n.ts 📈 +312 B (+19.08%) 1.6 kB → 1.9 kB
locale/es.json 📈 +101 B (+0.06%) 171.21 kB → 171.31 kB
View detailed bundle breakdown

Added
No assets were added

Removed
No assets were removed

Bigger

Asset File Size % Changed
static/js/index.js 9.22 MB → 9.22 MB (+312 B) +0.00%
static/js/es.js 171.21 kB → 171.31 kB (+101 B) +0.06%

Smaller
No assets were smaller

Unchanged

Asset File Size % Changed
static/js/indexeddb-main-thread-worker-e59fee74.js 12.94 kB 0%
static/js/workbox-window.prod.es5.js 5.64 kB 0%
static/js/da.js 106.62 kB 0%
static/js/de.js 177.78 kB 0%
static/js/en-GB.js 7.18 kB 0%
static/js/en.js 162.91 kB 0%
static/js/fr.js 179.72 kB 0%
static/js/it.js 171.54 kB 0%
static/js/nb-NO.js 157.23 kB 0%
static/js/nl.js 106.65 kB 0%
static/js/pl.js 88.64 kB 0%
static/js/pt-BR.js 146.62 kB 0%
static/js/ru.js 106.97 kB 0%
static/js/sv.js 78.2 kB 0%
static/js/th.js 182.35 kB 0%
static/js/uk.js 215.11 kB 0%
static/js/resize-observer.js 18.37 kB 0%
static/js/BackgroundImage.js 120.54 kB 0%
static/js/ReportRouter.js 1.11 MB 0%
static/js/narrow.js 641.19 kB 0%
static/js/TransactionList.js 105.97 kB 0%
static/js/wide.js 160.07 kB 0%
static/js/AppliedFilters.js 9.71 kB 0%
static/js/usePayeeRuleCounts.js 11.79 kB 0%
static/js/useTransactionBatchActions.js 13.23 kB 0%
static/js/FormulaEditor.js 1.04 MB 0%

loot-core

Total

Files count Total bundle size % Changed
1 5.84 MB 0%
View detailed bundle breakdown

Added
No assets were added

Removed
No assets were removed

Bigger
No assets were bigger

Smaller
No assets were smaller

Unchanged

Asset File Size % Changed
kcab.worker.DE5uAdQe.js 5.84 MB 0%

api

Total

Files count Total bundle size % Changed
1 4.38 MB 0%
View detailed bundle breakdown

Added
No assets were added

Removed
No assets were removed

Bigger
No assets were bigger

Smaller
No assets were smaller

Unchanged

Asset File Size % Changed
bundle.api.js 4.38 MB 0%

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/actualbudget/actual/pull/6812 **Author:** [@elevennprime](https://github.com/elevennprime) **Created:** 1/29/2026 **Status:** ✅ Merged **Merged:** 2/4/2026 **Merged by:** [@matt-fidd](https://github.com/matt-fidd) **Base:** `master` ← **Head:** `fix-ignoring-browser-languages-when-set` --- ### 📝 Commits (4) - [`c7b5543`](https://github.com/actualbudget/actual/commit/c7b554356990aaffde22c5c3c48ac7e0ba3d0569) fix(i18n): respect browser preferred languages when supported - [`a1a7bfe`](https://github.com/actualbudget/actual/commit/a1a7bfe2942802f6d4c8ad30d6b0fe4f98e961ad) chore: add the release notes related file - [`13a8ee5`](https://github.com/actualbudget/actual/commit/13a8ee5e8077aad3d1b4c0e81c7087b56ae9e41e) fix(i18n): check region locales support before falling back to base locale - [`98944e6`](https://github.com/actualbudget/actual/commit/98944e6afec7560631a8fb3f45f09380c73265df) fix(i18n): make the unit test aligned with the region locales checking ### 📊 Changes **3 files changed** (+49 additions, -32 deletions) <details> <summary>View changed files</summary> 📝 `packages/desktop-client/src/i18n.test.ts` (+1 -1) 📝 `packages/desktop-client/src/i18n.ts` (+42 -31) ➕ `upcoming-release-notes/6812.md` (+6 -0) </details> ### 📄 Description Instead of relying on the first browser language, the application now iterates through navigator.languages and selects the first supported locale, with a fallback to English. (Fixes #4229) <!--- actual-bot-sections ---> <hr /> <!--- bundlestats-action-comment key:combined start ---> ### Bundle Stats Bundle | Files count | Total bundle size | % Changed ------ | ----------- | ----------------- | --------- desktop-client | 28 | 14.47 MB → 14.47 MB (+413 B) | +0.00% loot-core | 1 | 5.84 MB | 0% api | 1 | 4.38 MB | 0% <details> <summary>View detailed bundle stats</summary> #### desktop-client **Total** Files count | Total bundle size | % Changed ----------- | ----------------- | --------- 28 | 14.47 MB → 14.47 MB (+413 B) | +0.00% <details> <summary>Changeset</summary> File | Δ | Size ---- | - | ---- `src/i18n.ts` | 📈 +312 B (+19.08%) | 1.6 kB → 1.9 kB `locale/es.json` | 📈 +101 B (+0.06%) | 171.21 kB → 171.31 kB </details> <details> <summary>View detailed bundle breakdown</summary> <div> **Added** No assets were added **Removed** No assets were removed **Bigger** Asset | File Size | % Changed ----- | --------- | --------- static/js/index.js | 9.22 MB → 9.22 MB (+312 B) | +0.00% static/js/es.js | 171.21 kB → 171.31 kB (+101 B) | +0.06% **Smaller** No assets were smaller **Unchanged** Asset | File Size | % Changed ----- | --------- | --------- static/js/indexeddb-main-thread-worker-e59fee74.js | 12.94 kB | 0% static/js/workbox-window.prod.es5.js | 5.64 kB | 0% static/js/da.js | 106.62 kB | 0% static/js/de.js | 177.78 kB | 0% static/js/en-GB.js | 7.18 kB | 0% static/js/en.js | 162.91 kB | 0% static/js/fr.js | 179.72 kB | 0% static/js/it.js | 171.54 kB | 0% static/js/nb-NO.js | 157.23 kB | 0% static/js/nl.js | 106.65 kB | 0% static/js/pl.js | 88.64 kB | 0% static/js/pt-BR.js | 146.62 kB | 0% static/js/ru.js | 106.97 kB | 0% static/js/sv.js | 78.2 kB | 0% static/js/th.js | 182.35 kB | 0% static/js/uk.js | 215.11 kB | 0% static/js/resize-observer.js | 18.37 kB | 0% static/js/BackgroundImage.js | 120.54 kB | 0% static/js/ReportRouter.js | 1.11 MB | 0% static/js/narrow.js | 641.19 kB | 0% static/js/TransactionList.js | 105.97 kB | 0% static/js/wide.js | 160.07 kB | 0% static/js/AppliedFilters.js | 9.71 kB | 0% static/js/usePayeeRuleCounts.js | 11.79 kB | 0% static/js/useTransactionBatchActions.js | 13.23 kB | 0% static/js/FormulaEditor.js | 1.04 MB | 0% </div> </details> --- #### loot-core **Total** Files count | Total bundle size | % Changed ----------- | ----------------- | --------- 1 | 5.84 MB | 0% <details> <summary>View detailed bundle breakdown</summary> <div> **Added** No assets were added **Removed** No assets were removed **Bigger** No assets were bigger **Smaller** No assets were smaller **Unchanged** Asset | File Size | % Changed ----- | --------- | --------- kcab.worker.DE5uAdQe.js | 5.84 MB | 0% </div> </details> --- #### api **Total** Files count | Total bundle size | % Changed ----------- | ----------------- | --------- 1 | 4.38 MB | 0% <details> <summary>View detailed bundle breakdown</summary> <div> **Added** No assets were added **Removed** No assets were removed **Bigger** No assets were bigger **Smaller** No assets were smaller **Unchanged** Asset | File Size | % Changed ----- | --------- | --------- bundle.api.js | 4.38 MB | 0% </div> </details> </details> <!--- bundlestats-action-comment key:combined end ---> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-04-10 22:03:33 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/actual#13808