Skip to content

fix: フォーカス・入力エラー・字幕更新のアクセシビリティを改善 - #37

Merged
Eotel merged 1 commit into
mainfrom
codex/accessibility-audit
Oct 26, 2026
Merged

Eotel merged 1 commit into
mainfrom
codex/accessibility-audit

Conversation

@Eotel

@Eotel Eotel commented Oct 26, 2026

Copy link
Copy Markdown
Contributor

Summary

入力エラーのある欄、取消・削除後の移動先、観客字幕の更新を支援技術から把握できるようにします。impeccableと実ブラウザで確認した7種類の問題へ対応しました。

入力・取消・辞書の追加/削除
  Mediator: エラー対象とフォーカス先を指定
    Presenter: 描画へ変換
      View: 欄の状態と説明を関連付け、フォーカスを移動

公開字幕 → 名前付きlog(追加・訂正を通知)
設定プレビュー → 通知なし

入力境界は白背景とのコントラストを約2.52:1から約4.74:1へ改善。右ペインはドラッグ・キーに加え、ボタンでも24pxずつ変更できます。ホームへ戻ると旧HTMLが再利用される現象も確認したため、3つのHTMLルートを再検証する設定にしました。

Evidence

  • Before: ルーム名・AI・辞書にエラーがあっても対象欄の aria-invalid / aria-describedby がない。取消・辞書削除後のフォーカスはBODYへ移り、幅はクリックだけでは変更できませんでした。回帰テストで再現しています。
    After: 該当欄だけをエラーとして示し、辞書はその欄へ移動。取消は起点へ、削除は隣の単語か追加ボタンへ戻ります。フォーム切替時の古いエラー指定も解除します。
  • 実FastAPIと背景の実ブラウザで、辞書の表記・読み・誤認識例のエラー、保存、ダイアログのEscape、384→408→384pxのクリック操作と下書き保持を確認。390×844pxでも設定へTabで到達し、横はみ出しはありません。
  • 実APIからの公開字幕の追加・訂正が、観客のlogへ16→17件で届くことを確認。プレビューのlogは0件。ブラウザのエラーログはありません。
  • bash scripts/check.sh 成功。Node 22.23.4、frontend 150件・backend 196件、型検査0エラー。frontendには別作業の未コミットテスト1件を含み、このPRには入れていません。
  • impeccableのソース・ホーム・操作画面の検出は指摘なし、終了コード0。別セッションの監査にP0・P1の残存指摘はありません。

確認範囲・根拠・限界 / 検証記録

読みのエラーにフォーカスを移動

クリックで幅を変えられる訂正欄

Merge Danger

Door: two-way

データ形式や保存形式の変更はありません。このコミットをrevertできます。

Blast Radius: accessibility

ホーム、設定、字幕表示とHTMLの再検証が対象です。実スクリーンリーダーの発話、音声操作、物理端末のタッチ、強制配色、200%拡大は未検証です。観客URLの検出器はタイムアウトしたため成功扱いにせず、実ブラウザで補っています。WCAG全体への適合を保証する評価ではありません。音声・外部AIは使っていません。

投稿セッション:合成音声でGemini・GPT動作確認

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 26, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-26T03:27:37.327981Z bb4b40a Manual request
🔒 Security Review ✅ Completed 2026-10-26T03:29:45.807334Z bb4b40a Manual request
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@Eotel Eotel self-assigned this Oct 26, 2026
@Eotel
Eotel merged commit bb4b40a into main Oct 26, 2026
2 checks passed

Eotel commented Oct 26, 2026

Copy link
Copy Markdown
Contributor Author

@codex security review

PR #37 のセキュリティレビューが running のままです。対象コミット 8e0e70c0414054623dd8616a49a81b42084a1dd2 の結果を確認するため、再レビューをお願いします。

投稿セッション:合成音声でGemini・GPT動作確認

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Hooray!

Reviewed commit: bb4b40a359

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@chatgpt-codex-connector

Copy link
Copy Markdown

🛡️ Codex Security Review · Automatically triggered

Security review completed. No security issues were found in this pull request.

Reviewed commit: bb4b40a359

View security finding report

Only the user who started this review can view the report in Codex.

ℹ️ About Codex security reviews in GitHub

This is an experimental Codex feature. Security reviews are triggered when:

  • You comment "@codex security review"
  • A regular code review gets triggered (for example, "@codex review" or when a PR is opened), and you’re opted in so security review runs alongside code review

Once complete, Codex will leave suggestions, or a comment if no findings are found.

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