fix(ui): restore Alerts sidebar row selection (Issue #34) #38

Merged
gronod merged 1 commits from fix/34-alerts-sidebar-badge into milestone/m6-bugfix-stability 2026-09-08 15:18:26 +01:00
Owner

Summary

.badge(Text?) applied after .tag("Alerts") on the List(selection:) row wrapped the tagged label in a view that no longer carried the tag — the row rendered (with badge "1") but was not selectable.

Fix

Replaced with an explicit Label { HStack } row: the count renders as a styled capsule inside the label, and .tag("Alerts") stays on the row itself. Show/hide behavior preserved (capsule only when alerts exist).

Verified

  • Build green for x86_64; swiftlint — 0 violations.

Fixes #34.

## Summary `.badge(Text?)` applied after `.tag("Alerts")` on the `List(selection:)` row wrapped the tagged label in a view that no longer carried the tag — the row rendered (with badge "1") but was not selectable. ### Fix Replaced with an explicit `Label { HStack }` row: the count renders as a styled capsule inside the label, and `.tag("Alerts")` stays on the row itself. Show/hide behavior preserved (capsule only when alerts exist). ### Verified - Build green for x86_64; `swiftlint` — 0 violations. Fixes #34.
gronod added 1 commit 2026-09-08 15:18:17 +01:00
Applying .badge after .tag on a List(selection:) row wrapped the tagged
label in a view that no longer carried the tag, making the "Alerts &
Notifications" row unselectable on macOS 14. Replaced with an explicit
Label + HStack row showing the active-alert count as a styled capsule —
tag stays on the row, count still renders when alerts are active.

Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
gronod added the Bug/UIKind/Bug
Priority
Medium
3
Project/Antigravity
labels 2026-09-08 15:18:24 +01:00
gronod merged commit d75873fa30 into milestone/m6-bugfix-stability 2026-09-08 15:18:26 +01:00
Sign in to join this conversation.
No Reviewers
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: gronod/MacMonitor#38