Skip to content

Mobile responsiveness for Recent Milestones section#3682

Closed
dhirukumar wants to merge 1 commit intoOWASP:mainfrom
dhirukumar:Mentorship-Programs
Closed

Mobile responsiveness for Recent Milestones section#3682
dhirukumar wants to merge 1 commit intoOWASP:mainfrom
dhirukumar:Mentorship-Programs

Conversation

@dhirukumar
Copy link
Contributor

@dhirukumar dhirukumar commented Jan 30, 2026

Proposed change

Resolves #3599
Updated the grid layout classes to use a mobile-first approach:

Result

✅ Fully responsive layout across all screen sizes
✅ No overflow or layout breaking on mobile devices
✅ Improved user experience on smaller screens

Checklist

I followed the contributing workflow
I verified that my code works as intended and resolves the issue as described
I ran make check-test locally: all warnings addressed, tests passed

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 30, 2026

Walkthrough

The change modifies grid container wrappers in CardDetailsPage.tsx to enforce explicit responsive grid behavior. It adjusts the RecentIssues/Milestones and RecentPullRequests/RecentReleases sections from default two-column layouts to single-column on small screens, switching to two columns at large breakpoints.

Changes

Cohort / File(s) Summary
Grid Layout Responsiveness
frontend/src/components/CardDetailsPage.tsx
Adjusts grid container wrappers for RecentIssues/Milestones and RecentPullRequests/RecentReleases sections to use explicit responsive grid with single-column default and two-column layout at large screens.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested labels

frontend

Suggested reviewers

  • arkid15r
  • kasya
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed Title clearly identifies the main change targeting the Recent Milestones section responsiveness on mobile, matching the core objective of the pull request.
Linked Issues check ✅ Passed Changes implement responsive grid layout adjustments for CardDetailsPage component to fix mobile responsiveness issues identified in #3599 for Recent Milestones section.
Out of Scope Changes check ✅ Passed All changes are narrowly focused on responsive grid layout in CardDetailsPage.tsx, directly addressing the mobile responsiveness objective from issue #3599.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description check ✅ Passed The pull request description is directly related to the changeset, explaining the mobile-first responsive grid layout update that resolves issue #3599.

✏️ 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

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.

@sonarqubecloud
Copy link

@kasya
Copy link
Collaborator

kasya commented Feb 3, 2026

@dhirukumar your branch doesn't have recent changes from main that would show Recent milestones for programs at all 🤔
I'm wondering how did you test this out locally...

Closing this in favor of #3674

@kasya kasya closed this Feb 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Mentorship Programs page is not responsive on smaller screens

2 participants