fix(theme): complete Color::Reset migration across all UI widgets (#601)#651
fix(theme): complete Color::Reset migration across all UI widgets (#601)#651merchloubna70-dot wants to merge 1 commit into
Conversation
|
Warning You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again! |
Triple-check finding: broken testThe production code was correctly changed to assert_eq!(theme.section_bg, palette::DEEPSEEK_INK);This will fail with |
`section_bg` was changed from `palette::DEEPSEEK_INK` to `Color::Reset` in the production code, but the test still asserted the old value, causing a runtime failure. Update assertion to match. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
Fixed — updated the test assertion at |
…g docstring Hmbown#651: fix test assertion — section_bg now Color::Reset (was DEEPSEEK_INK) Hmbown#645: replace expect() with Result in OpenSandboxBackend::new() Hmbown#653: correct resolve_prefixes docstring to describe deny-always-wins
…g docstring Hmbown#651: fix test assertion — section_bg now Color::Reset (was DEEPSEEK_INK) Hmbown#645: replace expect() with Result in OpenSandboxBackend::new() Hmbown#653: correct resolve_prefixes docstring to describe deny-always-wins
|
This PR was opened before the v0.8.41 rebrand and is now stale. Feel free to rebase onto current |
Follow-up to #600/#601, based on Gemini HIGH priority review.
The previous fix only changed one location. This PR replaces all hardcoded
Color::Blackbackgrounds withColor::Resetacross 10 UI files, making the TUI fully respect the user's terminal background color.Files changed: sidebar, session_picker, model_picker, provider_picker, command_palette, pager, plan_prompt, user_input, deepseek_theme, ui.
Implemented using
deepseek exec --model deepseek-v4-flash. 🐋