Skip to content

Fix repository activity display and mobile UI issues - #376

Merged
Ri1tik merged 2 commits into
AOSSIE-Org:mainfrom
zaibamachhaliya:fix/footer-repo-activity-mobile-modal
Oct 19, 2026
Merged

Ri1tik merged 2 commits into
AOSSIE-Org:mainfrom
zaibamachhaliya:fix/footer-repo-activity-mobile-modal

Conversation

@zaibamachhaliya

@zaibamachhaliya zaibamachhaliya commented Oct 19, 2026 •

Copy link
Copy Markdown
Contributor

Screenshots/Recordings:

Leptop recording

OrgExplorer.GitHub.Organization.Analytics.Repository.Insights.-.Google.Chrome.2026-10-19.07-55-48.mp4

Mobile recording

Record_2026-10-19-07-48-42_40deb401b9ffe8e1df2f1cc5ba480b12.mp4

Additional Notes:

Context

This PR addresses three reported issues:

  • Repository activity was not appearing under the correct column in the Repositories tab.
  • A vertical divider appeared through the footer on mobile screens.
  • The mobile navigation menu closed with the close button, but did not close when tapping outside it.

Changes

  • Added the missing Language cell to each repository row so table data aligns with the Language and Repository Activity headers. Activity status and the last-push date now appear under Repository Activity.
  • Hid footer divider elements on mobile screens to remove the unwanted vertical line.
  • Added outside-tap dismissal for the mobile navigation menu. The close button continues to work.

Files changed

  • src/pages/RepositoriesPage.jsx — added the missing Language cell.
  • src/styles/global.css — hid footer dividers on mobile.
  • src/components/Navbar.jsx — close the mobile menu when tapping outside it.

Verification

  • Confirmed the repository columns display in the correct places.
  • Checked the footer on a phone using the local development site.
  • Tested that the mobile menu closes using both the close button and an outside tap.

Summary by CodeRabbit

  • New Features
    • Repository tables now show each repository’s programming language, displaying “N/A” when it isn’t available.
  • Bug Fixes
    • The mobile navigation menu now closes when you click outside it or tap its overlay.
  • Style
    • Hid the divider in the mobile footer.

@github-actions github-actions Bot added no-issue-linked PR has no linked issue frontend Frontend changes javascript JavaScript/TypeScript changes size/S 11-50 lines changed labels Oct 19, 2026
@coderabbitai

coderabbitai Bot commented Oct 19, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Repository: AOSSIE-Org/OrgExplorer/.coderabbit.yaml
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: e656b8fd-aa4f-44e4-9243-0fb42af1c483
📥 Commits

Reviewing files that changed from the base of the PR and between e74bcd0 and 09a92a4.

📒 Files selected for processing: 3
  • src/components/Navbar.jsx
  • src/pages/RepositoriesPage.jsx
  • src/styles/global.css

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

Walkthrough

The mobile navigation menu now closes on outside pointer events and overlay clicks. Repository rows display language values or N/A. The mobile footer divider is hidden.

Changes

Mobile menu dismissal

Layer / File(s) Summary
Mobile menu dismissal behavior
src/components/Navbar.jsx
The menu closes when a pointer event occurs outside it or when the overlay itself is clicked. The document-level listener is removed during effect cleanup. The existing overflow-lock behavior remains.

Repository language display

Layer / File(s) Summary
Repository language cell
src/pages/RepositoriesPage.jsx
Repository rows display each repository’s language, or N/A when the language is unset.

Mobile footer styling

Layer / File(s) Summary
Hide mobile footer divider
src/styles/global.css
The mobile .site-footer-divider rule now sets display: none.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Bug fix

Suggested labels: Typescript Lang

Suggested reviewers: rahul-vyas-dev

Merge Risk: ⚪ Minimal · up to 09a92

The repository language fallback, mobile footer divider, and outside-tap menu dismissal have no supported unresolved issue in this review. The change appears ready to merge after normal checks.

Pre-merge checks: Passed 4
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly summarizes the main changes: repository activity alignment and mobile UI fixes.
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

A rabbit taps the menu’s side,
The open panel shuts with pride.
Repository tongues appear in rows,
The footer line fades as it goes.
Then hops away through leafy green.

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added external-contributor External contributor size/S 11-50 lines changed and removed size/S 11-50 lines changed labels Oct 19, 2026
@adityarana0310

Copy link
Copy Markdown

I noticed a small issue in the mobile footer fix: hiding all .site-footer-divider elements also removes the upper border that separates the main content from the footer. I'd like to help fix this by preserving the upper border and hiding only the redundant divider. Would it be okay if I worked on this correction and submitted a small PR?
Screenshot 2026-10-19 134341
Screenshot 2026-10-19 133504

@gitcordapp

gitcordapp Bot commented Oct 19, 2026

Copy link
Copy Markdown

Link your account with Gitcord

Thanks for opening this PR, @zaibamachhaliya!

To receive Discord notifications and contributor tracking for this organization:

  1. Join Discord: https://discord.gg/hjUhu33uAn
  2. In Discord, run /link zaibamachhaliya
  3. Paste the verification code into your GitHub bio (or a public gist)
  4. Click Verify in Discord (or run /verify-link zaibamachhaliya)

Once linked, Gitcord can notify you about reviews, merges, and more.

— Posted by Gitcord

@Ri1tik
Ri1tik merged commit d91d2b8 into AOSSIE-Org:main Oct 19, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

external-contributor External contributor frontend Frontend changes javascript JavaScript/TypeScript changes no-issue-linked PR has no linked issue size/S 11-50 lines changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants