Skip to content

[#289] Book-cover style story cards on home page#290

Merged
realproject7 merged 2 commits intomainfrom
task/289-book-cover-cards
Mar 18, 2026
Merged

[#289] Book-cover style story cards on home page#290
realproject7 merged 2 commits intomainfrom
task/289-book-cover-cards

Conversation

@realproject7
Copy link
Copy Markdown
Owner

Summary

  • Redesigned StoryCard as portrait book cover (2:3 aspect ratio) with genre tag at top, centered title, author at bottom
  • Metadata (plot count, TVL/price, rating) moved to a compact row below the card
  • Home page grid updated: 2 columns on mobile (was 1), 3 on desktop (unchanged)
  • Flat outline aesthetic preserved — no gradients, shadows, or 3D effects
  • Git tag pre-book-cards on main for easy revert

Revert

git revert <merge-commit-hash>

Test plan

  • Cards display as portrait book shape with outline border
  • Title centered, genre at top, author at bottom inside card
  • TVL, plot count, rating below the card
  • Desktop: 3-column grid
  • Mobile: 2-column grid (cards still readable)
  • Dark theme + monospace font preserved
  • npm run typecheck passes

Fixes #289

🤖 Generated with Claude Code

- StoryCard redesigned as portrait 2:3 aspect ratio book cover with
  genre tag (top), title (center), author (bottom)
- Metadata row (plot count, TVL, rating) moved below the card
- Grid: 2 columns on mobile, 3 on desktop
- Flat outline aesthetic preserved, no gradients or shadows

Fixes #289

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown
Collaborator

@project7-interns project7-interns left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

T2b Review — APPROVED

Book-cover layout is clean: 2:3 aspect ratio, genre/title/author layered correctly, flat outline preserved. Grid change (2-col mobile, 3-col desktop) matches spec.

One nit (non-blocking): the preview prop is still destructured in StoryCard and passed from page.tsx, but is no longer rendered. Dead code — remove the prop and the previews lookup in a follow-up or in this PR before merge.

Ship it.

Copy link
Copy Markdown
Collaborator

@project7-interns project7-interns left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verdict: APPROVE

Summary

The homepage card redesign stays within the requested scope: portrait cover-style cards, metadata row below, 2-column mobile grid, and the existing flat outline aesthetic preserved.

Findings

  • No blocking findings from this review pass.

Decision

Approving from my side. GitHub checks were still pending at the time of review.

Book cover cards don't display previews — remove the dead code path
and the extra Supabase query.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@realproject7 realproject7 merged commit 7d4744f into main Mar 18, 2026
1 check passed
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.

[design] Book-cover style story cards on home page

2 participants