Reverse the order of the Screen and Webcam selectors

[srec] / srec.py

diff --git a/srec.py b/srec.py
index da14735ca81d3043995afc46fd4e84e801270c0c..7780e511e4b423afa0fe7adaaabfdf11ab88af18 100644 (file)
--- a/srec.py
+++ b/srec.py
@@ -27,22 +27,24 @@ recording: Recording | None = None
 def make_filename() -> str:
     directory = os.environ.get(
         'XDG_VIDEOS_DIR',
-        os.path.expanduser('~/Videos'))
+        os.path.expanduser('~/Videos/SRec'))
     os.makedirs(directory, exist_ok=True)
     timestamp = datetime.now().strftime('%Y-%m-%d %H:%M:%S')
     return os.path.join(directory, f'srec {timestamp}.mkv')
 
 
+def video_source(stack: Gtk.Stack) -> list[str]:
+    if stack.get_child_by_name('not_recording').get_first_child().get_active():
+        return ['-f', 'x11grab', '-i', ':0.0+0,0']
+    return ['-f', 'v4l2', '-i', '/dev/video0']
+
+
 def on_start_recording(_: Gtk.Button, stack: Gtk.Stack) -> None:
     global recording  # pylint: disable=global-statement
     assert recording is None
     filename = make_filename()
-    command = [
-            'ffmpeg',
-            '-framerate', '25',
-            '-f', 'x11grab', '-i', ':0.0+0,0',
-            '-f', 'pulse', '-ac', '2', '-i', 'default',
-            filename]  # nopep8
+    command = (['ffmpeg', '-framerate', '25'] + video_source(stack) +
+               ['-f', 'pulse', '-ac', '2', '-i', 'default', filename])
     # pylint: disable=consider-using-with
     recording = Recording(
         filename=filename,
@@ -76,10 +78,15 @@ def make_button(label: str, action: Callable[[
 def on_activate(app: Gtk.Application) -> None:
     win = Gtk.ApplicationWindow(application=app)
     win.set_title('SRec')
+    win.set_icon_name('srec')
+
     stack = Gtk.Stack()
 
     nr_box = Gtk.Box()
     nr_box.set_orientation(Gtk.Orientation.VERTICAL)
+    screen = Gtk.CheckButton(label='Screen')
+    nr_box.append(screen)
+    nr_box.append(Gtk.CheckButton(label='Webcam', active=True, group=screen))
     nr_box.append(make_button("Start Recording", on_start_recording, stack))
     stack.add_named(nr_box, "not_recording")