feat(ui): label the diagnostics toggle as the terminal it opens - #70
Merged
Conversation
タイトルバーの `診断` ボタンの表示を `端末` に変えた。#57 と #62 で読む値が 参加者パネルへ移ったあと、このボタンが開く先は端末一枠だけになっている。 名前が面の中身の変化に追いついていなかった。 ボタンを名指ししている文も併せて直した。`src/main.ts` のセッション異常終了時の 状態メッセージと、`docs/0-requirements.md` の 2 箇所(文字サイズのピッカーの 位置、手順の節)である。表示だけを変えると、これらが画面上に無い名前を指す。 面の役割名としての `診断面` は仕様書にそのまま残している。要素の id (`toggle-diagnostics`) も、開閉の対象も自動で開く条件も変えていない。 #69 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
`README.md` の「「診断」を開くと」も `端末` へ揃えた。ボタンの表示が変わった以上、 この文は画面に無い名前を指していた。 「タイトルバーの」を補って、ボタンの「端末」と、そこに出る CLI の端末とを 書き分けている。`docs/0-requirements.md` の手順の節が既に同じ呼び方をしている。 当初の本文は名指し箇所を三つと数え、`README.md` を対象に含めていなかった。 `docs/` と `src/` にしか grep を掛けていなかったためである。実装中の掃きで 見つかり、Master の go-sign を受けて対象へ追加した。 `README.md:71` の「その端末が診断面に出ます」は面の役割名であるため据え置く。 #69 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
smileygames
commented
Aug 23, 2026
smileygames
left a comment
Member
Author
There was a problem hiding this comment.
自己レビュー(auto モード)
判定 = pass。4ファイル、テキストノードのみの変更。
完了条件の照合
| 完了条件 | 結果 | 確認 |
|---|---|---|
タイトルバーのボタンが 端末 と表示される |
✅ | index.html:39 |
| ボタンを名指ししている文が、画面に無い名前を指していない | ✅ | 下の掃きを参照 |
面の役割名としての 診断面 が仕様書に残っている |
✅ | docs/0-requirements.md に 15 箇所、### 診断面 の節も健在 |
| 開閉の挙動が変わっていない | ✅ | toggle-diagnostics の id・ハンドラ・自動オープン条件いずれも無変更 |
名指し箇所の掃き(ブランチ上で実測)
git grep 診断 を tracked file に掛け、診断面(役割名)を除外した結果:
docs/0-requirements.md— 0 件。ボタンを指す記述は残っていない。index.html/src/main.ts/src/styles.css— 0 件。README.md:71「その端末が診断面に出ます」— 役割名につき据え置き。issue 本文の判断どおり。design/Main.dc.html:28/:171— 設計スナップショット。issue 本文で対象外と記録済み。
四箇所すべてが変更されている。issue が当初「三つ」としていたのは親の grep 漏れであり、本文と対象ファイルは Master の go-sign を得て修正済み(本文の経緯節に記録)。
判断の入った一点
README.md:83 は語の入れ替えではなく タイトルバーの を足している。素の入れ替えでは「「端末」を開くと…端末が出ます」となり、引用したボタン名と出てくるものが同じ語で隣り合う。docs/0-requirements.md:544 が既に タイトルバーの「端末」 と書いているため、二つの文書が同じ呼び方に揃う。妥当と判断し、そのまま採る。
制約の照合
- 面の名前(
診断面)不変。 ✅ - 要素 id(
toggle-diagnostics)不変。 ✅ - 挙動不変。テキストノード以外に差分なし。 ✅
逸脱
無し。README.md の追加は実装中に検出したスコープ超過であり、コミット前に確認ゲートを立て、Master の go-sign(続行)を得た上で issue 本文の対象ファイルへ追加してから実装している。順序は仕様どおり。
CI = 2件とも pass。auto モードにつき人手のゲート無し。マージへ進む。
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.
Closes #69
タイトルバーの
診断ボタンの表示を端末に変えた。#57 と #62 で読む値が参加者パネルへ移ったあと、このボタンが開く先は#terminal一枠だけになっており、名前が面の中身に追いついていなかった。ボタンを名指ししている文も併せて直した。
src/main.tsのセッション異常終了時の状態メッセージ、docs/0-requirements.mdの文字サイズのピッカーの位置と手順の節の 2 箇所、README.mdの端末の節である。README.mdは当初 issue の対象ファイルに無かった。名指し箇所を三つと数えていたのが誤りで、実装中の grep 掃きで見つかったため、Master の go-sign を受けて対象へ追加した(本文の「経緯」節)。面の役割名としての
診断面は仕様書とREADME.md:71に残している。design/Main.dc.htmlは設計スナップショットのため対象外。要素の id (toggle-diagnostics) も、開閉の対象も自動で開く条件も変えていない。