Skip to content

feat: redesign runner list controls and add collapsible groups - #78

Open
fortrel wants to merge 5 commits into
omniaura:mainfrom
fortrel:feature/enhancedGui
Open

feat: redesign runner list controls and add collapsible groups#78
fortrel wants to merge 5 commits into
omniaura:mainfrom
fortrel:feature/enhancedGui

Conversation

@fortrel

@fortrel fortrel commented Jul 23, 2026

Copy link
Copy Markdown

Summary

  • Replace the flat red/green stop-start icons with monochrome icons in a circular button, and fix runner label badges so they no longer wrap mid-word — they now flow onto additional lines as needed instead of being clipped.
  • Make each repo/org group header in the runner list tappable to collapse/expand its runners, with an animated chevron indicator.
  • Add a dismiss (×) button to the update-available banner so it can be hidden without installing; it reappears automatically if a newer version becomes available.
  • Fix Makefile's run and app targets, which referenced stale binary paths/names (.build/debug/MacRunner and .build/release/MacRunner) that no longer matched the package's actual product name (mac-runner) or swift build's multi-arch output location, and were failing for local development builds.

Test plan

  • swift build succeeds
  • swift test — 105/105 tests pass
  • make app builds a working .app bundle
  • Verified button, label-wrapping, collapse/expand, and update-banner dismiss behavior via isolated SwiftUI ImageRenderer snapshots (no Screen Recording/Accessibility permission available in this environment to drive the live popover)

🤖 Generated with Claude Code

Summary by CodeRabbit

  • New Features

    • Runner groups can now be collapsed or expanded individually, with animated indicators.
    • Update notifications can be dismissed for a specific available version.
    • Runner labels and status indicators now wrap neatly across multiple lines.
  • UI Improvements

    • Improved spacing and sizing for runner information.
    • Refined start/stop button styling with clearer circular icon treatment.

@coderabbitai

coderabbitai Bot commented Jul 23, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: bca6106d-97ce-4464-93b8-84ce8e09c748

📥 Commits

Reviewing files that changed from the base of the PR and between 2f710c3 and 9ff0c18.

📒 Files selected for processing (3)
  • .gitignore
  • Makefile
  • Sources/Views/MenuBarView.swift

📝 Walkthrough

Walkthrough

The PR updates Swift build and app-bundle executable paths, adds ignored build and VS Code artifacts, introduces collapsible runner groups and dismissible update prompts, and replaces runner-row chip layout with a wrapping custom FlowLayout.

Changes

Menubar and build updates

Layer / File(s) Summary
Build artifact and executable paths
.gitignore, Makefile
Ignore rules and Makefile commands now target the updated Swift build artifacts and executable names.
Runner group and update controls
Sources/Views/MenuBarView.swift
Runner groups can be collapsed per target, and update prompts can be dismissed for a specific version.
Runner row wrapping and controls
Sources/Views/MenuBarView.swift
Runner chips wrap across rows, align to the leading edge, and use updated start/stop icon styling through the new FlowLayout.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Possibly related PRs

Suggested reviewers: peyton-spencer

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly matches the main UI redesign and the new collapsible group behavior.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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 PR with unit tests

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant