From 7dcbd96b13d02d6360a4a51cf949160570a1591d Mon Sep 17 00:00:00 2001
From: Tim Evans <timevans666@gmail.com>
Date: Thu, 16 Jul 2026 22:39:52 +0100
Subject: [PATCH] =?UTF-8?q?Fix=20pause=20overlay=20showing=20GAME=20PAUSED?=
 =?UTF-8?q?=20while=20unpaused=20=E2=80=94=20v2.0.5?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Root cause of the toggle cycle bug: the theme set display:flex on
#pause.dg2-pause unconditionally, out-specificity-ing core's hidden
(unpaused) state. After unpausing, the pause element stayed visible and
showed Foundry's default GAME PAUSED markup (the overlay injector
correctly skips unpaused renders). Scope the flex layout to
#pause.dg2-pause.paused and explicitly hide :not(.paused).

Verified three full pause/unpause cycles: paused = radar scope visible,
unpaused = element hidden. (Earlier tests misread visibility via
offsetParent, which is always null for position:fixed elements.)

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
---
 css/dg2-tactical.css | 9 +++++++++
 system.json          | 2 +-
 2 files changed, 10 insertions(+), 1 deletion(-)

diff --git a/css/dg2-tactical.css b/css/dg2-tactical.css
index e6a246b..d880dce 100644
--- a/css/dg2-tactical.css
+++ b/css/dg2-tactical.css
@@ -539,12 +539,21 @@
 
 #pause.dg2-pause {
   background: none;
+}
+
+/* Layout ONLY while actually paused - never override Foundry's hidden
+   (unpaused) state, or the stock GAME PAUSED markup shows after unpausing. */
+#pause.dg2-pause.paused {
   display: flex;
   flex-direction: column;
   align-items: center;
   gap: 14px;
 }
 
+#pause.dg2-pause:not(.paused) {
+  display: none;
+}
+
 #pause.dg2-pause img { display: none; }
 
 .dg2-pause-scope {
diff --git a/system.json b/system.json
index 82a8f20..4288289 100644
--- a/system.json
+++ b/system.json
@@ -15,7 +15,7 @@
             "discord": "jalensailin"
         }
     ],
-    "version": "2.0.4",
+    "version": "2.0.5",
     "compatibility": {
         "minimum": "13",
         "verified": "14",