docs(requirements): narrow the chat text size entry to what is still unverified - #67
Merged
Merged
Conversation
…unverified #60 の実機確認が一部済んだので、未実装(実機未確認)の該当行を確認できた 範囲まで縮めた。 確定したのは二点である。キー操作(`Ctrl` + `=` / `-` / `0`)で会話面の寸法 が動くこと、そしてその範囲が会話面に留まり、参加者パネル・診断面・端末・ 入力欄が動かないことである(実機観察、2026-08-24)。 後者が webview のズームの件を閉じる。`stepRoomFontSize` はキーを受けたら 無条件に走るため、寸法が動いたことだけでは `event.preventDefault()` が効いた かどうかは決まらない。決めるのは会話面以外が一緒に動いたかどうかであり、 動かなかった以上、webview 自身のズームは走っていない。よって「webview の ズームが `Ctrl` + `=` を先に取らないことも未計測である」を削除した。 ヘッダのピッカー、再起動後の保持、上下限は未確認のまま残している。実装済み の一覧の行と実装そのものには触れていない。 #65
smileygames
commented
Aug 23, 2026
smileygames
left a comment
Member
Author
There was a problem hiding this comment.
自己レビュー(auto モード)
判定 = pass。差分は docs/0-requirements.md の未実装リスト1行のみ。
完了条件の照合
| 完了条件 | 結果 |
|---|---|
| webview のズームに関する未計測の記述が無くなっている | ✅ 「未計測である」の文が消え、閉じた理由の記述に置き換わっている |
| キー操作と範囲の限定が実機で確認済みであることが読める | ✅ 「2026-08-24 の実機観察で確認済み」と日付付きで明記 |
| ピッカー・再起動後の保持・上下限が未確認として残っている | ✅ 行頭で三点を名指しし、未確認側に置いている |
制約の照合
- 対象は未実装(実機未確認)の該当行のみ。実装ファイルへの差分なし。 ✅
- 実装済みの一覧の行は無変更。 ✅
- 確認できていない三点を確認済みとして書いていない。 ✅
literal の照合
Master の実機観察は「キー操作も効いた」と「ほかのウィンドウは固定のままだね」の二つ。書かれているのもその二つに限られており、ピッカー・保持・上下限へは踏み込んでいない。過剰な確認済み化は無い。
webview のズームを閉じる論証も literal と一致する。src/main.ts で event.preventDefault() の直後に stepRoomFontSize(step) が無条件で走るため、寸法が動いたこと自体は既定動作を止めた証拠にならない。決めるのは会話面以外が動かなかったことであり、本文はそう書いている。
逸脱
無し。括弧内が「特に端末の桁数と行数」から四面(参加者パネル・診断面・端末・入力欄)の列挙へ変わっているが、これは #60 の仕様節および本 issue の前提の literal に合わせたもので、同節の数詞不一致は #60 の時点で四つ側へ収束済み。整合を取る変更であり範囲外ではない。
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 #65
docs/0-requirements.mdの未実装(実機未確認)の該当行を、2026-08-24 の実機観察で確認できた範囲まで縮めた。キー操作(Ctrl+=/-/0)と、効く範囲が会話面に留まること(参加者パネル・診断面・端末・入力欄が動かないこと)を確認済みとして書いている。後者が webview のズームの件を閉じるため、「webview のズームが
Ctrl+=を先に取らないことも未計測である」を削除した。ヘッダのピッカー・再起動後の保持・上下限は未確認のまま残している。変更は当該 1 行のみで、実装済みの一覧の行にも実装にも触れていない。