Skip to content

Rework menu bar: word count in icon, Report button in header, Check for Updates in footer#199

Merged
FuJacob merged 1 commit into
mainfrom
remove-word-counter
May 24, 2026
Merged

Rework menu bar: word count in icon, Report button in header, Check for Updates in footer#199
FuJacob merged 1 commit into
mainfrom
remove-word-counter

Conversation

@FuJacob
Copy link
Copy Markdown
Owner

@FuJacob FuJacob commented May 24, 2026

Summary

  • Menu bar icon: Surface the running total of accepted (tabbed) words as a compact label next to the paw-print icon. Uses abbreviated formatting (1.2k, 3.5M) and only appears after at least one word is accepted.
  • Menu header: Replace the "N words accepted" counter with the "Report Bug / Feedback" button for quicker access.
  • Menu footer: Move "Check for Updates" to the footer where Report used to be.

Validation

xcodebuild -project tabby.xcodeproj -scheme tabby -destination 'platform=macOS' build
# ** BUILD SUCCEEDED **

xcodebuild -project tabby.xcodeproj -scheme tabby -destination 'platform=macOS' build-for-testing
# ** TEST BUILD SUCCEEDED **

swiftlint lint --quiet
# No new warnings (only pre-existing ones in unrelated files)

Linked issues

None.

Risk / rollout notes

  • MenuBarStatusLabelView now observes SuggestionCoordinator to display the word count. The label re-renders only when the count changes.
  • MenuBarView no longer depends on SuggestionCoordinator; it takes AppUpdateManager instead.

…ates to footer

The "N words accepted" counter in the menu bar header provided little
actionable value. Replace it with the "Report Bug / Feedback" button
(previously in the footer) for quicker access, and put "Check for Updates"
in the footer where Report used to be.
@FuJacob FuJacob merged commit 4b22d9b into main May 24, 2026
6 checks passed
@FuJacob FuJacob changed the title Replace word counter with Report button, move Check for Updates to footer Rework menu bar: word count in icon, Report button in header, Check for Updates in footer May 24, 2026
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