[#822] Replace dual-render with CSS grid-template-areas

[realproject7]

Summary

• Replaced hidden sm:block + sm:hidden dual-render pattern with CSS grid-template-areas
• Stats grid and CTA button now rendered once in the DOM instead of twice
• Mobile: stats span full width below cover+info row ('cover info' / 'stats stats')
• Desktop: stats align with info column only ('cover info' / '. stats')
• MarketCapBox and DeadlineCountdown now mount once (no duplicate API calls/timers)
• No visual changes on any breakpoint

Test plan

• Desktop: stats grid + CTA next to cover in right column
• Mobile: stats grid full-width below cover+info row
• Mobile: CTA button full-width
• Desktop: CTA button inline/auto-width
• Inspect DOM: stats grid and CTA appear only once
• Verify MarketCapBox network calls fire once (not twice)

Fixes #822

🤖 Generated with Claude Code

[vercel]

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

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
plotlink	Ignored		Apr 4, 2026 5:21am

[project7-interns]

LGTM. Grid-template-areas approach correctly eliminates the dual-render pattern. Stats and CTA mount once, grid columns preserve the 100px/160px cover sizing, and the responsive sm: reset on CTA width (w-auto) matches the previous default. No visual or logic regressions.

[project7-interns]

Verdict: REQUEST CHANGES

Summary

The single-render grid approach is the right fix for issue #822, but this version introduces a layout regression when priceInfo is missing.

Findings

• [medium] The new stats grid item is always rendered with mt-4 sm:mt-6, even when both statsGrid and AddPlotButton render nothing. On untokenized stories, that leaves an empty spacer below the header for all viewers, and when only the CTA is present it adds extra top spacing versus the old layout.
  • File: src/app/story/[storylineId]/page.tsx:384
  • Suggestion: Keep the grid-area wrapper marginless and apply the responsive top margin only to statsGrid, so the CTA-only / empty states preserve the previous spacing.

Decision

Requesting changes because the issue requires no visual regressions, and this change alters the header spacing for stories without price data.

[project7-interns]

Verdict: APPROVE

Summary

The follow-up commit fixes the header spacing regression from the earlier review while preserving the single-render grid-template-areas solution for issue #822.

Findings

• None. The mt-4 sm:mt-6 spacing now applies only when statsGrid exists, so untokenized stories no longer get empty spacer height and CTA-only states keep their previous spacing.

Decision

Approving because the PR now meets the acceptance criteria: stats and CTA render once in the DOM, side effects mount once, and the prior visual regression is resolved.