Skip to content

fix(mobile): tighten repo header actions and hide empty reports inbox - #399

Merged
chriswritescode-dev merged 1 commit into
mainfrom
fix/mobile-header-actions
Oct 8, 2026
Merged

fix(mobile): tighten repo header actions and hide empty reports inbox#399
chriswritescode-dev merged 1 commit into
mainfrom
fix/mobile-header-actions

Conversation

@chriswritescode-dev

@chriswritescode-dev chriswritescode-dev commented Oct 8, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Owner

Summary

  • Group the back button with the repo title so the name truncates with an ellipsis instead of colliding with the action buttons.
  • Unify the mobile header buttons (project actions, reports inbox, multi-run) into a single 40px ghost-icon cluster; the filled + stays the only prominent action. Desktop keeps the outlined labeled Multi-run button.
  • Hide the reports inbox when there are no unread schedule runs (it stays open if already expanded so the all-caught-up view survives Mark-all-read).
  • Header.Title now applies min-w-0, so titles can actually truncate on every page that uses it.

Test plan

  • vitest run for RepoDetail, ProjectActionsMenu, notifications, navigation suites (153 tests)
  • tsc --noEmit clean
  • eslint clean (one pre-existing unrelated warning)
  • Manual phone check on the repo/session headers (verified visually on the test instance at :5007)

Summary by CodeRabbit

  • Bug Fixes

    • The schedule reports bell is hidden when there are no unread reports and the panel is closed.

  • Style

    • Updated repository header and action button layouts for improved fit and usability on smaller screens.
    • Increased the size of action buttons and icons on smaller screens, returning to their previous dimensions at larger sizes.

Chris Scott
fix(mobile): tighten repo header actions and hide empty reports inbox
fbdc14b 
Group the back button with the title so the repo name truncates instead
of colliding with actions, unify the mobile header buttons to 40px
ghost icons, collapse the reports inbox when nothing is unread, and let
Header.Title truncate on every page.
@coderabbitai

coderabbitai Bot commented Oct 8, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Repository UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: a82640cc-e9b7-40ad-8525-6d687b14a7eb
📥 Commits

Reviewing files that changed from the base of the PR and between 22fd909 and fbdc14b.

📒 Files selected for processing (6)
  • frontend/src/components/notifications/ScheduleReportsBell.test.tsx
  • frontend/src/components/notifications/ScheduleReportsBell.tsx
  • frontend/src/components/repo/ProjectActionsMenu.tsx
  • frontend/src/components/ui/header.tsx
  • frontend/src/components/ui/pending-action-badge.tsx
  • frontend/src/pages/RepoDetail.tsx

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

📝 Walkthrough

Walkthrough

The reports button is hidden when no unread runs remain and the panel is closed. Repository header layout and action controls now use responsive sizing and spacing.

Changes

Scheduled reports visibility

Layer / File(s) Summary
Reports button visibility
frontend/src/components/notifications/ScheduleReportsBell.tsx, frontend/src/components/notifications/ScheduleReportsBell.test.tsx		 The button is absent when there are no unread runs and the panel is closed. Tests cover this state and the all-caught-up state after the unread total reaches zero.

Responsive repository controls

Layer / File(s) Summary
Repository header layout
frontend/src/components/ui/header.tsx, frontend/src/pages/RepoDetail.tsx		 The header title and leading group can shrink. The Multi-run action uses an icon button on small screens and a labeled button on larger screens.
Small-screen action sizing
frontend/src/components/repo/ProjectActionsMenu.tsx, frontend/src/components/ui/pending-action-badge.tsx		 The Project actions trigger and pending-action badge use larger button and icon dimensions below the small breakpoint.

Priority: ⬇️ Low

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

Change: Bug fix

Merge Risk: ⚪ Minimal · up to fbdc1

The reports inbox is hidden when empty and closed while remaining open to show the caught-up view, and header controls are adjusted for small screens. No concrete regression requiring resolution before merge is established.

🚥 Pre-merge checks | ✅ 4 | ❌ 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%. Docstring coverage is scoped to functions touched by this diff. Analyzed 5 functions across 6 files. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main changes: improving mobile repository header actions and hiding the empty reports inbox.
Description check ✅ Passed The description clearly explains the layout, reports-inbox, and truncation changes. It also includes test results and validation details. The template headings for Type of Change and Checklist are not…
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.
  • Fix all pre-merge checks with AI
✨ Finishing Touches 💡 1
📝 Generate docstrings 💡
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autopilot · 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

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

@chriswritescode-dev
chriswritescode-dev merged commit bdbcc60 into main Oct 8, 2026
2 checks passed
@chriswritescode-dev
chriswritescode-dev deleted the fix/mobile-header-actions branch October 8, 2026 03:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@chriswritescode-dev