Skip to content

feat(ui): label the diagnostics toggle as the terminal it opens - #70

Merged
smileygames merged 2 commits into
mainfrom
issue-69-terminal-label
Aug 23, 2026
Merged

feat(ui): label the diagnostics toggle as the terminal it opens#70
smileygames merged 2 commits into
mainfrom
issue-69-terminal-label

Conversation

@smileygames

@smileygames smileygames commented Aug 23, 2026

Copy link
Copy Markdown
Member

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) も、開閉の対象も自動で開く条件も変えていない。

タイトルバーの `診断` ボタンの表示を `端末` に変えた。#57#62 で読む値が
参加者パネルへ移ったあと、このボタンが開く先は端末一枠だけになっている。
名前が面の中身の変化に追いついていなかった。

ボタンを名指ししている文も併せて直した。`src/main.ts` のセッション異常終了時の
状態メッセージと、`docs/0-requirements.md` の 2 箇所(文字サイズのピッカーの
位置、手順の節)である。表示だけを変えると、これらが画面上に無い名前を指す。

面の役割名としての `診断面` は仕様書にそのまま残している。要素の id
(`toggle-diagnostics`) も、開閉の対象も自動で開く条件も変えていない。

#69

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@smileygames smileygames linked an issue Aug 23, 2026 that may be closed by this pull request
@smileygames smileygames self-assigned this Aug 23, 2026
`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 smileygames left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

自己レビュー(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 モードにつき人手のゲート無し。マージへ進む。

@smileygames
smileygames merged commit 3ca1641 into main Aug 23, 2026
2 checks passed
@smileygames
smileygames deleted the issue-69-terminal-label branch August 23, 2026 23:45
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.

feat(ui): label the diagnostics toggle as the terminal it opens

1 participant