Ethanfel/8-cut
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: defer playlist scroll restore to next event loop iteration

Browse Source 
Qt processes layout changes asynchronously, so restoring the scrollbar
immediately after _after_load was too early. Use QTimer.singleShot(0)
to restore after Qt finishes processing pending layout events.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
Ethanfel
2026-04-13 23:44:03 +02:00
parent 7c776e24af
commit 7051cc5b93
1 changed files with 8 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
main.py
+8 -4
View File
@@ -2246,10 +2246,9 @@ class MainWindow(QMainWindow):
self._spn_spread.setValue(float(self._settings.value("spread", "3.0"))) self._spn_spread.setValue(float(self._settings.value("spread", "3.0")))
self._preview_win.show() self._preview_win.show()
self._preview_timer.start() self._preview_timer.start()
# Restore playlist scroll that video load disturbed. # Restore playlist scroll after Qt processes pending layout events.
sb = self._playlist.verticalScrollBar() if hasattr(self, '_playlist_scroll_stash'):
if sb and hasattr(self, '_playlist_scroll_stash'): QTimer.singleShot(0, self._restore_playlist_scroll)
sb.setValue(self._playlist_scroll_stash)
# Run DB fuzzy match off the main thread — can be slow on large databases. # Run DB fuzzy match off the main thread — can be slow on large databases.
filename = os.path.basename(self._file_path) filename = os.path.basename(self._file_path)
@@ -2267,6 +2266,11 @@ class MainWindow(QMainWindow):
self.statusBar().clearMessage() self.statusBar().clearMessage()
self._timeline.set_markers(markers) self._timeline.set_markers(markers)
def _restore_playlist_scroll(self) -> None:
sb = self._playlist.verticalScrollBar()
if sb:
sb.setValue(self._playlist_scroll_stash)
def _refresh_markers(self) -> None: def _refresh_markers(self) -> None:
filename = os.path.basename(self._file_path) filename = os.path.basename(self._file_path)
markers = self._db.get_markers(filename, self._profile) markers = self._db.get_markers(filename, self._profile)