fix(overlay): honor mpv subtitle binding config and tidy modal close

src/core/services/mpv-protocol.test.ts

@@ -135,6 +135,20 @@ test('dispatchMpvProtocolMessage enforces sub-visibility hidden when overlay sup
   });
 });
 
+test('dispatchMpvProtocolMessage skips sub-visibility suppression when overlay binding is disabled', async () => {
+  const { deps, state } = createDeps({
+    shouldBindVisibleOverlayToMpvSubVisibility: () => false,
+    isVisibleOverlayVisible: () => true,
+  });
+
+  await dispatchMpvProtocolMessage(
+    { event: 'property-change', name: 'sub-visibility', data: 'yes' },
+    deps,
+  );
+
+  assert.equal(state.commands.length, 0);
+});
+
 test('dispatchMpvProtocolMessage sets secondary subtitle track based on track list response', async () => {
   const { deps, state } = createDeps();
 

src/core/services/mpv-protocol.ts

@@ -220,7 +220,8 @@ export async function dispatchMpvProtocolMessage(
     } else if (msg.name === 'sub-visibility') {
       if (
         deps.isVisibleOverlayVisible() &&
-        asBoolean(msg.data, false)
+        asBoolean(msg.data, false) &&
+        (deps.shouldBindVisibleOverlayToMpvSubVisibility?.() ?? true)
       ) {
         deps.sendCommand({ command: ['set_property', 'sub-visibility', 'no'] });
       }

src/main.ts

@@ -758,7 +758,10 @@ const restoreOverlayMpvSubtitles = createRestoreOverlayMpvSubtitlesHandler({
 });
 
 function shouldSuppressMpvSubtitlesForOverlay(): boolean {
-  return overlayManager.getVisibleOverlayVisible();
+  return (
+    overlayManager.getVisibleOverlayVisible() &&
+    configDerivedRuntime.shouldBindVisibleOverlayToMpvSubVisibility()
+  );
 }
 
 function syncOverlayMpvSubtitleSuppression(): void {

src/main/overlay-runtime.ts

@@ -232,15 +232,13 @@ export function createOverlayModalRuntimeService(
   const handleOverlayModalClosed = (modal: OverlayHostedModal): void => {
     if (!restoreVisibleOverlayOnModalClose.has(modal)) return;
     restoreVisibleOverlayOnModalClose.delete(modal);
+    const modalWindow = deps.getModalWindow();
     if (restoreVisibleOverlayOnModalClose.size === 0) {
       clearPendingModalWindowReveal();
       notifyModalStateChange(false);
-    }
+      if (modalWindow && !modalWindow.isDestroyed()) {
-
+        modalWindow.hide();
-    const modalWindow = deps.getModalWindow();
+      }
-    if (!modalWindow || modalWindow.isDestroyed()) return;
-    if (restoreVisibleOverlayOnModalClose.size === 0) {
-      modalWindow.hide();
     }
   };