[#416] Moleskine notebook cards on home page#418
Conversation
- Redesign StoryCard with Moleskine notebook aesthetic: rounded corners (5px 15px 15px 5px), elastic band near right edge, label band with title - Cover opens on hover (rotateY(-50deg)) revealing page underneath - Use @media (hover: hover) to limit animation to desktop only - Keep warm palette (--accent: #8B4513, --bg: #E8DFD0) - Replace shelf rows with plain responsive grid (2-col mobile, 3-col desktop) Fixes #416 Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
project7-interns
left a comment
There was a problem hiding this comment.
Verdict: REQUEST CHANGES
Summary
The PR matches the requested scope for issue #416 from the diff I reviewed, and I do not have code findings to raise.
Findings
- [medium]
lint-and-typecheckis still pending, so the PR does not yet satisfy the review checklist requirement that the build passes.- File:
GitHub Actions / lint-and-typecheck - Suggestion: Re-route for final approval once the required check completes successfully.
- File:
Decision
Holding approval until CI is green. I can re-review immediately once lint-and-typecheck passes.
project7-interns
left a comment
There was a problem hiding this comment.
T2b Review — APPROVED
All requirements from #416 verified:
- Moleskine shape: Rounded corners
5px 15px 15px 5pxon cover ✅ - Elastic band: Positioned near right edge with striped gradient + subtle shadow ✅
- Label band: Cream
#e8e8e0band with centered Montserrat title ✅ - Hover animation:
rotateY(-50deg)on cover +rotateZ(-10deg)container tilt, revealing ruled page underneath ✅ - Mobile safety:
@media (hover: hover)gates all animation — static on touch devices ✅ - Palette: Cover uses
var(--accent)(#8B4513), page#FFF8EE, no dark/vivid CodePen colors ✅ - Grid: Shelf rows replaced with clean 2-col/3-col responsive grid ✅
CSS architecture is clean — animation classes in globals.css, component in StoryCard.tsx. The transform-origin: left center correctly opens the cover like a book from the spine edge. z-index: 999 on hover is fine for a grid layout.
Minor observation (non-blocking): TVL info appears both on the revealed page and in the metadata below the card. On desktop hover, both are visible simultaneously. Acceptable since mobile users only see the metadata below.
Clean, well-scoped PR.
project7-interns
left a comment
There was a problem hiding this comment.
Verdict: APPROVE
Summary
Re-review complete. The PR matches issue #416 scope, and the required lint-and-typecheck check is now passing.
Findings
- No blocking findings.
Decision
Approved.
Summary
rotateY(-50deg)+ container tiltsrotateZ(-10deg), revealing a ruled page underneath — desktop only via@media (hover: hover)var(--accent)(#8B4513), page uses#FFF8EE, label band#e8e8e0Fixes #416
Test plan
@media (hover: hover))npm run typecheckpassesnpm run lintpasses🤖 Generated with Claude Code