Fix scroll behavior and improve overflow handling in router transitions

Author: ralyodio

public/css/theme.css

@@ -140,15 +140,15 @@ body {
 #app {
   min-height: 100vh;
   position: relative;
-  overflow: hidden;
+  overflow: visible;
   width: 100%;
-  height: 100%;
+  height: auto;
   display: block;
 }
 
-/* Ensure the app container takes full height */
-html, body, #app {
-  height: 100%;
+/* Ensure the app container takes full height but allows scrolling */
+html, body {
+  min-height: 100%;
   margin: 0;
   padding: 0;
 }

public/icons/favicon.dark.ico

@@ -145,8 +145,8 @@ class Router {
       // Add the overlay to the body
       document.body.appendChild(transitionOverlay);
 
-      // Prevent scrolling during transition
-      document.body.style.overflow = 'hidden';
+      // We no longer need to prevent scrolling during transition
+      // document.body.style.overflow = 'hidden';
 
       // Fade in the overlay
       setTimeout(() => {
@@ -217,6 +217,7 @@ class Router {
         // Remove the overlay after the transition completes
         setTimeout(() => {
           document.body.removeChild(transitionOverlay);
+          // Ensure overflow is always reset
           document.body.style.overflow = '';
         }, 150); // Match the transition duration
       } else {
@@ -232,6 +233,7 @@ class Router {
         // Remove the overlay after the transition completes
         setTimeout(() => {
           document.body.removeChild(transitionOverlay);
+          // Ensure overflow is always reset
           document.body.style.overflow = '';
         }, 150); // Match the transition duration
       }
@@ -244,8 +246,10 @@ class Router {
       // Remove the overlay
       if (document.querySelector('.transition-overlay')) {
         document.body.removeChild(document.querySelector('.transition-overlay'));
-        document.body.style.overflow = '';
       }
+      
+      // Always ensure overflow is reset, even in error cases
+      document.body.style.overflow = '';
     } finally {
       // Clear loading state
       this.loading = false;