Skip to content

feat: toggle to hide the ticking menu-bar clock#12

Open
Wegi wants to merge 1 commit into
feat/issue-6-stable-clockfrom
feat/issue-3-hide-menubar-clock
Open

feat: toggle to hide the ticking menu-bar clock#12
Wegi wants to merge 1 commit into
feat/issue-6-stable-clockfrom
feat/issue-3-hide-menubar-clock

Conversation

@Wegi

@Wegi Wegi commented Jul 11, 2026

Copy link
Copy Markdown
Contributor

Closes #3

What

New setting "Hide timer in menu bar" so clients in a screen share don't see a ticking clock. While tracking with the clock hidden, the menu bar shows a static record.circle glyph instead — you still see that tracking is active, just not the time. The popover is unaffected.

How

  • New UserDefaults key SessionStore.hideMenuBarClockDefaultsKey, following the exact remindWhenIdle pattern.
  • MenuBarLabel reads it via @AppStorage and branches: running+hidden → record.circle, running → ticking clock, idle → timer glyph (unchanged).
  • Checkbox added to the settings block in the popover.

Tests

The key constant lives in ConanCore; the branching itself is a trivial SwiftUI view (app target, not unit-testable). Full suite still green (35 tests). Verified by building; visual check: toggle on → glyph while running, toggle off → clock returns.

Stack note: stacked on #11 (base branch feat/issue-6-stable-clock); merge order #10#11 → this.

Adds a "Hide timer in menu bar" setting (UserDefaults-backed, same
pattern as the idle reminder). While running with the clock hidden the
menu bar shows a static record.circle glyph, so tracking is still
visible without a ticking clock — e.g. during customer screen shares.
@Wegi Wegi force-pushed the feat/issue-3-hide-menubar-clock branch from b5d5042 to e85fb19 Compare July 11, 2026 08:51
@Wegi Wegi force-pushed the feat/issue-6-stable-clock branch from c8e8dc7 to 281e2af Compare July 11, 2026 08:51
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