Align TUI colors with imogen brand palette - #3
Merged
Conversation
ACCENT was #E0A162, a hand-approximation of the web UI's "safelight" accent (#E39B5C). The tile/border color (#2A2D32) was likewise an approximation of the brand's line color (#262A2F). Both now match the canonical values from imogen-server exactly, and the border color is hoisted into a BORDER constant next to ACCENT/MUTED since it was duplicated across six call sites. Co-Authored-By: Claude with claude-sonnet-5
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
1 participant
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.
Summary
ACCENTinsrc/tui/ui.rswas#E0A162, a hand-approximation of the web UI's "safelight" accent color. Changed to the canonical#E39B5C.#2A2D32at six call sites insrc/tui/ui.rs, was likewise an approximation of the brand's line color#262A2F. Changed to the canonical value and hoisted into a newBORDERconstant next to the existingACCENT/MUTEDconstants.MUTED(#9096A0) already matched the brand palette and was left unchanged, as was the CLI-local destructive confirm-dialog color (#E07A5F), which is deliberate and not part of the brand palette.Color::Rgbvalues exist outsideui.rs.Test plan
cargo fmt --check— cleancargo build— passescargo clippy --all-targets -- -D warnings— cleancargo test— 53 passed, 1 ignored (a visual-only test not meant for CI), 0 failedhttps://claude.ai/code/session_01B9q2xWbtYxQU1KdQGCmBxt