Skip to content

fix(leaderboard): fix layout centering#215

Merged
AlinaRyabova merged 12 commits into
developfrom
feature/leaderboard-style-update
Jan 26, 2026
Merged

fix(leaderboard): fix layout centering#215
AlinaRyabova merged 12 commits into
developfrom
feature/leaderboard-style-update

Conversation

@AlinaRyabova
Copy link
Copy Markdown
Collaborator

@AlinaRyabova AlinaRyabova commented Jan 26, 2026

fix layout centering

Summary by CodeRabbit

  • Style
    • Leaderboard table now spans full width for improved visibility.
    • Enhanced border styling to better distinguish the current user's row from other entries.

✏️ Tip: You can customize this high-level summary in your review settings.

@netlify
Copy link
Copy Markdown

netlify Bot commented Jan 26, 2026

Deploy Preview for develop-devlovers ready!

Name Link
🔨 Latest commit 1a8585f
🔍 Latest deploy log https://app.netlify.com/projects/develop-devlovers/deploys/69779274e048ca0008101679
😎 Deploy Preview https://deploy-preview-215--develop-devlovers.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Jan 26, 2026

📝 Walkthrough

Walkthrough

The pull request modifies the LeaderboardTable component by removing the maximum width constraint on the container, allowing it to use full width, and introduces an explicit rightBorderClass constant for styling table row borders, enabling distinct right border styling for current users versus other users.

Changes

Cohort / File(s) Summary
Leaderboard Table Layout & Border Styling
frontend/components/leaderboard/LeaderboardTable.tsx
Removed max-w-4xl constraint on outer container div to enable full-width layout; introduced explicit rightBorderClass constant in TableRow for distinct right border styling alongside existing leftBorderClass.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • #192: Modifies the same LeaderboardTable.tsx file focusing on row and border styling adjustments.
  • #214: Updates LeaderboardTable.tsx with similar rightBorderClass and table layout/width changes.
  • #206: Introduces the same rightBorderClass constant and adjusts border styling for current-user versus other users in the same component.

Suggested reviewers

  • AM1007
  • ViktorSvertoka

Poem

🐰 A leaderboard grows wide and free,
With borders left and right to see,
No width constraints to bind our sight,
Full width now shines so very bright!

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'fix(leaderboard): fix layout centering' directly relates to the main change: removing the max-w-4xl constraint and adjusting the container to use full width (w-full), which addresses layout centering issues.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

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 and usage tips.

@AlinaRyabova AlinaRyabova merged commit b7a5e14 into develop Jan 26, 2026
8 checks passed
@AlinaRyabova AlinaRyabova deleted the feature/leaderboard-style-update branch January 26, 2026 16:15
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.

2 participants