Update postergeist.py to 1.4

# Postergeist Slideshow - Changelog

All notable changes to this project will be documented in this file.

---

## v2.0.0 - 2025-09-26

This is a significant update that introduces a major new feature for themed overlays and fixes a critical bug related to video playback.

### ✨ New Features

* **Themed Overlay Tagging:**
    * You can now assign specific overlays to entire folders of media.
    * **How it works:** Rename a subfolder inside your `posters` directory using the format `FolderName_tag`. For example, a folder named `MyMovies_vhs` will use the tag `vhs`.
    * The script will automatically look for an overlay file in the `overlays` folder with a filename that matches the tag (e.g., `vhs.png`, `vhs.gif`).
    * If no matching overlay is found for a tag, or if a folder has no tag, a random overlay will be used as a fallback.

* **Subdirectory Scanning:**
    * The script now automatically scans for media files in all subfolders within the main `posters` directory. This allows for better organization of your media library and is essential for the new tagging feature.

### 🐞 Bug Fixes

* **Video Transition Stall:**
    * Fixed a critical bug where the slideshow would not advance to the next item after a video file finished playing.
    * **Cause:** The `root.after()` scheduler was being given a reference to the `get_delay` method instead of the *result* of calling the method.
    * **Solution:** The call was corrected from `` `self.root.after(self.get_delay, ...)` `` to `` `self.root.after(self.get_delay(), ...)` ``, ensuring the scheduler receives a valid millisecond value.

### 🔧 Code & Logic Improvements

* **`load_files()` Function:**
    * Rewritten to use `os.walk()` to recursively search through the entire directory tree of the specified `posters` folder.

* **`_select_new_overlay()` Method:**
    * Refactored to accept the `media_path` of the current file as an argument. It now parses the parent directory name to check for a `_tag`.

* **`show_image()` and `show_video()` Methods:**
    * Updated to pass the current file's path to the `_select_new_overlay()` method, enabling the new tag-based logic.

Author: pasiegel

postergeist.py

@@ -22,7 +22,6 @@ def __init__(self, root, folder, overlay_folder, delay, random_delay, start_rota
         self.delay = delay
         self.random_delay = random_delay
         self.fade_height_percent = fade_height
-        # --- NEW ---
         self.performance_mode = performance_mode
 
         self.files = self.load_files(folder)
@@ -73,8 +72,13 @@ def __init__(self, root, folder, overlay_folder, delay, random_delay, start_rota
 
     def load_files(self, folder):
         exts = (".jpg", ".jpeg", ".png", ".webp", ".bmp", ".gif", ".mp4", ".mov", ".avi", ".mkv")
+        file_list = []
         try:
-            return [os.path.join(folder, f) for f in os.listdir(folder) if f.lower().endswith(exts)]
+            for root, _, files in os.walk(folder):
+                for f in files:
+                    if f.lower().endswith(exts):
+                        file_list.append(os.path.join(root, f))
+            return file_list
         except FileNotFoundError:
             return []
 
@@ -121,7 +125,6 @@ def prepare_base_frame(self, img_to_process, is_video=False):
             self.suppress_overlay = True
         else:
             self.suppress_overlay = False
-            # --- MODIFIED: Glow effect is skipped in performance mode ---
             if not self.performance_mode:
                 glow = poster.filter(ImageFilter.GaussianBlur(25))
                 glow_x = (canvas_w - p_w) // 2
@@ -188,24 +191,38 @@ def do_step(step=0):
 
         do_step()
 
-    def _select_new_overlay(self):
+    def _select_new_overlay(self, media_path):
         self.active_overlay_path = None
         self.active_overlay_frames = []
         self.active_overlay_index = 0
-        if self.overlays:
+        tagged_overlay_found = False
+
+        folder_name = os.path.basename(os.path.dirname(media_path))
+
+        if '_' in folder_name:
+            tag = folder_name.rsplit('_', 1)[-1]
+            for overlay_file_path in self.overlays:
+                overlay_filename_no_ext = os.path.splitext(os.path.basename(overlay_file_path))[0]
+                if overlay_filename_no_ext == tag:
+                    self.active_overlay_path = overlay_file_path
+                    tagged_overlay_found = True
+                    break
+
+        if not tagged_overlay_found and self.overlays:
             self.active_overlay_path = random.choice(self.overlays)
-            if self.active_overlay_path.lower().endswith((".gif", ".apng")):
-                try:
-                    with Image.open(self.active_overlay_path) as img:
-                        self.active_overlay_duration = img.info.get('duration', 100)
-                        self.active_overlay_frames = [frame.copy() for frame in ImageSequence.Iterator(img)]
-                except Exception as e:
-                    print(f"Error loading animated overlay {os.path.basename(self.active_overlay_path)}: {e}")
-                    self.active_overlay_path = None
+
+        if self.active_overlay_path and self.active_overlay_path.lower().endswith((".gif", ".apng")):
+            try:
+                with Image.open(self.active_overlay_path) as img:
+                    self.active_overlay_duration = img.info.get('duration', 100)
+                    self.active_overlay_frames = [frame.copy() for frame in ImageSequence.Iterator(img)]
+            except Exception as e:
+                print(f"Error loading animated overlay {os.path.basename(self.active_overlay_path)}: {e}")
+                self.active_overlay_path = None
 
     def show_image(self, path):
         self.cached_video_background = None
-        self._select_new_overlay()
+        self._select_new_overlay(path)
         try:
             with Image.open(path) as img:
                 img_rgba = img.convert("RGBA")
@@ -233,7 +250,7 @@ def show_video(self, path):
         screen_w = self.root.winfo_width()
         screen_h = self.root.winfo_height()
         if vid_w < screen_w * 0.95 or vid_h < screen_h * 0.95:
-            self._select_new_overlay()
+            self._select_new_overlay(path)
         else:
             self.active_overlay_path = None
             self.active_overlay_frames = []
@@ -318,7 +335,6 @@ def rotate_poster(self):
         if self.video_capture and self.video_capture.isOpened():
             self.show_file()
         else:
-            # Need to reload the static image to redraw it
             if self.files:
                 self.show_image(self.files[self.index])
 
@@ -357,7 +373,6 @@ def main():
     parser.add_argument("--rotate", type=int, default=0, choices=[0, 90, 180, 270], help="Starting rotation")
     parser.add_argument("--fade-height", type=int, default=20,
                         help="Fade height at bottom of poster as a percentage (e.g., 20)")
-    # --- NEW ARGUMENT ---
     parser.add_argument("--performance-mode", action="store_true",
                         help="Disable intensive effects like glow for better performance")
     args = parser.parse_args()