Skip to content

fix(web): remove accent color picker from navbar - #101

Merged
mashkovd merged 1 commit into
mainfrom
fix/remove-accent-picker
May 21, 2026
Merged

mashkovd merged 1 commit into
mainfrom
fix/remove-accent-picker

Conversation

@mashkovd

Copy link
Copy Markdown
Member

Summary

  • Removes the 4-color accent swatch widget from the navbar (cyan/lime/vermilion/lilac squares)
  • Deletes CSS blocks: .accent-picker, .accent-swatch, [data-accent='...'] palette overrides, light-mode accent overrides
  • Deletes JS: setAccent() function and its event listener wiring
  • Cleans up pre-paint script: removes mctl-accent localStorage read and data-accent restore
  • Light mode --accent now uses the hardcoded #00e5ff literal (was var(--mctl-accent-cyan-primary))

The page always renders with the default cyan accent; dark/light toggle is unaffected.

Test plan

  • Open tg.mctl.ai: navbar shows nav links + moon/sun toggle only, no colored squares
  • Dark/light toggle still works
  • Light mode accent color (links, borders, code) renders correctly

🤖 Generated with Claude Code

The four-color swatch widget is no longer needed.
Removes HTML, CSS (.accent-picker / .accent-swatch), the setAccent() JS
function, its localStorage restore in the pre-paint script, and the
[data-accent] palette-override rules.
@claude

claude Bot commented May 21, 2026 •

Copy link
Copy Markdown

Claude finished @mashkovd's task in 1m 33s —— View job


Code Review

  • Gather context
  • Review diff
  • Post findings

0 P1 · 0 P2 · 2 P3

P3 — internal/web/landing.html line 60 (unchanged context, outside the diff): the block comment still says "accent swaps are handled by [data-theme] / [data-accent] layers automatically". The [data-accent] reference is now dead; the comment should read "handled by [data-theme] layers automatically."

P3 — internal/web/landing.html ~line 33: the removal of the three [data-accent] rules leaves an extra blank line immediately before </style>. Minor whitespace nit.

Both are cosmetic — neither blocks correctness, security, or dark/light toggle behaviour.

Verdict: No P1/P2 findings, good to merge.

@mashkovd

Copy link
Copy Markdown
Member Author

@claude review

1 similar comment
@mashkovd

Copy link
Copy Markdown
Member Author

@claude review

@mashkovd
mashkovd merged commit 1e13883 into main May 21, 2026
6 checks passed
@mashkovd
mashkovd deleted the fix/remove-accent-picker branch May 21, 2026 07:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants