[#775] Redesign storyline header: Writer label + cleaner layout#781
[#775] Redesign storyline header: Writer label + cleaner layout#781realproject7 merged 2 commits intomainfrom
Conversation
- Change "by" to "Writer" prefix for author line - Split header into 3 clear sections: Title, Writer (with AgentBadge), Stats+Badges (plots, views, rating, genre, language) - Tighter spacing and better visual grouping - Mobile-friendly with flex-wrap Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
project7-interns
left a comment
There was a problem hiding this comment.
T2b APPROVE
Clean header restructure into 3 groups:
- Title: unchanged
- Writer: "by" → "Writer", AgentBadge correctly moved next to writer identity
- Stats+Badges: plots, views, rating, genre, language — all with
flex-wrapfor mobile
Good calls: tighter mt-2 spacing, gap-x-3 for stats row, AgentBadge placement is more logical next to the writer name. No concerns.
project7-interns
left a comment
There was a problem hiding this comment.
Verdict: REQUEST CHANGES
Summary
The Writer label change is correct, but the redesign still merges the stats, badges, and rating into one wrap row instead of separating them into the distinct groups requested by issue #775.
Findings
- [medium] The new header keeps
plots,views,rating,genre, andlanguagein a singleStats + Badgesrow, so the layout does not actually provide the requested grouping ofStats row,Genre + language badges, andRatingas separate sections.- File:
src/app/story/[storylineId]/page.tsx:278 - Suggestion: Keep the
plots/viewsrow together, move the genre/language pills to their own row or container, and renderRatingSummaryas its own section so the header structure matches the ticket.
- File:
Decision
Requesting changes because the PR improves spacing, but it does not fully implement the header grouping specified in issue #775.
…sections - Stats row: plots + views - Badges row: genre + language - Rating: separate section below 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: APPROVE
Summary
The header now matches issue #775: the author label is Writer, and the top info is split into distinct writer, stats, badges, and rating sections with mobile-safe wrapping.
Findings
- None.
Decision
Approving because the redesign now matches the ticket scope and stays focused on the StoryHeader layout update.
Summary
mt-2between groups,gap-x-3for statsflex-wrapon all rowsFixes #775
Test plan
npm run buildpasses🤖 Generated with Claude Code