Skip to content

[#823] Increase mobile Moleskine cover width to 130px#826

Merged
realproject7 merged 1 commit intomainfrom
task/823-cover-width-mobile
Apr 4, 2026
Merged

[#823] Increase mobile Moleskine cover width to 130px#826
realproject7 merged 1 commit intomainfrom
task/823-cover-width-mobile

Conversation

@realproject7
Copy link
Copy Markdown
Owner

Summary

  • Increased mobile grid column from 100px to 130px so genre tags like "SCIENCE FICTION" fit on one line
  • Desktop cover unchanged at 160px
  • Matches the Moleskine size from the Writer tab on the profile page
  • On 320px screens: 320 - 130 - 16 gap = 174px for info column

Test plan

  • Mobile: genre tags like "SCIENCE FICTION" fit on one line
  • Mobile: cover width visually matches Writer tab reference
  • Desktop: cover unchanged at 160px
  • 320px viewport: info column still has adequate space

Fixes #823

🤖 Generated with Claude Code

Genre tags like "SCIENCE FICTION" were wrapping to two lines on mobile.
Widen the grid column from 100px to 130px to match the Writer tab
reference. Desktop stays at 160px.

Fixes #823

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

vercel Bot commented Apr 4, 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 4, 2026 5:28am

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.

LGTM. Single-line grid-cols change from 100px to 130px on mobile. Matches Writer tab reference, desktop unchanged at 160px, and 320px viewport still leaves 174px for the info column.

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 PR matches issue #823 exactly: it widens the mobile cover column to 130px, keeps desktop at 160px, and stays scoped to a single responsive layout adjustment.

Findings

  • None. The one-line grid-cols update in src/app/story/[storylineId]/page.tsx satisfies the acceptance criteria without introducing logic or structural changes.

Decision

Approving because the change is minimal, follows the existing header pattern, and preserves the 320px-space calculation called out in the issue.

@realproject7 realproject7 merged commit a8e1619 into main Apr 4, 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] Storyline header: Moleskine cover too narrow on mobile — genre tag breaks

2 participants