Skip to content

[WIP] feat: Home page with banner, notifications, and dashboards#1574

Draft
kolbeyang wants to merge 4 commits intodevfrom
feat/home-page
Draft

[WIP] feat: Home page with banner, notifications, and dashboards#1574
kolbeyang wants to merge 4 commits intodevfrom
feat/home-page

Conversation

@kolbeyang
Copy link
Copy Markdown
Collaborator

Summary

  • Adds new "Home" page as the first sidebar navigation item
  • Auto-animating feature banner carousel (5 slides, 5s interval, vertical dot nav)
  • Recent Activity section with expandable notification cards (dummy data for now, BE doesn't exist yet)
  • Dashboards section with 3-column grid and pin toggle (local state only, no BE changes)

What's left

  • Backend for notifications/signals data
  • Backend for dashboard pinning persistence
  • Visual QA pass against Figma designs
  • Eve (QA) and Nathan (nit-pick) review passes

Test plan

  • Verify "Home" appears as first sidenav item
  • Banner auto-rotates every 5s, manual nav resets timer
  • Notification cards expand on click, dismiss on X, "Clear all" works
  • Dashboard cards render charts, pin toggle works
  • pnpm lint and pnpm type-check pass

🤖 Generated with Claude Code

kolbeyang and others added 4 commits April 2, 2026 22:54
…ards

Add new Home page as the first sidebar item with three sections:
- Auto-animating feature banner carousel with 5-second rotation and dot navigation
- Recent Activity section with expandable notification cards (dummy data)
- Dashboards grid section with pin toggle, reusing existing chart rendering

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…button

- Add h-[200px] to expanded activity card overlay to match Figma spec
- Change top padding from py-8 to pt-[65px] pb-8 to match Figma gap
- Wire Delete button on new_signal cards to call onDismiss

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Wrap the card in an onClick handler that navigates to the dashboard page
via router.push, while preserving the pin button with stopPropagation.
Previously only the chart name text was a clickable link.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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.

1 participant