fix: improve mobile responsiveness of Releases page#170
Conversation
✅ Deploy Preview for cv-community-dashboard ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
WalkthroughUpdates the Releases feature to improve responsive layout and spacing: adds Possibly related PRs
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: Organization UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
components/Releases/ReleaseCard.tsx (1)
12-26: Prevent overflow in thesm:flex-rowheader by allowing the left column to shrink.Consider adding
min-w-0to the left column, and limitingnowrapto larger screens.Proposed fix
- <div className="flex flex-col sm:flex-row sm:items-start sm:justify-between gap-2 sm:gap-4"> - <div> + <div className="flex flex-col sm:flex-row sm:items-start sm:justify-between gap-2 sm:gap-4"> + <div className="min-w-0"> <h3 className="text-base sm:text-lg font-semibold break-words"> {release.repo} — {release.version} </h3> <p className="text-sm text-muted-foreground break-words"> {release.summary} </p> </div> - <span className="text-xs sm:text-sm text-zinc-500 whitespace-nowrap sm:self-start"> + <span className="text-xs sm:text-sm text-zinc-500 whitespace-normal sm:whitespace-nowrap sm:self-start"> {release.date} </span> </div>
📜 Review details
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (3)
.gitignoreapp/releases/page.tsxcomponents/Releases/ReleaseCard.tsx
🔇 Additional comments (2)
components/Releases/ReleaseCard.tsx (1)
28-58: Contributors wrap + tighter gap looks good for mobile.app/releases/page.tsx (1)
8-25: Mobile-first container + responsive typography/spacing are consistent and should fix the small-viewport breakage.Minor: worth confirming
repoSlugis always defined and thatrepoSlug + versionis unique across the dataset (otherwise React key collisions).
| !app/icons.gen.ts | ||
| !app/overrides.gen.css No newline at end of file | ||
| !app/overrides.gen.css | ||
|
|
||
| public/releases/releases.json No newline at end of file |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
# Search for any references to releases.json or public/releases
rg "releases\.json" --type-listRepository: CircuitVerse/community-dashboard
Length of output: 465
🏁 Script executed:
# Find all references to releases.json
rg "releases\.json" -iRepository: CircuitVerse/community-dashboard
Length of output: 193
🏁 Script executed:
# Check for any build/generation scripts
rg "releases|getReleases" -i -A 2 --type ts --type js --type jsonRepository: CircuitVerse/community-dashboard
Length of output: 7500
🏁 Script executed:
# Look at the current .gitignore file to understand context
cat -n .gitignore | head -75Repository: CircuitVerse/community-dashboard
Length of output: 1209
🏁 Script executed:
# Check package.json for build, generate, or prepare scripts
rg "generateReleases|generate" package.jsonRepository: CircuitVerse/community-dashboard
Length of output: 209
🏁 Script executed:
# Look at the full package.json to see all scripts
cat package.json | grep -A 30 '"scripts"'Repository: CircuitVerse/community-dashboard
Length of output: 1098
🏁 Script executed:
# Check if there's a build config that runs the generation
cat next.config.js || echo "no next.config.js"Repository: CircuitVerse/community-dashboard
Length of output: 157
🏁 Script executed:
# Also check for pre-commit or CI automation
fd "\.github|\.gitlab|Makefile|package\.json" --type f | head -20Repository: CircuitVerse/community-dashboard
Length of output: 88
The releases.json file is required at runtime but won't exist on fresh clones or CI.
The app/releases/page.tsx calls getReleases() at runtime, which reads from public/releases/releases.json. However, scripts/generateReleases.ts (which generates this file) is not included in the build pipeline—unlike generate:leaderboard and generate:workflow which have dedicated package.json scripts. Since the file is also ignored in .gitignore, it will be missing on fresh clones and CI runs, breaking the releases page.
Add the generation script to the build pipeline (e.g., as a prebuild step or explicit generate:releases script), or ensure the app gracefully handles the missing file. Additionally, anchor the .gitignore pattern with /public/releases/releases.json to avoid accidental matches elsewhere.
Git-HimanshuRathi
left a comment
There was a problem hiding this comment.
@Ayush1169 fix the code rabbit suggestions .
|
Thanks for the review @Git-HimanshuRathi Just to clarify, The file is intentionally ignored and generated via the workflow when needed. All UI-related CodeRabbit suggestions have been addressed. |
Git-HimanshuRathi
left a comment
There was a problem hiding this comment.
The file is generated later by a workflow when needed, and the Releases page is designed to work even if the file is not present. Because of that, it’s not required to be part of the build process. so can u please fix what code rabbit has suggested also there is some conflicts too .
Description
This PR fixes mobile responsiveness issues on the Releases page.
The layout has been updated to be mobile-first so that release cards render cleanly on smaller screens without layout breaks or cramped content.
Related Issue
Fixes #169
Type of change
Checklist
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.