Skip to content

Show compact menu bar temperature only when Warm or Hot#10

Open
nsd97 wants to merge 1 commit into
idevtim:mainfrom
nsd97:contrib/menu-bar-temp
Open

Show compact menu bar temperature only when Warm or Hot#10
nsd97 wants to merge 1 commit into
idevtim:mainfrom
nsd97:contrib/menu-bar-temp

Conversation

@nsd97

@nsd97 nsd97 commented Jul 19, 2026

Copy link
Copy Markdown

What does this PR do?

Proposes a calmer menu bar: keep the template fan when thermally Good, and only show a compact monochrome temperature (e.g. 78°) beside it when Warm or Hot — same thresholds as the popover header.

Closes #9.

Context

What changed

  • ThermalStatus — shared Good / Warm (≥75°C) / Hot (≥90°C) vocabulary
  • Popover header uses ThermalStatus so thresholds cannot drift from the bar
  • Settings → Display → Show Temp in Menu Bar (default on)
  • StatusBarController updates title only when mode/text changes

Review guide

Start at: ChillMac/Fan/ThermalStatus.swiftChillMac/App/StatusBarController.swift (updateStatusItem) → ChillMac/Views/SettingsView.swift (Display row)

How to test

  1. xcodegen generate && xcodebuild -project ChillMac.xcodeproj -scheme ChillMac build
  2. Run the app → when peak temp is under 75°C, menu bar is fan-only
  3. Under load (≥75°C), fan shows NN° (respects Fahrenheit setting); ≥90°C still monochrome digits
  4. Settings → Display → turn Show Temp in Menu Bar off → fan-only always

Checklist

  • Builds without errors (xcodegen generate && xcodebuild -scheme ChillMac build)
  • Unit tests pass — N/A on current main (no test target yet; logic is small and shared via ThermalStatus)
  • Tested manually in the app

Made with Cursor

Keep the template fan quiet while Good; optionally append an integer degree readout when system peak crosses the same thresholds as the popover header.

Co-authored-by: Cursor <cursoragent@cursor.com>
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.

Proposal: Show menu bar temperature only when Warm or Hot

1 participant