fix(present): bug with Qt and Windows + debug messages

.bumpversion.cfg

@@ -1,7 +1,7 @@
 [bumpversion]
 current_version = 5.1.3
 parse = (?P<major>\d+)\.(?P<minor>\d+)\.(?P<patch>\d+)(-rc(?P<release>\d+))?
-serialize = 
+serialize =
 	{major}.{minor}.{patch}-rc{release}
 	{major}.{minor}.{patch}
 commit = True

manim_slides/present/player.py

@@ -186,6 +186,8 @@
     ):
         super().__init__()
 
+        logger.debug("Instantiating Player instance.")
+
         # Wizard's config
 
         self.config = config
@@ -206,21 +208,29 @@
         # Widgets
 
         if screen:
+            logger.debug(f"Moving window to specific {screen =}.")
             self.setScreen(screen)
             self.move(screen.geometry().topLeft())
 
         if full_screen:
+            logger.debug("Toggling window full screen.")
             self.setWindowState(Qt.WindowFullScreen)
         else:
             w, h = self.current_presentation_config.resolution
+            logger.debug(
+                f"Setting window size accordingly to first presentation resolution: {w}-by-{h}."
+            )
             geometry = self.geometry()
             geometry.setWidth(w)
             geometry.setHeight(h)
             self.setGeometry(geometry)
 
         if hide_mouse:
+            logger.debug("Hiding mouse cursor.")
             self.setCursor(Qt.BlankCursor)
 
+        logger.debug("Creating main window.")
+
         self.setWindowTitle(WINDOW_NAME)
         self.icon = QIcon(":/icon.png")
         self.setWindowIcon(self.icon)
@@ -237,6 +247,8 @@
         self.presentation_changed.connect(self.presentation_changed_callback)
         self.slide_changed.connect(self.slide_changed_callback)
 
+        logger.debug("Creating (secondary) info window.")
+
         self.info = Info(
             full_screen=full_screen, aspect_ratio_mode=aspect_ratio_mode, screen=screen
         )
@@ -249,6 +261,8 @@
 
         # Connecting key callbacks
 
+        logger.debug("Attaching callbacks")
+
         self.config.keys.QUIT.connect(self.close)
         self.config.keys.PLAY_PAUSE.connect(self.play_pause)
         self.config.keys.NEXT.connect(self.next)
@@ -268,6 +282,7 @@
         # Setting-up everything
 
         if skip_all:
+            logger.debug("All slides will be skipped after being played.")
 
             def media_status_changed(status: QMediaPlayer.MediaStatus) -> None:
                 self.media_player.setLoops(1)  # Otherwise looping slides never end
@@ -277,6 +292,9 @@
             self.media_player.mediaStatusChanged.connect(media_status_changed)
 
         else:
+            logger.debug(
+                "Adding a custom signal handler to skip slide if `--auto-next` is used."
+            )
 
             def media_status_changed(status: QMediaPlayer.MediaStatus) -> None:
                 if (
@@ -288,6 +306,7 @@
             self.media_player.mediaStatusChanged.connect(media_status_changed)
 
         if self.current_slide_config.loop:
+            logger.debug("First slide is a loop.")
             self.media_player.setLoops(-1)
 
         self.load_current_media(start_paused=start_paused)
@@ -391,6 +410,7 @@
 
     def load_current_media(self, start_paused: bool = False) -> None:
         url = QUrl.fromLocalFile(str(self.current_file))
+        logger.debug(f"Loading media from {url = }.")
         self.media_player.setSource(url)
 
         if self.playing_reversed_slide:
@@ -403,11 +423,14 @@
             )
 
         if start_paused:
+            logger.debug("Media is not playing, starting paused.")
             self.media_player.pause()
         else:
+            logger.debug("Playing the media...")
             self.media_player.play()
 
     def load_current_slide(self) -> None:
+        logger.debug("Loading (current) slide.")
         slide_config = self.current_slide_config
         self.current_file = slide_config.file
 
