fix(web): only show selection checkbox on normal view

[jabloink]

not sure if fix or feat

think border is enough for compact and ultra
can refactor to move badge if you want to keep the checkbox

Summary by CodeRabbit

• Bug Fixes
  • Fixed selection checkbox visibility to display only in normal view mode when selection is enabled.

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

[coderabbitai]

Walkthrough

The SwipeableCard component in TorrentCardsMobile now restricts checkbox visibility to selection mode with "normal" viewMode, instead of rendering whenever selection mode is active. An additional viewMode condition gates the checkbox rendering behavior.

Changes

Cohort / File(s)	Summary
Mobile Torrent Card Checkbox Rendering web/src/components/torrents/TorrentCardsMobile.tsx	Modified SwipeableCard checkbox visibility to render only when selectionMode is true AND viewMode is "normal", adding stricter conditional logic for checkbox display

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

• Single file change with straightforward conditional logic modification
• Additive condition (viewMode === "normal") to existing checkbox render check

Poem

🐰 A checkbox hides when views don't align,
Now "normal" mode makes conditions divine,
SwipeableCard learns to be more refined,
Selection and view in harmony bind! ✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'fix(web): only show selection checkbox on normal view' accurately describes the main change: adding a viewMode condition to render the selection checkbox only in normal view mode.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

Warning

Review ran into problems

🔥 Problems

Git: Failed to clone repository. Please run the @coderabbitai full review command to re-trigger a full review. If the issue persists, set path_filters to include or exclude specific files.

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (1)

web/src/components/torrents/TorrentCardsMobile.tsx (1)

643-648: Align text padding with checkbox visibility.

The pr-8 padding is currently applied whenever selectionMode is true (lines 645, 702, 737), but the checkbox is only visible when viewMode === "normal". This creates unnecessary right padding in compact and ultra-compact views during selection mode.

Consider updating the padding condition to match the checkbox visibility:

🔎 Proposed fix

 <h3 className={cn(
   "font-medium text-xs inline-block",
-  selectionMode && "pr-8"
+  selectionMode && viewMode === "normal" && "pr-8"
 )} title={displayName}>

Apply the same change at lines 702 and 737 for consistency across all view modes.

Also applies to: 700-706, 735-740

📜 Review details

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e9fcbda and 47c583e.

📒 Files selected for processing (1)

• web/src/components/torrents/TorrentCardsMobile.tsx (1 hunks)

🧰 Additional context used

🧠 Learnings (2)

📓 Common learnings

Learnt from: s0up4200
Repo: autobrr/qui PR: 643
File: web/src/components/torrents/FilterSidebar.tsx:1487-1494
Timestamp: 2025-11-29T10:55:09.170Z
Learning: The "dense" view mode in the FilterSidebar and torrent table is desktop-only. Mobile devices use a separate set of view modes: "normal", "compact", and "ultra-compact", explicitly excluding "dense" mode.

📚 Learning: 2025-12-03T18:11:08.682Z

Learnt from: finevan
Repo: autobrr/qui PR: 677
File: web/src/components/torrents/AddTorrentDialog.tsx:496-504
Timestamp: 2025-12-03T18:11:08.682Z
Learning: In the AddTorrentDialog component (web/src/components/torrents/AddTorrentDialog.tsx), temporary path settings (useDownloadPath/downloadPath) should be applied per-torrent rather than updating global instance preferences. The UI/UX is designed to suggest that these options apply to the individual torrent being added.

Applied to files:

• web/src/components/torrents/TorrentCardsMobile.tsx

🔇 Additional comments (1)

web/src/components/torrents/TorrentCardsMobile.tsx (1)

618-628: LGTM! Checkbox visibility correctly restricted to normal view.

The change appropriately limits checkbox rendering to normal view mode while preserving selection functionality in compact and ultra-compact views through the selection ring border and background color changes. This aligns with the PR objectives and provides a cleaner mobile UX for space-constrained views.