Skip to content

feat(home): center cards and contain posters so actions stay visible - #124

Merged
CodeWithMa merged 3 commits into
CodeWithMa:devfrom
CodeWithMaBot:t3code/117-redesign-home
Sep 4, 2026
Merged

feat(home): center cards and contain posters so actions stay visible#124
CodeWithMa merged 3 commits into
CodeWithMa:devfrom
CodeWithMaBot:t3code/117-redesign-home

Conversation

@CodeWithMaBot

@CodeWithMaBot CodeWithMaBot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

fixes #117

Summary by CodeRabbit

  • UI Updates
    • Restyled and reorganized the Next Series and Next Movie panels with a centered, compact layout and uppercase headings.
    • Updated action button styling and layout for improved presentation.
    • Refined empty-state messaging with a dashed border.
  • Changes
    • Removed the “Mark Completed” action from both panels.

@vercel

vercel Bot commented Sep 4, 2026

Copy link
Copy Markdown

@CodeWithMaBot is attempting to deploy a commit to the Ma's projects Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai

coderabbitai Bot commented Sep 4, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Team

Run ID: 910152de-e309-4f37-834c-4cbc218d564d

📥 Commits

Reviewing files that changed from the base of the PR and between 5e9d858 and d7a4caa.

📒 Files selected for processing (1)
  • src/app/components/home/home.component.ts

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The Next Series and Next Movie panels now use narrower centered layouts, updated headings and empty states, and three actions. The Mark Completed action was removed from both panels.

Changes

Home panel redesign

Layer / File(s) Summary
Panel layout and action updates
src/app/components/home/home.component.ts
The Next Series and Next Movie panels use centered 280px columns, uppercase headings, wrapped buttons, and dashed-border empty states. The Mark Completed button was removed from both panels.

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: ⚪ Minimal · up to d7a4c

The home panels now use a narrower centered layout and three visible actions without introducing an identified merge-readiness risk.

Suggested reviewers: codewithma

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the main changes: centered cards, contained posters, and visible actions.
Linked Issues check ✅ Passed The changes satisfy issue [#117] by reducing poster space, centering the home-page panels, showing full poster artwork, and keeping the remaining actions visible.
Out of Scope Changes check ✅ Passed The layout, poster, button, and empty-state changes are directly related to the home-page redesign objective in [#117]. No unrelated changes are identified.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1…
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@vercel

vercel Bot commented Sep 4, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated
watch-list Ready Ready Preview Sep 4, 2026 8:22am UTC

@CodeWithMa
CodeWithMa merged commit b839074 into CodeWithMa:dev Sep 4, 2026
6 checks passed
@CodeWithMaBot
CodeWithMaBot deleted the t3code/117-redesign-home branch September 5, 2026 06:35
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.

Redesign Home

2 participants