add MPV button and improve layout

Author: sven337

examples/download_motions.py

@@ -7,8 +7,9 @@
 
 import sys
 import re
+import subprocess
 from enum import Flag
-from PyQt6.QtWidgets import QApplication, QVBoxLayout, QHBoxLayout, QWidget, QTableWidget, QTableWidgetItem, QPushButton, QLabel, QFileDialog, QHeaderView, QStyle, QSlider, QStyleOptionSlider
+from PyQt6.QtWidgets import QApplication, QVBoxLayout, QHBoxLayout, QWidget, QTableWidget, QTableWidgetItem, QPushButton, QLabel, QFileDialog, QHeaderView, QStyle, QSlider, QStyleOptionSlider, QSplitter, QToolTip
 from PyQt6.QtMultimedia import QMediaPlayer, QAudioOutput
 from PyQt6.QtMultimediaWidgets import QVideoWidget
 from PyQt6.QtCore import Qt, QUrl, QTimer
@@ -94,7 +95,7 @@ def pixelPosToRangeValue(self, pos):
 class VideoPlayer(QWidget):
     def __init__(self, video_files):
         super().__init__()
-        self.setWindowTitle("Video Player")
+        self.setWindowTitle("Reolink Video Review GUI")
 
         # Create media player
         self.media_player = QMediaPlayer()
@@ -120,6 +121,9 @@ def __init__(self, video_files):
         self.play_button = QPushButton()
         self.play_button.setIcon(self.style().standardIcon(QStyle.StandardPixmap.SP_MediaPlay))
         self.play_button.clicked.connect(self.play_pause)
+
+        self.mpv_button = QPushButton("MPV")
+        self.mpv_button.clicked.connect(self.open_in_mpv)
 
         # Create seek slider
         self.seek_slider = ClickableSlider(Qt.Orientation.Horizontal)
@@ -138,26 +142,43 @@ def __init__(self, video_files):
         speed_label = QLabel("Speed: 2.0x")
         self.speed_slider.valueChanged.connect(lambda v: speed_label.setText(f"Speed: {v/100:.1f}x"))
 
-        # Set up layout
         main_layout = QHBoxLayout()
-        left_layout = QVBoxLayout()
+        # Create a splitter
+        splitter = QSplitter(Qt.Orientation.Horizontal)
+        
+        # Left side (table and open button)
+        left_widget = QWidget()
+        left_layout = QVBoxLayout(left_widget)
         left_layout.addWidget(self.video_table)
         left_layout.addWidget(self.open_button)
-        main_layout.addLayout(left_layout)
+        splitter.addWidget(left_widget)
 
-        right_layout = QVBoxLayout()
-        right_layout.addWidget(self.video_widget)
+        # Right side (video player and controls)
+        right_widget = QWidget()
+        right_layout = QVBoxLayout(right_widget)
+        right_layout.addWidget(self.video_widget, 1)
+
+        controls_widget = QWidget()
+        controls_layout = QVBoxLayout(controls_widget)
+
         control_layout = QHBoxLayout()
         control_layout.addWidget(self.play_button)
         control_layout.addWidget(self.seek_slider)
-        right_layout.addLayout(control_layout)
+        control_layout.addWidget(self.mpv_button)
+        controls_layout.addLayout(control_layout)
+
         speed_layout = QHBoxLayout()
         speed_layout.addWidget(QLabel("Speed:"))
         speed_layout.addWidget(self.speed_slider)
         speed_layout.addWidget(speed_label)
-        right_layout.addLayout(speed_layout)
-        main_layout.addLayout(right_layout)
+        controls_layout.addLayout(speed_layout)
+        right_layout.addWidget(controls_widget)
+        splitter.addWidget(right_widget)
+        
+        # Set initial sizes
+        splitter.setSizes([300, 700])  # Adjust these values as needed
 
+        main_layout.addWidget(splitter)
         self.setLayout(main_layout)
         self.add_initial_videos(video_files)
 
@@ -183,7 +204,14 @@ def add_video(self, video_path):
             start_datetime_str = parsed_data['start_datetime'].strftime("%Y-%m-%d %H:%M:%S")
             start_datetime_item = QTableWidgetItem(start_datetime_str)
             start_datetime_item.setData(Qt.ItemDataRole.UserRole, parsed_data['start_datetime'])
-            self.video_table.setItem(row_position, 0, QTableWidgetItem(base_file_name))
+    
+            # Create the item for the first column with the base file name
+            file_name_item = QTableWidgetItem(base_file_name)
+
+            # Set the full path as tooltip
+            file_name_item.setToolTip(base_file_name)
+
+            self.video_table.setItem(row_position, 0, file_name_item)
             self.video_table.setItem(row_position, 1, start_datetime_item)
             self.video_table.setItem(row_position, 2, QTableWidgetItem(parsed_data['end_time']))
             self.video_table.setItem(row_position, 3, QTableWidgetItem(f"{parsed_data['channel']}"))
@@ -196,14 +224,24 @@ def add_video(self, video_path):
             self.video_table.setItem(row_position, 8, QTableWidgetItem("✓" if parsed_data['triggers'] & VOD_trigger.TIMER else ""))
             self.video_table.setItem(row_position, 9, QTableWidgetItem(parsed_data['unknown_hex']))
 
+            # Set smaller default column widths
+            self.video_table.setColumnWidth(0, 120)  # Video Path
+            self.video_table.setColumnWidth(1, 130)  # Start Datetime
+            self.video_table.setColumnWidth(2, 80)   # End Time
+            self.video_table.setColumnWidth(3, 35)   # Channel
+            self.video_table.setColumnWidth(4, 35)   # Person
+            self.video_table.setColumnWidth(5, 35)   # Vehicle
+            self.video_table.setColumnWidth(6, 35)   # Pet
+            self.video_table.setColumnWidth(7, 35)   # Motion
+            self.video_table.setColumnWidth(8, 30)   # Timer
+            self.video_table.setColumnWidth(9, 20)   # Unknown Hex
+
             # Make the fields non-editable
             for column in range(self.video_table.columnCount()):
                 item = self.video_table.item(row_position, column)
                 item.setFlags(item.flags() & ~Qt.ItemFlag.ItemIsEditable)
         else:
             print(f"Could not parse file {video_path}")
-        for i in range(1, self.video_table.columnCount()):
-            self.video_table.resizeColumnToContents(i)
 
     def play_video(self, row, column):
         video_path = self.video_table.item(row, 0).text()
@@ -254,6 +292,18 @@ def update_duration(self, duration):
     def set_speed(self, speed):
         self.media_player.setPlaybackRate(speed / 100)
 
+    def open_in_mpv(self):
+        current_video = self.media_player.source().toString()
+        if current_video:
+            # Remove the 'file://' prefix if present
+            video_path = current_video.replace('file://', '')
+            try:
+                subprocess.Popen(['mpv', video_path])
+            except FileNotFoundError:
+                print("Error: MPV player not found. Make sure it's installed and in your system PATH.")
+        else:
+            print("No video is currently selected.")
+
 def read_config(props_path: str) -> dict:
     """Reads in a properties file into variables.