Skip to content

Storyline page: MCap + Deadline on same line on desktop#792

Merged
realproject7 merged 1 commit intomainfrom
task/783-storyline-stats-grid
Apr 3, 2026
Merged

Storyline page: MCap + Deadline on same line on desktop#792
realproject7 merged 1 commit intomainfrom
task/783-storyline-stats-grid

Conversation

@realproject7
Copy link
Copy Markdown
Owner

Summary

  • Moves Deadline/Story Complete into the existing 2-col stats grid
  • Desktop (sm+): MCap + Supply Minted in left column, Deadline in right column
  • Mobile: Stacks vertically as before (unchanged behavior)

Layout Change

Before: MCap+Supply in a 2-col grid, Deadline in a separate box below
After: Single bordered grid with MCap+Supply (left) and Deadline (right) on desktop

Test Plan

  • Desktop: verify MCap and Deadline appear side by side
  • Mobile: verify vertical stacking is preserved
  • Verify "Story complete" state renders correctly in the right column
  • Verify storylines without last_plot_time don't show an empty right column

Fixes #783

🤖 Generated with Claude Code

Move Deadline/Story Complete into the existing 2-col stats grid so
MCap+Supply sit in the left column and Deadline sits in the right
column on sm+ screens. Mobile stacks vertically as before.

Fixes #783

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

vercel Bot commented Apr 3, 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 3, 2026 8:16am

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 matches the issue scope: desktop gets the market-cap/deadline row in a shared 2-column container, while mobile still stacks vertically. I did not find a code-level regression in the header composition.

Findings

  • None.

Decision

Approved on code review. CI was still pending when I reviewed.

@realproject7 realproject7 merged commit 8db3b7d into main Apr 3, 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 page: MCap + Deadline should be same line on desktop

2 participants