i18n: RTL readiness + text-expansion robustness (R-48)#45
Merged
ChelseaKR merged 2 commits intoJul 8, 2026
Conversation
Make the PWA RTL-ready, locale-format-correct, and tolerant of text expansion ahead of new language bundles. - app/styles.css: convert remaining physical-direction properties to logical ones (skip-link `left` -> `inset-inline-start`, result list `padding-left` -> `padding-inline-start`); add wrap tolerance (white-space: normal / overflow-wrap / flex-wrap) to buttons, badges, and language labels so long localized strings wrap at 320px. - app/app.js: add an RTL_LANGS map and set `dir` (rtl/ltr) alongside `lang` in setLanguage. Date/number rendering already uses Intl keyed to the active language. - app/index.html: add explicit `dir="ltr"` default on <html>. - tests/test_app_i18n.py: guard no physical-direction CSS, that setLanguage sets `dir` beside `lang`, and a static pseudo-locale expansion check that compact-UI labels stay bounded. - docs: I18N.md G10 DEFERRED -> PARTIAL; localization-guide soften the RTL/format cautions (dir set, logical CSS, Intl formats) while keeping the native-speaker visual-QA requirement before shipping an RTL bundle. - docs/research/synthetic-personas-feedback.md: mark R-48 done. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Make the PWA RTL-ready, locale-format-correct, and tolerant of text
expansion ahead of new language bundles.
logical ones (skip-link
left->inset-inline-start, result listpadding-left->padding-inline-start); add wrap tolerance(white-space: normal / overflow-wrap / flex-wrap) to buttons, badges,
and language labels so long localized strings wrap at 320px.
dir(rtl/ltr) alongsidelangin setLanguage. Date/number rendering already uses Intl keyedto the active language.
dir="ltr"default on .setLanguage sets
dirbesidelang, and a static pseudo-localeexpansion check that compact-UI labels stay bounded.
RTL/format cautions (dir set, logical CSS, Intl formats) while keeping
the native-speaker visual-QA requirement before shipping an RTL bundle.
Co-Authored-By: Claude Fable 5 noreply@anthropic.com
Backfill PR for a completed roadmap item (
roadmap/rtl-readiness-and-text-expansion-robustn-12, 1 commit(s) overmain). Part of the portfolio roadmap batch.