From db8178e5730129febe538efd7199ed534178f301 Mon Sep 17 00:00:00 2001
From: thePR0M3TH3AN <53631862+PR0M3TH3AN@users.noreply.github.com>
Date: Mon, 29 Sep 2025 09:47:09 -0400
Subject: [PATCH] Ensure desktop modal owns its scroll surface

---
 components/video-modal.html |  8 ++---
 css/style.css               | 62 +++++++++----------------------------
 2 files changed, 18 insertions(+), 52 deletions(-)

diff --git a/components/video-modal.html b/components/video-modal.html
index af38141e..ce969f19 100644
--- a/components/video-modal.html
+++ b/components/video-modal.html
@@ -1,7 +1,7 @@
 <!-- components/video-modal.html -->
 <div id="playerModal" class="player-modal hidden">
-  <div class="modal-container player-modal__container">
-    <div class="modal-content player-modal__content">
+  <div class="player-modal__container">
+    <div class="player-modal__content">
       <!-- Navigation bar - sliding at top -->
       <div
         id="modalNav"
@@ -119,11 +119,11 @@
         </div>
       </div>
 
-      <div class="video-container player-modal__video">
+      <div class="player-modal__video">
         <video id="modalVideo" class="w-full aspect-video" controls></video>
       </div>
 
-      <div class="video-info player-modal__info">
+      <div class="player-modal__info">
         <!-- Title and icons row -->
         <div class="flex items-center justify-between mb-2">
           <h2 id="videoTitle" class="text-2xl font-bold text-white"></h2>
diff --git a/css/style.css b/css/style.css
index 0188dfc5..11c074ee 100644
--- a/css/style.css
+++ b/css/style.css
@@ -252,7 +252,8 @@ header img {
   flex-direction: column;
   align-items: stretch;
   justify-content: stretch;
-  background-color: #000;
+  background-color: #0f172a;
+  color: #e2e8f0;
   overflow: hidden;
   overscroll-behavior: contain;
 }
@@ -269,7 +270,7 @@ header img {
   display: flex;
 }
 
-#playerModal .modal-container {
+#playerModal .player-modal__container {
   flex: 1;
   display: flex;
   flex-direction: column;
@@ -281,7 +282,7 @@ header img {
   overflow: hidden;
 }
 
-#playerModal .modal-content {
+#playerModal .player-modal__content {
   position: relative;
   flex: 1;
   display: flex;
@@ -290,32 +291,30 @@ header img {
   height: 100%;
   min-height: 0;
   max-width: none;
-  max-height: none;
-  border-radius: 0;
   background-color: #0f172a;
   overflow-x: hidden;
   overflow-y: auto;
   box-shadow: none;
   animation: fadeIn var(--modal-motion-duration) ease-out;
+  -webkit-overflow-scrolling: touch;
 }
 
-#playerModal .video-container {
+#playerModal .player-modal__video {
   width: 100%;
   background-color: #000;
-  border-radius: 0;
   position: sticky;
   top: 0;
   z-index: 51;
   flex-shrink: 0;
 }
 
-#playerModal .video-container video {
+#playerModal .player-modal__video video {
   width: 100%;
   aspect-ratio: 16 / 9;
   background-color: #000;
 }
 
-#playerModal .video-info {
+#playerModal .player-modal__info {
   flex: 0 0 auto;
   overflow: visible;
   padding: 1.5rem;
@@ -326,6 +325,10 @@ body.modal-open {
   overflow: hidden;
 }
 
+body.modal-open #app {
+  overflow: hidden;
+}
+
 /* Modal Content Container */
 .modal-content {
   width: 100%;
@@ -353,15 +356,6 @@ body.modal-open {
   }
 }
 
-/* Video Container */
-.video-container {
-  width: 100%;
-  background-color: black;
-  position: sticky;
-  top: 0;
-  z-index: 51;
-}
-
 /* Modal Video */
 #modalVideo {
   width: 100%;
@@ -369,21 +363,8 @@ body.modal-open {
   background-color: black;
 }
 
-/* Video Info Section */
-.video-info {
-  padding: 1rem;
-  flex: 1;
-  overflow-y: auto;
-}
-
 /* Responsive Adjustments */
 @media (min-width: 768px) {
-  #playerModal {
-    padding: 0;
-    align-items: stretch;
-    justify-content: stretch;
-  }
-
   .modal-content {
     max-width: 64rem;
     max-height: calc(100vh - 4rem); /* Account for padding */
@@ -391,20 +372,9 @@ body.modal-open {
     overflow: visible;
   }
 
-  #playerModal .modal-content {
+  #playerModal .player-modal__content {
     max-width: none;
-    max-height: none;
     border-radius: 0;
-    overflow-x: hidden;
-    overflow-y: auto;
-  }
-
-  .video-container {
-    position: relative;
-  }
-
-  #playerModal .video-container {
-    position: sticky;
   }
 }
 
@@ -661,14 +631,10 @@ footer a:hover {
   .modal-container {
     padding: 2rem;
   }
-
-  #playerModal .modal-container {
-    padding: 0;
-  }
 }
 
 /* Video info cards */
-.video-info .bg-gray-800\/50 {
+.player-modal__info .bg-gray-800\/50 {
   background-color: rgb(31 41 55 / 0.5);
   backdrop-filter: blur(0.25rem);
 }