From 661d477e7806d322d3499c19537541c5dd4e7d82 Mon Sep 17 00:00:00 2001 From: "Danilo M." Date: Tue, 15 Sep 2026 12:29:42 +0200 Subject: 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. --- desktop/modules/status/StatusTile.qml | 4 ---- 1 file changed, 4 deletions(-) (limited to 'desktop/modules/status/StatusTile.qml') diff --git a/desktop/modules/status/StatusTile.qml b/desktop/modules/status/StatusTile.qml index 9bc6f04..c4b87d4 100644 --- a/desktop/modules/status/StatusTile.qml +++ b/desktop/modules/status/StatusTile.qml @@ -12,11 +12,7 @@ import QtQuick import "../.." -// Injected as st, never mod: a property named the same as the enclosing -// object's id binds to itself and arrives undefined. See AGENTS.md. Text { - required property var st - width: parent ? parent.width : implicitWidth elide: Text.ElideRight font { family: Theme.fontFamily; pixelSize: Theme.fontSize - 4 } -- cgit v1.2.3