-
Notifications
You must be signed in to change notification settings - Fork 0
Feature/style adjustments #102
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
95 commits
Select commit
Hold shift + click to select a range
a5bbb80
add new design guidance
meganrm 18958b9
update styles
meganrm 1aafdc4
update theme
meganrm 239193d
apply new styles
meganrm ac52005
update file
meganrm 1e88986
simplifying css
meganrm 832b524
add images from theme
meganrm 307c489
feat: add new Allen Institute logo asset
meganrm ebb6c97
feat: add logo and sub-label styles to header
meganrm fc2e5e3
feat: replace text logotype with new Allen Institute SVG logo
meganrm 9f05cbc
refactor: remove unused title prop from AppHeader
meganrm 57fb75d
feat: add hero section styles to index page
meganrm 44e458e
feat: add Allen Blue hero section to homepage
meganrm 98c16f6
fix: use section element for hero, improve breadcrumb contrast
meganrm e9a45bb
ignore new plugin
meganrm 675fa98
apply new style
meganrm 2cf9bc7
docs: add IdeaRoll ListItem redesign spec
meganrm 38db19c
docs: add IdeaRoll ListItem redesign implementation plan
meganrm 1a4cde0
feat: add accent color CSS rules to idea-roll list item
meganrm 37e69d9
feat: add deterministic accent colors to IdeaRoll list items
meganrm 27c72eb
fix: use item.id as React key on List.Item
meganrm a24e836
fix: match avatar color to item accent color
meganrm 7105050
fix: use uniform Allen Violet for card borders, titles, and avatars
meganrm c2ac107
feat: add full brand palette and semantic aliases to theme.ts
meganrm 4c3981e
fix: replace yarn with npm run in lefthook hooks
meganrm 6d5dade
refactor: replace direct color refs with semantic tokens in all CSS f…
meganrm 824c528
remove specs
meganrm 4e0ed53
ignore docs
meganrm f3a951a
revert changes to lefthook
meganrm 7572bf7
Merge branch 'main' into feature/update-brand
meganrm 49e7825
use height
meganrm 395be1c
styles not needed
meganrm 4a21547
remove unused class
meganrm 9981c72
feat: add type field to IdeaPost schema
meganrm 786910b
feat: add program and type fields to idea post query
meganrm 31bf05d
refactor: add optional className prop to TagPopover
meganrm 0de9585
refactor: change ResourceItem root element to li for dot-bullet lists
meganrm 759507b
chore: remove ResourceItem test (testing-library not yet installed)
meganrm 83bb1e0
style: replace idea-post CSS with new Allen 2025 brand design
meganrm bb13c63
feat: restructure idea post header — blue block, metadata strip, tag row
meganrm 6eaac80
refactor: update MaterialsAndMethods to use ul/li and new CSS class n…
meganrm 70666c5
style: cap content wrapper at 1200px max-width
meganrm a432ecd
style: remove orphaned 1800px media query from contentWrapperFullWidth
meganrm 76f1713
style: restyle page-nav sider to page background with border
meganrm e3db4a1
style: add framing borders to content column on wide viewports
meganrm 6442a6d
use semantic names
meganrm f6a0143
adjust styles
meganrm 023b7b7
adjust setting
meganrm 56ef57a
update claude md
meganrm 831fea2
add index page redesign spec
meganrm 11b6dcd
use semantic color names in index page redesign spec
meganrm 171944b
feat: add --content-padding-x layout tokens to colors.css
meganrm 5c84627
feat: make index page full-width and wrap hero text in constrained in…
meganrm 76882aa
feat: rewrite index-page styles for full-bleed hero and token-based p…
meganrm 482ec66
feat: replace antd List with native editorial ul/li in IdeaRoll
meganrm 247185e
fix: remove gap:0 no-op, explicit count check, delete dead IconText c…
meganrm 42eed6d
fix: use --color-primary token and document antd Tag override in eyeb…
meganrm 07f5556
small style tweaks
meganrm 126007d
rename colors
meganrm 6146ef5
use max width vars
meganrm ce8f09a
uncomment out the breakpoints
meganrm 75d758e
Merge branch 'feature/update-brand' of https://github.com/AllenCell/i…
meganrm 2dd1e51
update color names
meganrm 373d200
use border color
meganrm c4fbfa9
clean up styles
meganrm 353cb9c
Merge branch 'main' of https://github.com/AllenCell/idea-board into f…
meganrm b26e274
pull in changes from upstream
meganrm b66a99d
use new color name
meganrm 0259de3
use variable
meganrm 04f627c
update comment
meganrm dc4bab1
dont make a link if it's missing
meganrm 2dac9b7
add index page redesign spec
meganrm c842408
use semantic color names in index page redesign spec
meganrm 0189832
feat: add --content-padding-x layout tokens to colors.css
meganrm 79e72a6
feat: make index page full-width and wrap hero text in constrained in…
meganrm e55d053
feat: rewrite index-page styles for full-bleed hero and token-based p…
meganrm e81af9c
feat: replace antd List with native editorial ul/li in IdeaRoll
meganrm bdd7880
fix: remove gap:0 no-op, explicit count check, delete dead IconText c…
meganrm c3a0c49
fix: use --color-primary token and document antd Tag override in eyeb…
meganrm 48536b6
small style tweaks
meganrm 108bc1e
use new color name
meganrm ef24dfa
Merge branch 'fix/replace-list' of https://github.com/AllenCell/idea-…
meganrm b9d0aaf
fix color
meganrm 6ed5da0
Merge branch 'main' of https://github.com/AllenCell/idea-board into f…
meganrm abbeb2b
fix merge mistake
meganrm 1c4f231
Potential fix for pull request finding
meganrm 440da44
Potential fix for pull request finding
meganrm 926bf05
Potential fix for pull request finding
meganrm 0702f23
Potential fix for pull request finding
meganrm 3098ca8
Potential fix for pull request finding
meganrm d39754c
format
meganrm 7878d19
Add design spec for proposal callout color update
meganrm 6c50af6
changes based on comments
meganrm a2423ca
Merge branch 'main' of https://github.com/AllenCell/idea-board into f…
meganrm 9c47e0a
use orange for the call out box
meganrm File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
63 changes: 63 additions & 0 deletions
63
docs/design/specs/2026-05-29-proposal-callout-color-design.md
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,63 @@ | ||
| # Proposal Callout Box — Color Update Design | ||
|
|
||
| **Date:** 2026-05-29 | ||
| **Status:** Approved | ||
|
|
||
| ## Problem | ||
|
|
||
| The proposal callout box (`.proposal` in `idea-post.module.css`) uses `--accent-color` (→ `--ALLEN_GREEN` `#CDEB05`) as its background. The lime green is too bright and reduces readability. It also depletes the accent color's visual impact by using it as a large filled background rather than a sparse highlight. | ||
|
|
||
| ## Decision | ||
|
|
||
| Replace the lime green background with a muted warm-gray fill and an orange left border, using tokens already defined in the design system. | ||
|
|
||
| ### Color choices explored | ||
|
|
||
| - `--ALLEN_ORANGE` (`#FF6E00`) and `--ALLEN_ROSE` (`#FF00FF`) were evaluated as alternatives to `--ALLEN_GREEN` | ||
| - Orange was preferred: warmer, readable, distinctive without competing with the blue primary | ||
| - Orange is kept **exclusively on the proposal callout** — not extended to tags, links, or hero text — so it functions as a semantic signal for actionable/proposal content rather than a general accent | ||
|
|
||
| ### Callout style chosen | ||
|
|
||
| - **Background:** `--PAGE_2` (`#DED9D1`) — warm gray, slightly darker than the page background, clearly delineated without a filled accent color | ||
| - **Border:** 4px solid `--ALLEN_ORANGE` (`#FF6E00`) left border only — provides the color energy at the edge without flooding the reading area | ||
|
|
||
| A variant with fading gradient lines on the top and bottom was also considered but rejected in favor of the simpler solid-left-border approach. | ||
|
|
||
| ## Changes | ||
|
|
||
| ### `src/style/colors.css` | ||
|
|
||
| Update `--callout-box-border-color` (currently pointing to `--GRAY_1`) to point to `--ALLEN_ORANGE`: | ||
|
|
||
| ```css | ||
| --callout-box-border-color: var(--ALLEN_ORANGE); | ||
| ``` | ||
|
|
||
| ### `src/style/idea-post.module.css` | ||
|
|
||
| Wire `.proposal` to the callout tokens instead of hardcoding accent and dark colors: | ||
|
|
||
| ```css | ||
| .proposal { | ||
| background: var(--callout-box-bg-color); /* was: var(--accent-color) */ | ||
| border-left: 4px solid var(--callout-box-border-color); /* was: 4px solid var(--bg-dark-color) */ | ||
| padding: 14px 18px; | ||
| margin: 6px 0; | ||
| } | ||
| ``` | ||
|
|
||
| ## What does not change | ||
|
|
||
| - `--accent-color` remains `--ALLEN_GREEN` and continues to be used in the hero subtitle and nav hover states | ||
| - `--primary-color` (`--ALLEN_BLUE`) is unchanged on all tags, links, author names, and the post header background | ||
| - No new tokens are added — only `--callout-box-border-color` is redirected | ||
|
|
||
| ## Token inventory after change | ||
|
|
||
| | Token | Value | Used by | | ||
| |---|---|---| | ||
| | `--callout-box-bg-color` | `var(--PAGE_2)` → `#DED9D1` | `.proposal` background | | ||
| | `--callout-box-border-color` | `var(--ALLEN_ORANGE)` → `#FF6E00` | `.proposal` left border | | ||
| | `--accent-color` | `var(--ALLEN_GREEN)` → `#CDEB05` | Hero subtitle, nav hover, header sublabel | | ||
| | `--primary-color` | `var(--ALLEN_BLUE)` → `#6464FF` | Tags, links, post header, idea titles | | ||
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
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
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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this file be persisted in the repo?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.