feat: add PACE display option in menu bar #169
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add a new setting to toggle between displaying percent, pace, or both in the menu bar when using brand icon mode. - Add MenuBarDisplayMode enum with three options: percent, pace, both - Add menuBarDisplayMode setting to SettingsStore - Implement menuBarPaceText() for generating pace delta strings - Add menuBarDisplayText() that combines percent/pace based on mode - Add segmented picker in Advanced preferences pane The pace indicator shows how your usage compares to expected usage (e.g., "+5%" means 5% ahead of expected pace). Falls back to percent display for providers that don't support pace calculation or when pace data isn't available. Closes: steipete#168
steipete
force-pushed
the
feat/menu-bar-pace-display
branch
from
32b59de to
4b51106
Compare
Owner
|
Landed via temp rebase onto main.\n\n- Gate: pnpm lint && pnpm build && pnpm test\n- Land commit: 4b51106\n- Merge commit: 59a83d5\n\nThanks @PhilETaylor! |
Owner
|
Thanks Phil! Don't worry... I see PRs more as prompt requests. I cleaned up :) |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Warning. This is 100% Claude generated to scratch my own itch - please dont blindly merge it without checking it - im a PHP dev, not a swift dev!
Also happy if you choose to reject this feature request ... no worries. Scratching my own itch.
Summary
Adds a new setting to display PACE information in the menu bar, addressing the feature request in #168.
When "Menu bar shows percent" is enabled, users can now choose between three display modes:
45%) - default behavior+14%)45% +14%)Changes
MenuBarDisplayModeenum with three options:percent,pace,bothmenuBarDisplayModesetting toSettingsStorewith UserDefaults persistencemenuBarPaceText()for generating pace delta strings from the weekly windowmenuBarDisplayText()that combines percent/pace based on selected modeHow it works
The pace indicator shows how your usage compares to expected usage based on time elapsed in the weekly quota window:
+14%means you're 14% ahead of expected pace (using faster than expected)-5%means you're 5% behind expected pace (using slower than expected)This matches the existing "Pace: Ahead (+14%)" display in the dropdown menu.
Screenshots
The display mode picker appears in Settings → Advanced, below the "Menu bar shows percent" toggle.
Test plan
Closes #168
🤖 Generated with Claude Code