Skip to content

fix: localize network settings timeout options#677

Merged
everpcpc merged 1 commit intomainfrom
codex/fix-network-settings-localization
Mar 19, 2026
Merged

fix: localize network settings timeout options#677
everpcpc merged 1 commit intomainfrom
codex/fix-network-settings-localization

Conversation

@everpcpc
Copy link
Copy Markdown
Owner

Problem

The Network section in Settings showed untranslated timeout option text because SettingsNetworkView built localization keys dynamically, so Xcode string extraction did not keep the timeout entries in Localizable.xcstrings in sync.

Approach

Replace the dynamic string-key plumbing with static LocalizedStringResource arguments in timeoutRow, so the timeout labels and descriptions are visible to the localization pipeline. Then add the missing request, download, and authentication timeout translations and resync the string catalog after a full build.

Scope

  • Make the Network timeout rows use static localized resources in SettingsNetworkView
  • Add translations for the missing Network timeout keys in Localizable.xcstrings
  • Keep the current stale catalog cleanup that removes the unused Always Show Page Number entry

Validation

  • make build
  • make localize
  • ./misc/translate.py list

Replace dynamically constructed network settings localization keys with static LocalizedStringResource values so Xcode string extraction can discover them.

Add translations for the request, download, and authentication timeout rows in Localizable.xcstrings and sync the catalog after a full build.

This also carries forward the current stale-string cleanup for the removed "Always Show Page Number" entry.
@everpcpc everpcpc merged commit 6a1d575 into main Mar 19, 2026
3 checks passed
@everpcpc everpcpc deleted the codex/fix-network-settings-localization branch March 19, 2026 14:14
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