@@ -419,6 +442,7 @@
         self.load_current_media()
 
     def load_previous_slide(self) -> None:
+        logger.debug("Loading previous slide.")
         self.playing_reversed_slide = False
 
         if self.current_slide_index > 0:
@@ -433,6 +457,7 @@
         self.load_current_slide()
 
     def load_next_slide(self) -> None:
+        logger.debug("Loading next slide.")
         if self.playing_reversed_slide:
             self.playing_reversed_slide = False
             self.preview_next_slide()  # Slide number did not change, but next did
@@ -451,6 +476,7 @@
         self.load_current_slide()
 
     def load_reversed_slide(self) -> None:
+        logger.debug("Loading reversed slide.")
         self.playing_reversed_slide = True
         self.current_file = self.current_slide_config.rev_file
         self.load_current_media()
@@ -461,19 +487,22 @@
 
     @Slot()
     def presentation_changed_callback(self) -> None:
+        logger.debug("Signal that presentation changed.")
         index = self.current_presentation_index
         count = self.presentations_count
         self.info.scene_label.setText(f"{index+1:4d}/{count:4<d}")
 
     @Slot()
     def slide_changed_callback(self) -> None:
+        logger.debug("Signal that slide changed.")
         index = self.current_slide_index
         count = self.current_slides_count
         self.info.slide_label.setText(f"{index+1:4d}/{count:4<d}")
         self.info.slide_notes.setText(self.current_slide_config.notes)
         self.preview_next_slide()
 
     def preview_next_slide(self) -> None:
+        logger.debug("Previewing next slide (if any).")
         if slide_config := self.next_slide_config:
             url = QUrl.fromLocalFile(str(slide_config.file))
             self.info.next_media_player.setSource(url)
@@ -493,6 +522,7 @@
 
     @Slot()
     def next(self) -> None:
+        logger.debug("[USER] Calling next slide.")
         if self.media_player.playbackState() == QMediaPlayer.PlaybackState.PausedState:
             self.media_player.play()
         elif self.next_terminates_loop and self.media_player.loops() != 1:
@@ -506,44 +536,55 @@
 
     @Slot()
     def previous(self) -> None:
+        logger.debug("[USER] Calling previous slide.")
         self.load_previous_slide()
 
     @Slot()
     def reverse(self) -> None:
+        logger.debug("[USER] Reversing current slide.")
         self.load_reversed_slide()
         self.preview_next_slide()
 
     @Slot()
     def replay(self) -> None:
+        logger.debug("[USER] Starting the current slide from the beginning.")
         self.media_player.setPosition(0)
         self.media_player.play()
 
     @Slot()
     def play_pause(self) -> None:
         state = self.media_player.playbackState()
         if state == QMediaPlayer.PlaybackState.PausedState:
+            logger.debug("[USER] Playing the slide.")
             self.media_player.play()
         elif state == QMediaPlayer.PlaybackState.PlayingState:
+            logger.debug("[USER] Pausing the slide.")
             self.media_player.pause()
 
     @Slot()
     def full_screen(self) -> None:
         if self.windowState() == Qt.WindowFullScreen:
+            logger.debug("[USER] Disabling full screen.")
             self.setWindowState(Qt.WindowNoState)
         else:
+            logger.debug("[USER] Toggling full screen.")
             self.setWindowState(Qt.WindowFullScreen)
 
     @Slot()
     def hide_mouse(self) -> None:
         if self.cursor().shape() == Qt.BlankCursor:
+            logger.debug("[USER] Showing the mouse cursor.")
             self.setCursor(Qt.ArrowCursor)
         else:
+            logger.debug("[USER] Hiding the mouse cursor.")
             self.setCursor(Qt.BlankCursor)
 
     def closeEvent(self, event: QCloseEvent) -> None:  # noqa: N802
+        logger.debug("[USER] Close event.")
         self.close()
 
     def keyPressEvent(self, event: QKeyEvent) -> None:  # noqa: N802
+        logger.debug(f"[USER] Key press event {event}.")
         key = event.key()
         self.dispatch(key)
         event.accept()