Skip to content

[#836] Fix desktop stats boxes falling below Moleskine cover#853

Merged
realproject7 merged 1 commit intomainfrom
task/836-desktop-stats-beside-cover
Apr 12, 2026
Merged

[#836] Fix desktop stats boxes falling below Moleskine cover#853
realproject7 merged 1 commit intomainfrom
task/836-desktop-stats-beside-cover

Conversation

@realproject7
Copy link
Copy Markdown
Owner

Summary

  • Replaced grid-template-areas with explicit grid placement (row-span-2 on cover, col-span-2 sm:col-span-1 sm:col-start-2 on stats) so the cover spans both rows on desktop and stats sit in the right column beside it
  • Mobile layout is completely unchanged — stats still span full width below cover on small screens
  • Patch version bump 0.1.18 → 0.1.19

Fixes #836

Test plan

  • Desktop (≥640px): stats boxes appear to the RIGHT of the Moleskine cover, not below
  • Desktop: "Add a new Plot" button also in the right column
  • Mobile (<640px): layout unchanged — stats span full width below cover+info row
  • No visual regressions on story detail page

🤖 Generated with Claude Code

Replace grid-template-areas with explicit grid placement (row-span,
col-span, col-start) so the cover spans both rows on desktop and
stats sit beside it in column 2. Mobile layout unchanged.

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

vercel Bot commented Apr 12, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
plotlink Ignored Ignored Apr 12, 2026 2:45am

Request Review

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 layout change is minimal and matches issue #836: on desktop the cover spans both rows and the stats/CTA move into the right column, while mobile behavior remains unchanged.

Findings

  • [info] GitHub checks are still pending at review time (e2e, lint-and-typecheck, visual-regression)
    • File: PR #853
    • Suggestion: Let CI finish before merge.

Decision

Approved because the implementation is focused, follows the existing Tailwind layout pattern, and satisfies the acceptance criteria for #836.

Copy link
Copy Markdown
Owner Author

@realproject7 realproject7 left a comment

Choose a reason for hiding this comment

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

re2 APPROVE — LGTM. Explicit grid placement (row-span-2 / col-start-2) is cleaner than grid-template-areas and the mobile/desktop logic is correct. No regressions expected.

@realproject7 realproject7 merged commit 3b633e1 into main Apr 12, 2026
5 checks 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.

[Bug] Desktop: stats boxes fall below Moleskine cover instead of beside it

2 participants