pax/booru-viewer
1
0
Fork 0
Code Issues Releases 18 Activity

Loop/Next toggle takes effect immediately during playback

Browse Source 
Use manual looping via EndOfMedia instead of QMediaPlayer.Loops.Infinite
so toggling mid-playback works without waiting for next video.
This commit is contained in:
pax 2026-04-05 03:19:22 -05:00
parent 192397f1ec
commit 68e04776b1
1 changed files with 7 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
booru_viewer/gui/preview.py
View File

@ -423,9 +423,7 @@ class VideoPlayer(QWidget):
def play_file(self, path: str, info: str = "") -> None: def play_file(self, path: str, info: str = "") -> None:
self._current_file = path self._current_file = path
self._error_fired = False self._error_fired = False
self._player.setLoops( self._player.setLoops(1)
QMediaPlayer.Loops.Infinite if self._loop_mode else 1
)
self._player.setSource(QUrl.fromLocalFile(path)) self._player.setSource(QUrl.fromLocalFile(path))
if self._autoplay: if self._autoplay:
self._player.play() self._player.play()
@ -439,9 +437,6 @@ class VideoPlayer(QWidget):
def _toggle_loop(self) -> None: def _toggle_loop(self) -> None:
self._loop_mode = self._loop_btn.isChecked() self._loop_mode = self._loop_btn.isChecked()
self._loop_btn.setText("Loop" if self._loop_mode else "Next") self._loop_btn.setText("Loop" if self._loop_mode else "Next")
self._player.setLoops(
QMediaPlayer.Loops.Infinite if self._loop_mode else 1
)
def stop(self) -> None: def stop(self) -> None:
self._player.stop() self._player.stop()
@ -483,8 +478,12 @@ class VideoPlayer(QWidget):
self._play_btn.setText("Play") self._play_btn.setText("Play")
def _on_media_status(self, status) -> None: def _on_media_status(self, status) -> None:
if status == QMediaPlayer.MediaStatus.EndOfMedia and not self._loop_mode: if status == QMediaPlayer.MediaStatus.EndOfMedia:
self.play_next.emit() if self._loop_mode:
self._player.setPosition(0)
self._player.play()
else:
self.play_next.emit()
def _on_error(self, error, msg: str = "") -> None: def _on_error(self, error, msg: str = "") -> None:
if self._current_file and not self._error_fired: if self._current_file and not self._error_fired: