Skip to content

Conversation

@jabloink
Copy link
Contributor

@jabloink jabloink commented Jan 27, 2026

Before

image

After

image

Summary by CodeRabbit

  • Style
    • Adjusted torrent card dimensions and spacing in mobile view
    • Repositioned scroll-to-top button in mobile layout for improved visibility

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

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 27, 2026

Walkthrough

This PR adjusts layout parameters in the mobile torrents card component by reducing the virtualizer item height estimate from 204 to 180 pixels and repositioning the scroll-to-top button closer to the bottom edge of the mobile screen.

Changes

Cohort / File(s) Summary
Mobile UI Layout Adjustments
web/src/components/torrents/TorrentCardsMobile.tsx
Reduced virtualizer estimateSize from 204 to 180 for non-ultra-compact/compact views; decreased ScrollToTopButton bottom offset from calc(8rem+env(safe-area-inset-bottom)) to calc(3rem+env(safe-area-inset-bottom))

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested labels

web, mobile, cosmetic, bugfix

Suggested reviewers

  • s0up4200

Poem

🐇 A button hops down, closer to the ground,
Items shrink in height, more cards abound,
Mobile scrolls smoother, spacing refined,
Little tweaks of layout, carefully designed! 📱✨

🚥 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 accurately reflects the main changes: adjusting card spacing on mobile by modifying virtualizer height calculation and ScrollToTopButton positioning.

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

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/mobile-card-spacing

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.

@s0up4200 s0up4200 changed the title Fix/mobile card spacing fix(web): mobile card spacing Jan 27, 2026
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.

3 participants