Complete Arabic localization with UI strings and RTL configuration#9292
Complete Arabic localization with UI strings and RTL configuration#9292aslamalkarywk7 wants to merge 2 commits into
Conversation
|
Hi @joshistoast and @lstein, I noticed that the CI build ( I am currently fixing this path mapping issue and will push the update shortly to get the workflow checks passing. Sorry for the oversight! |
joshistoast
left a comment
There was a problem hiding this comment.
-
High: Arabic pages link back to English docs via root-relative URLs.**
docs/src/content/docs/ar/index.mdx:81,docs/src/content/docs/ar/features/gallery.mdx:135,docs/src/content/docs/ar/contributing/index.md:13
I found 38 root-relative internal links like/start-here/installation/,/concepts/prompting-guide/, and/development/. From an Arabic page these navigate to the default English locale instead of/ar/..., so users fall out of Arabic navigation. Use relative links or prefix internal absolute links with/ar/. -
Medium: Several relative links resolve to non-existent nested routes.
Examples:
docs/src/content/docs/ar/features/Canvas/shapes-tool.mdx:28uses./lasso-tool/, which resolves from/ar/features/canvas/shapes-tool/to/ar/features/canvas/shapes-tool/lasso-tool/.
docs/src/content/docs/ar/features/Multi-User Mode/user-guide.mdx:321uses./api-guide/, resolving to/ar/features/multi-user-mode/user-guide/api-guide/.
docs/src/content/docs/ar/features/Workflows/editor-interface.mdx:16uses../../concepts/diffusion, resolving to/ar/features/concepts/diffusion.
These should use sibling/ancestor-correct paths such as../lasso-tool/,../api-guide/, and../../../concepts/diffusion. -
Medium: Sidebar group labels remain English in Arabic locale.
docs/src/config/sidebar.ts:8,docs/src/config/sidebar.ts:16,docs/src/config/sidebar.ts:24
The PR addsarto Starlight locales, but the sidebar section labels are hard-coded as English:Start Here,Configuration,Concepts, etc. Arabic page titles are translated, but the main navigation categories will still be English. -
Low/Medium: Arabic assets duplicate existing assets byte-for-byte.
docs/src/content/docs/ar/assets/*
The PR adds about 15 MB of duplicate images underar/assets; every checked image matched an existing English asset by SHA-256. Unless localized screenshots are planned, this bloats the repo and build inputs unnecessarily. The Arabic docs can reference the existing asset locations instead.
Summary
This PR introduces full Arabic localization for the InvokeAI documentation suite. It features the translation of the core documentation pages, UI localization strings, and configures the layout for Right-to-Left (RTL) reading orientation.
docs) / i18ndocs/src/content/docs/ar/.src/content/i18n/ar.json.dir: 'rtl'insideastro.config.mjs.Related Issues / Discussions
QA Instructions
I have thoroughly tested the documentation build locally to ensure a seamless layout:
pnpm dev./ar/) documentation path.Merge Plan
This is a large PR containing documentation files and assets only. It does not touch any core application logic, DB schemas, or code repositories. It can be safely merged once the documentation CI checks pass.
Checklist
What's Newcopy (if doing a release after this PR)