diff options
| author | Danilo M. <danix@danix.xyz> | 2026-09-15 12:29:42 +0200 |
|---|---|---|
| committer | Danilo M. <danix@danix.xyz> | 2026-09-15 12:29:42 +0200 |
| commit | 661d477e7806d322d3499c19537541c5dd4e7d82 (patch) | |
| tree | b23199038db02f9bdbc9c4d91eb467b8d35b7e8f /docs/superpowers/specs/2026-09-15-status-registry-design.md | |
| parent | fceeb5dc23c5e5a951f725e7682eab6814191c47 (diff) | |
| download | quickshell-661d477e7806d322d3499c19537541c5dd4e7d82.tar.gz quickshell-661d477e7806d322d3499c19537541c5dd4e7d82.zip | |
fix(status): address final review findings
F1: after a click the shared Switch writes checked directly, dropping
StatusRow's declarative binding, so an external mode change from waybar
or statusctl no longer moved the switch. Resync via onValueChanged.
F2: the spec claimed hyprctl clients counts idle inhibitors, which is
false for a layer-surface inhibitor; replace with the behavioral
hypridle marker check.
F3: README said the check covers both watch states; it exercises the
activated report and the absent-file down report.
F4: give breakProc an onExited check so a failed breaktimer.sh verb is
visible instead of silent.
F5: drop StatusTile's unused required st property and its injection.
F6: comment the startup-order limit on dndBeforePresentation.
Diffstat (limited to 'docs/superpowers/specs/2026-09-15-status-registry-design.md')
| -rw-r--r-- | docs/superpowers/specs/2026-09-15-status-registry-design.md | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/docs/superpowers/specs/2026-09-15-status-registry-design.md b/docs/superpowers/specs/2026-09-15-status-registry-design.md index 34e5945..db1e79c 100644 --- a/docs/superpowers/specs/2026-09-15-status-registry-design.md +++ b/docs/superpowers/specs/2026-09-15-status-registry-design.md @@ -222,8 +222,10 @@ That covers the parse, the write, the atomic rename and the watch together, and it fails if any of them break. The effects need the shell and are confirmed by hand, once, because each is -observable: `hyprctl clients` counts inhibitors, `breaktimer.sh status` reports -`paused`, and the notification daemon's own behaviour under DND is the second -spec's verification. +observable: a throwaway `hypridle -c` instance with a short timeout writes a +marker on `on-timeout`, and the marker must not appear while presentation is on +and must appear within a few seconds of it being turned off; `breaktimer.sh +status` reports `paused`; and the notification daemon's own behaviour under DND +is the second spec's verification. Per AGENTS.md, anything visual is confirmed by the user rather than screenshotted. |