From 24b1a3da5fa1b295d48edafdff87a547c1a5e464 Mon Sep 17 00:00:00 2001
From: mycroftw <30244806+mycroftw@users.noreply.github.com>
Date: Wed, 2 Oct 2024 22:22:06 -0600
Subject: [PATCH] Add: Double Break warning, "End Break" label

1. If "Go to Break" is pressed, and a break is already scheduled for
   this round (or time has already been added, manually or in settings,
   with "invisible breaks"), a warning pops up saying so.  For 1/2 a
   second, so it doesn't stick around for ever.
2. During a visible break, the "End Round" button changes to
   "End Break".
3. Fixed sizing of "Round #" so that visible break text doesn't
   overflow.
---
 bridge_clock_main.py | 19 ++++++++++++++-----
 1 file changed, 14 insertions(+), 5 deletions(-)

diff --git a/bridge_clock_main.py b/bridge_clock_main.py
index fb23151..1f7b463 100644
--- a/bridge_clock_main.py
+++ b/bridge_clock_main.py
@@ -294,6 +294,7 @@ def _next_round(self) -> None:
             else:
                 self.round += 1
                 self._in_break = False
+                self.button_end_round.SetLabel("End Round")
                 self._update_round()
                 self._reset_clock()
                 if not self.settings.break_visible and self._break_this_round():
@@ -312,10 +313,11 @@ def _go_to_break(self) -> None:
         self._in_break = True
         self.label_clock.SetBackgroundColour(BREAK_COLOUR)
         self.label_round.SetBackgroundColour(BREAK_COLOUR)
-        self.label_round.SetLabelText("TIME TO NEXT ROUND:")
+        self.label_round.SetLabelText("NEXT ROUND IN:")
         self.round_end = wx.DateTime.Now() + wx.TimeSpan.Minutes(
             self.settings.break_length
         )
+        self.button_end_round.SetLabel("End Break")
         self._update_clock()
 
     def _game_over(self) -> None:
@@ -445,10 +447,7 @@ def on_resize(self, event) -> None:
         """Hack to resolve sizer not auto-sizing with panel on maximize/unmaximize."""
         self.Layout()
         self.sizer_1.SetDimension(self.panel_1.GetPosition(), self.panel_1.GetSize())
-        self._handle_resize(
-            self.label_round,
-            "ROUND 8" if self.settings.rounds < 10 else "ROUND 88",
-        )
+        self._handle_resize(self.label_round, "NEXT ROUND IN:")
         self._handle_resize(
             self.label_clock,
             "88:88" if self.settings.round_length < 100 else "888:88",
@@ -523,6 +522,16 @@ def on_goto_break(self, event):
             if not self.settings.break_visible:  # treat as "add break minutes to clock"
                 self.round_end.Add(wx.TimeSpan.Minutes(self.settings.break_length))
                 self._update_clock()
+        else:
+            dlg = wx.GenericProgressDialog(
+                "",
+                "Break already scheduled for this round.",
+                style=wx.PD_AUTO_HIDE,
+            )
+            dlg.SetPosition(  # show above "go to break" button)
+                self.button_break.GetScreenPosition() - (0, dlg.GetSize().GetHeight())
+            )
+            wx.CallLater(500, dlg.Update, 100)
         event.Skip()
 
     def on_button_end_round(self, event) -> None: