Researcher deep pages: design-audit sweep + ProjectDetail declutter (batch unit 8)#38
Merged
Merged
Conversation
…declutter
- Token/color sweep of unit-8 CSS ranges: raw golds/purples/teals/navies and
gradients replaced with token-driven values (var(--accent/--gold/--card-bg/
--bg-muted/--accent-subtle) with legacy fallbacks); flattened comp-hero,
seg "brand edition", bar/kpi golds, cal-day gradient, drive/fm/dm chrome.
- Killed now-redundant explicit dark-mode rules in owned ranges (drive, fm,
kpi, safety menu, danger zone, comp-hero) - tokens + alphas cover them.
- Fixed .dm-modal background: var(--bg) had no fallback -> var(--card-bg, #fff).
- ProjectDetail declutter: local OverflowMenu ("...", SafetyMenu pattern +
styles) demotes per-row Remove (calendar events, resource links) and the
role editor trigger; removed redundant side-tab accent borders.
- Reachability: People header gains a "Find a mentor" ghost link; Apply page
gains a quiet "Also in Grow: Programs - Competitions" link row.
- Groups/GroupDetail banner placeholders flattened (.banner-ph); Community
image-dismiss scrim moved to .img-dismiss class; comp-hero glow divs removed.
- Motion polish: ease-out transitions, scale(0.97) presses, reduced-motion
kept intact; removed .bar-fill width animation (layout-property transition).
Note: shared/design-system.css (not owned by this unit) still overrides
.comp-hero with a blue gradient via !important - the styles.css flatten
takes effect once that override is removed by its owner.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
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
src/styles.cssranges + researcher page JSX): raw golds (#f59e0b,#ffd700gradients,#fffcf3/#f3e3ae/#b45309), off-palette purple (#8b5cf6.dot-event) and teal (#14b8a6.dot-mentor), ad-hoc navies (#16202e/#111a28/#0e1622dark overrides), and glass/gradient chrome replaced with token-driven values (--accent,--gold,--card-bg,--bg-muted,--accent-subtle,--shadow-*) with legacy-token fallbacks so the file renders correctly before and after the token foundation lands..banner-ph) flattened, body canvas -> flat--bg-muted..dm-modalused undefinedvar(--bg)with no fallback (transparent modal beforeshared/design-system.cssloads) ->var(--card-bg, #fff).OverflowMenu/MenuItem(same<details>pattern as SafetyMenu, reusing its popover styles for one consistent menu) demotes per-row Remove buttons (project calendar events, resource links) and the role-editor trigger into a quiet "..." menu; primary actions stay visible.btn-primary. Removed redundant 3px side-tab accent borders (info already carried by badges/headings)./researcher/mentors); Apply page -> quiet "Also in Grow: Programs · Competitions" link row.scale(0.97)presses on fm/drive buttons, removed.bar-fillwidth animation (layout-property transition), reduced-motion rules kept.Coordinator note
shared/design-system.css(owned by another unit; loads afterstyles.css) still re-imposes the.comp-heroblue gradient + glow via!important(lines ~1205-1231). The flatten in this PR takes effect as soon as that override block is removed by its owner — no further change needed here.Testing
npm run buildpasses (docs/ build output restored, not committed).sam@example.comdemo login): My Projects -> ProjectDetail (all 7 tabs; overflow menus open/close; role set via menu persists through API; primary actions intact), Calendar, Drive (grid/list, light + dark), Messages, People (Find-a-mentor link), Apply (Grow links), Groups + GroupDetail, Community feed, Account (danger zone). Light + dark at 1280px, ProjectDetail at 375px. No console errors.🤖 Generated with Claude Code