From 454222b91a135b081db190f0a325df3f5471d9e2 Mon Sep 17 00:00:00 2001 From: David Perl Date: Wed, 12 Aug 2026 15:40:07 +0200 Subject: [PATCH] fix: show viewing mode for all states --- src/aare/gui/widgets/busy_overlay.py | 20 ++++++-------------- 1 file changed, 6 insertions(+), 14 deletions(-) diff --git a/src/aare/gui/widgets/busy_overlay.py b/src/aare/gui/widgets/busy_overlay.py index 6013a7b2..69ebe94c 100644 --- a/src/aare/gui/widgets/busy_overlay.py +++ b/src/aare/gui/widgets/busy_overlay.py @@ -108,7 +108,11 @@ def build_busy_overlay_style( tell_state: TellStateModel | None, session_state: SessionsStateEnum | None = None, ) -> BusyOverlayStyle | None: - if session_state == SessionsStateEnum.Vacant: + if session_state == { + SessionsStateEnum.OwnedByElse, + SessionsStateEnum.PendingYouToElse, + SessionsStateEnum.Vacant, + }: return BusyOverlayStyle( text="In viewing mode", badge_bg=BUSY_YELLOW, @@ -117,20 +121,8 @@ def build_busy_overlay_style( overlay_border=qcolor(BUSY_YELLOW_BORDER, 235), overlay_text=qcolor(WHITE), accent_dot=BUSY_YELLOW_DOT, - subtext="Click here to grab baton if need to interact with GUI", + subtext="Click here to grab the baton if you need to interact with GUI", ) - - if session_state in {SessionsStateEnum.OwnedByElse, SessionsStateEnum.PendingYouToElse}: - return BusyOverlayStyle( - text="GUEST MODE", - badge_bg=BUSY_PURPLE, - badge_fg=WHITE, - overlay_fill=qcolor(BUSY_PURPLE, 190), - overlay_border=qcolor(BUSY_PURPLE_BORDER, 230), - overlay_text=qcolor(WHITE), - accent_dot=BUSY_PURPLE_DOT, - ) - if not is_busy: return None