Skip to content

docs: localize README context diagram alt text#2311

Merged
senamakel merged 2 commits into
tinyhumansai:mainfrom
sjh9714:docs-localize-readme-alt-text
May 21, 2026
Merged

docs: localize README context diagram alt text#2311
senamakel merged 2 commits into
tinyhumansai:mainfrom
sjh9714:docs-localize-readme-alt-text

Conversation

@sjh9714
Copy link
Copy Markdown
Contributor

@sjh9714 sjh9714 commented May 20, 2026

Summary

  • Localized the context diagram alt text in the Japanese README.
  • Localized the context diagram alt text in the Korean README.
  • Kept the English README unchanged because its current alt text is already appropriate.

Problem

#2162 requested localized equivalents for README context-diagram alt text. The original missing bash fences and basic image alt attributes are already present on main, but the Japanese and Korean READMEs still used the English OpenHuman context diagram alt text.

Solution

  • Replaced the remaining English context-diagram alt strings in README.ja-JP.md and README.ko.md with localized text.
  • Kept the change docs-only and limited to the two localized README files.

Submission Checklist

If a section does not apply to this change, mark the item as N/A with a one-line reason. Do not delete items.

  • N/A: Tests added or updated — docs-only image alt text change, no executable behavior.
  • N/A: Diff coverage ≥ 80% — docs-only README change, no coverage-producing code paths.
  • N/A: Coverage matrix updated — no feature row added, removed, or renamed.
  • N/A: All affected feature IDs listed — no affected feature IDs for README alt text.
  • No new external network dependencies introduced.
  • N/A: Manual smoke checklist updated — not a release-cut surface.
  • N/A: Linked issue closed via closing keyword — this is a small follow-up for the localized alt text portion only.

Impact

Docs/accessibility metadata only. No runtime, platform, performance, security, migration, or compatibility impact.

Related


AI Authored PR Metadata (required for Codex/Linear PRs)

Keep this section for AI-authored PRs. For human-only PRs, mark each field N/A.

Linear Issue

  • Key: N/A
  • URL: N/A

Commit & Branch

  • Branch: docs-localize-readme-alt-text
  • Commit SHA: bd58f7a

Validation Run

  • N/A: pnpm --filter openhuman-app format:check blocked at Rust formatting step; see Validation Blocked.
  • N/A: pnpm typecheck — docs-only README change.
  • Focused tests: PATH="/Users/sungjh/.cache/codex-runtimes/codex-primary-runtime/dependencies/node/bin:$PATH" pnpm --filter openhuman-app exec prettier --check .
  • Focused tests: git diff --check
  • N/A: Rust fmt/check — no Rust files changed.
  • N/A: Tauri fmt/check — no Tauri files changed.

Validation Blocked

  • command: pnpm format:check
  • error: app Prettier reported All matched files use Prettier code style!, then pnpm rust:format:check failed with sh: cargo: command not found because this local environment does not have Rust/Cargo on PATH.
  • impact: Rust/Tauri formatting was not run locally; this PR changes only README markdown image alt text.

Behavior Changes

  • Intended behavior change: Japanese and Korean README context-diagram alt text now matches each README locale.
  • User-visible effect: screen readers and markdown consumers get localized image descriptions in those README files.

Parity Contract

  • Legacy behavior preserved: image source, layout, and English README text are unchanged.
  • Guard/fallback/dispatch parity checks: N/A for docs-only change.

Duplicate / Superseded PR Handling

@sjh9714 sjh9714 requested a review from a team May 20, 2026 08:26
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 20, 2026

📝 Walkthrough

Walkthrough

This PR updates the alt text for a diagram image in locale-specific README files, changing the English caption "OpenHuman context diagram" to Japanese ("OpenHuman コンテキスト構築図") in README.ja-JP.md and Korean ("OpenHuman 컨텍스트 구축 다이어그램") in README.ko.md.

Changes

README Localization

Layer / File(s) Summary
Localize diagram alt text
README.ja-JP.md, README.ko.md
The image alt text for the OpenHuman context diagram is updated from English to Japanese in README.ja-JP.md (line 92) and from English to Korean in README.ko.md (line 93).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related issues

Possibly related PRs

Suggested reviewers

  • graycyrus
  • senamakel

Poem

🐰 A diagram speaks in English—how mundane! ✨
Now コンテキスト and 컨텍스트 stake their claim.
Alt text blooms in native tongue with care,
Readers round the globe find meaning there.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The pull request title accurately describes the main change: localizing the context diagram alt text in the README files for Japanese and Korean versions.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

coderabbitai[bot]
coderabbitai Bot previously approved these changes May 20, 2026
@senamakel senamakel merged commit 2073a53 into tinyhumansai:main May 21, 2026
23 checks passed
@senamakel
Copy link
Copy Markdown
Member

huge thanks @sjh9714 for landing this 🙌 localizing the context diagram alt text in the japanese and korean readmes is exactly the kind of polish that makes the docs feel native to every reader. since this is your first merged pr here, we'd love to have you in our builder discord at https://discord.tinyhumans.ai/ for some exclusive community access ✨

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.

2 participants