Feature/style adjustments#102
Conversation
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Mirrors the CSS custom properties in colors.css — adds missing brand colors (ALLEN_MAROON, ALLEN_ROSE, ALLEN_ORANGE, ALLEN_OCHRE, ALLEN_YELLOW) and semantic aliases (COLOR_BG_BASE, COLOR_TEXT_PRIMARY, LINK_COLOR, etc.) so components can import named tokens instead of raw hex values. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Fixes "yarn: command not found" when committing via VS Code's git UI, which doesn't inherit the shell PATH. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…iles Adds --color-accent, --color-surface, --color-bg-dark, --color-feature, --color-secondary to colors.css, then updates all module CSS files to use semantic names instead of palette names (--ALLEN_BLUE, --PAGE_1, --GRAY_2, --WHITE, --BLACK, etc.). Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…board into fix/replace-list
…ix/replace-list
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…eature/style-adjustments
✅ Deploy Preview for project-idea-board ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
Pull request overview
Adjusts the “proposal” callout styling to reduce visual harshness by routing it through design tokens and switching the callout border to Allen orange.
Changes:
- Update proposal callout styles to use
--callout-box-*CSS tokens rather than--accent-color/--bg-dark-color. - Change
--callout-box-border-colorto--ALLEN_ORANGE. - Add a design spec documenting the callout color decision.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| src/style/theme.ts | Updates Ant Design theme token usage for links (and contains a TS token mirror that should remain consistent with CSS tokens). |
| src/style/idea-post.module.css | Wires .proposal to callout-box design tokens for background and left border. |
| src/style/colors.css | Redirects the callout border token to --ALLEN_ORANGE. |
| docs/design/specs/2026-05-29-proposal-callout-color-design.md | Documents rationale and exact token-based styling for the callout update. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
ShrimpCryptid
left a comment
ShrimpCryptid
left a comment
There was a problem hiding this comment.
Had a question about a file that was checked in. There are a few Copilot comments that seem relevant also?
| @@ -0,0 +1,63 @@ | |||
| # Proposal Callout Box — Color Update Design | |||
There was a problem hiding this comment.
Should this file be persisted in the repo?
There was a problem hiding this comment.
you know, I'm unsure. On the one hand they are really just for the implementation so could be deleted, but on the other hand I can see having a history of rational and design might be helpful?
There was a problem hiding this comment.
I worry a bit about having a folder that's constantly expanding with each new feature that's merged. I wish it could be captured in an issue or something outside of the repo itself.
There was a problem hiding this comment.
why? they're fairly small overall. I guess with a release I could have the docs combined, but I don't see what problem it would cause.
Problem
The green callout box was a little too bright
Solution
Changed it to an ALLEN_ORANGE as the border
NOTE: the number of commits is because I squashed merged but this branch was from main before that.
Type of change
Please delete options that are not relevant.
style change
Steps to Verify:
Screenshots (optional):
BEFORE:
AFTER: