Repository navigation
Conversation
Theme the UIs injected into web pages: the page action recorder and runner, the step detail popover, the input menu, toasts, and dialogs. - Add `dark:` counterparts next to the existing neutral color classes so the light theme renders unchanged. - Use a denser panel background (gray-900/80) for the recorder and runner; a translucent gray-800 turns muddy over light pages. - Pass the resolved theme to the sonner Toaster. - Give DialogContent `dark:text-foreground`, since in the content script the dialog text otherwise inherits the page's (usually dark) color. - Leave the focus outline and the link-drag progress ring as they are: they are drawn directly on the page, whose background is unknown. Refs #495 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_019DvJ25jdbq4pvn4K8Wk7Mx
The result popup and the API success icon referenced `--gray-*` and `--blue-500`, whose definitions were dropped from App.css in 292e1b3. The declarations were invalid at computed-value time, so the text inherited the host page's color (unreadable in the dark theme) and the copy button lost its hover background. Point them at Tailwind v4's `--color-*` variables and add dark variants. Refs #495 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_019DvJ25jdbq4pvn4K8Wk7Mx
This branch has not been deployed
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.
概要
#495 のダークモード対応の第4段階です。Webページ上に表示する UI(content script)をダークモードに対応しました。
変更内容
dark:付きのクラスを追加しています。既存のクラスは消していないので、ライト時の見た目は変わりません。グレー系の対応表は feat(extension): オプション画面のダークモード対応 #512 と同じです。dark:bg-gray-900/80にしました。半透明のgray-800/60だと、白いページの上で濁った中間色になり、文字が読みにくかったためです。Toasterに解決済みのテーマを渡しました。レビュー依頼トーストは、直書きの色にdark:を追加しています。DialogContentにdark:text-foregroundを追加しました。#e5e7eb)を継承していたので、トークンの色(hsl(213 31% 91%))に変わっても違いはごくわずかです。実行結果ポップアップの色の修正(既存の不具合)
ResultPopup.module.cssとMenu.module.css(.apiIconSuccess)は、--gray-*と--blue-500を参照しています。ところが、これらの変数の定義は 292e1b3 でApp.cssから削除されていました。そのため、宣言が無効になっていて、次の問題が起きていました。Tailwind v4 の
--color-*を参照するように直し、ダーク用の色も追加しました。本来のデザインに戻るので、ライト時の見た目もこの部分だけは変わります。:host()セレクタの詳細度がsvg.buttonSuccessを上回るため、成功時の色はダーク用にも書き直しています。オプション画面への波及
InputMenu(入力メニュー)は、オプション画面の AI プロンプト編集とユーザー変数の欄でも使われています。今回のdark:クラスで、オプション画面のダーク時に文字が暗いまま残っていた問題も解消されます。確認
tsc -b、yarn lint(エラー0件)、vitest(121ファイル/1586件すべて成功)、yarn buildanimate-spin-slow)の部分以外は同一でした。これで、#495 で挙げられた画面(ポップアップ、Open in Tab、オプション画面、オンボーディング、その他 Webページ上の UI)はすべて対応済みです。
Refs #495
🤖 Generated with Claude Code
https://claude.ai/code/session_019DvJ25jdbq4pvn4K8Wk7Mx