fix(ui): draw controls on the window ground, not on the panel tint (src/styles.css, docs) - #76
Merged
Conversation
操作の背景を面の色味(`--surface`)から窓の地(`--bg`)へ揃えた。対象は タイトルバーの入力欄・寸法ピッカー・`端末` ボタン、入力欄の `宛先` と `送信` とテキストエリア、参加者の行の三つの記号ボタン、端末ヘッダの寸法 ピッカーと `✕`、アカウントのモーダルの入力欄と選択肢とボタン、`終了` の 確認ダイアログのボタン、そして選択中の端末タブである。 `--surface` は面そのものの地に残した——参加者パネルの地と、地がすでに `--bg` である場所で浮かせる要素(端末タブの `✕` のホバー)である。パネル 自身の地は変えていない。行の三つのボタンは `background: none` でパネルの 地が透けていたため、明示的に `var(--bg)` を持たせた。 選択中の端末タブは、これで地色によって選択を言えなくなる。代わりに言うのは 枠(未選択の `transparent` から `--line` へ)と文字色(`--muted` から参加者 自身の色へ)であり、参加者パネルの選択中の行と同じ形になる。それまでは パネルの行が `--bg` で浮き、端末のタブが `--surface` で沈むという、同じ 選択を二つの面が逆向きに描く状態だった。 色は直値で書かず、すべて `var(--bg)` / `var(--line)` である。ダークテーマ では `--bg` は `#17171a` であり、`#ffffff` と書けばそこで壊れる。 docs/0-requirements.md には「二つの地色」の節を足し、面と操作でどちらの地色 を使うかを記述した。実機での見た目は未確認であるため、その旨を「未実装」へ 一行足してある。 #75
smileygames
commented
Aug 24, 2026
smileygames
left a comment
Member
Author
There was a problem hiding this comment.
自己レビュー結果: PASS
#75 の受け入れ基準に対して差分を確認した。
受け入れ基準
- 操作が
--bgに移った —#titlebar/#composer/#terminal-headの input・select・textarea・button、#account-dialogの input・select・actions button、#end-dialogの actions button、参加者パネルの行の三つのボタン、選択中のタブ。 - 行のボタンは
noneから明示のvar(--bg)へ —noneのままではパネルの地が透け、アプリ内で唯一そこだけ別の地に乗る操作になっていた。 - 色を直値で書いていない — 差分に
#ffffffの追加は無い。ファイル内の#ffffffは:rootのトークン宣言のみで、これは元からある。ダークテーマは--bgの再定義でそのまま追随する。 #participantsの地は変えていない(#75 の制約)— 残ったbackground: var(--surface)は二箇所だけで、#participants自身の地と、--bgの地に乗る.tab .close:hoverの浮き。どちらも意図どおり。- 選択中のタブの印 — 地色が同じになった分は、未選択が
transparentを持つ枠が--lineに変わることと、名前が--mutedから参加者の色へ変わることが持つ。参加者パネルの選択行と同じ形になった。
issue 本文からの差分(受容)
#end-dialog .actions button は #75 の前提の列挙に無いが、実装者の判断で含めている。#end-dialog は #account-dialog の枠の値を「同じ種類の面」として借りており、片方だけ残せば同種の面のボタンに二色が乗る。決まったこと(操作は --bg)はボタン一般を覆っており、対象ファイルも否定制約も超えていない。妥当と判断し、受容する。
コメントの手当て
変更前の状態から論を立てていたコメント三箇所(選択中のタブ、#terminal-head の操作、行の三つのボタン)が書き直されている。二つの地色の使い分けは :root のトークン宣言の隣に置かれた。地色の判断の在り処としては妥当な場所。
実機未確認
見え方は実機で未確認。docs の未実装節に記録済み。
次の一手
実行モード auto。この自己レビュー通過をもって squash merge する。リリース種別は patch 相当(観測可能だが、トークンの割り当てを揃えた増分の変更であり、構造変更ではない)。
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 #75
何をしたか
操作の背景を面の色味(
--surface)から窓の地(--bg)へ揃えた。Master の実機指示(2026-08-24、「すべて背景白に」「選択中のタブは白表示に」)に対応する。src/styles.css:#titlebar input / select / button--surface--bg#composer select / textarea / button--surface--bg#terminal-head select / button--surface--bg#terminal-tabs .tab.shown--surface--bg#participants .member .edit / .start / .endnone(パネルの地が透ける)--bg#account-dialog input / select--surface--bg#account-dialog .actions button--surface--bg#end-dialog .actions button--surface--bg--surfaceを残した箇所は二つだけである。#participantsの地(issue の制約どおり変えない)と、#terminal-tabs .tab .close:hover(地がすでに--bgである場所で浮かせる要素)。:rootのトークン宣言の隣に、二つの地色の使い分けを書いた。既存のコメントのうち、揃える前の状態を根拠にしていた三箇所(選択中のタブ、端末ヘッダの操作、行の三つのボタン)を書き直した。docs/0-requirements.md:### 二つの地色を追加。面と操作でどちらの地色を使うか、なぜ塗りではなく枠で分けるか、直値を書かない理由を記述した。選択が見えることについて
タブの地が
--bgになると、地色では選択を言えなくなる。代わりに言うのは枠(未選択のタブはborder: 1px solid transparentを持つので、border-color: var(--line)が入るかどうか)と文字色(--mutedから参加者自身の色へ)である。参加者パネルの選択中の行と同じ形になり、それまで二つの面が逆向きに描いていた同じ選択が、一つの手になる。issue 本文からの差分
issue の「前提」の列挙に
#end-dialog .actions buttonが入っていないが、これも--surfaceを持つボタンである。#end-dialogは#account-dialogの枠を借りている面(styles.css のコメント)であり、片方だけ残すと同じ種類の面のボタンが二色になる。「決まったこと」の規則(操作 = input / select / textarea / button は--bg)がボタンを一般に含むため、揃えた。target files も制約も超えていない。検証
npm run build(tsc && vite build)通過。var(--surface)の残存 2 箇所を grep で確認(上記のとおり意図したもの)。#ffffffの直値は:rootのトークン宣言のみ。