fix playbar seek position

resources/playbar.blp

@@ -66,7 +66,7 @@ template $AudreyUiPlaybar: Adw.Bin {
             "numeric",
           ]
 
-          label: bind $format_timestamp (template.playbin as <$AudreyPlaybin>.position) as <string>;
+          label: bind $format_timestamp (template.position) as <string>;
         }
 
         [center]
@@ -74,12 +74,12 @@ template $AudreyUiPlaybar: Adw.Bin {
           name: "seek-scale";
           orientation: horizontal;
           width-request: 400;
-          sensitive: bind $playbin_active (template.playbin as <$AudreyPlaybin>.state as <$AudreyPlaybinState>) as <bool>;
+          //sensitive: bind $playbin_active (template.playbin as <$AudreyPlaybin>.state as <$AudreyPlaybinState>) as <bool>;
 
           adjustment: Adjustment {
             lower: 0;
-            value: bind template.playbin as <$AudreyPlaybin>.position;
-            upper: bind template.playbin as <$AudreyPlaybin>.duration;
+            value: bind template.position;
+            upper: bind template.duration;
           };
 
           change-value => $on_play_position_seek () swapped;
@@ -92,7 +92,7 @@ template $AudreyUiPlaybar: Adw.Bin {
             "numeric",
           ]
 
-          label: bind $format_timestamp (template.playbin as <$AudreyPlaybin>.duration) as <string>;
+          label: bind $format_timestamp (template.duration) as <string>;
         }
       }
 
@@ -103,7 +103,7 @@ template $AudreyUiPlaybar: Adw.Bin {
         Button {
           icon-name: "media-skip-backward";
           valign: center;
-          sensitive: bind $playbin_active (template.playbin as <$AudreyPlaybin>.state as <$AudreyPlaybinState>) as <bool>;
+          //sensitive: bind $playbin_active (template.playbin as <$AudreyPlaybin>.state as <$AudreyPlaybinState>) as <bool>;
 
           clicked => $on_skip_backward_clicked () swapped;
         }
@@ -111,15 +111,16 @@ template $AudreyUiPlaybar: Adw.Bin {
         Button {
           icon-name: "media-seek-backward";
           valign: center;
-          sensitive: bind $playbin_active (template.playbin as <$AudreyPlaybin>.state as <$AudreyPlaybinState>) as <bool>;
+          //sensitive: bind $playbin_active (template.playbin as <$AudreyPlaybin>.state as <$AudreyPlaybinState>) as <bool>;
 
           clicked => $seek_backward () swapped;
         }
 
         Button {
-          icon-name: bind $play_pause_icon_name (template.playbin as <$AudreyPlaybin>.state as <$AudreyPlaybinState>) as <string>;
+          icon-name: "media-playback-start";
+          //icon-name: bind $play_pause_icon_name (template.playbin as <$AudreyPlaybin>.state as <$AudreyPlaybinState>) as <string>;
           valign: center;
-          sensitive: bind $can_press_play (template.playbin as <$AudreyPlaybin>.state as <$AudreyPlaybinState>, template.playbin as <$AudreyPlaybin>.play-queue-length) as <bool>;
+          //sensitive: bind $can_press_play (template.playbin as <$AudreyPlaybin>.state as <$AudreyPlaybinState>, template.playbin as <$AudreyPlaybin>.play-queue-length) as <bool>;
 
           clicked => $on_play_pause_clicked () swapped;
         }
@@ -127,7 +128,7 @@ template $AudreyUiPlaybar: Adw.Bin {
         Button {
           icon-name: "media-seek-forward";
           valign: center;
-          sensitive: bind $playbin_active (template.playbin as <$AudreyPlaybin>.state as <$AudreyPlaybinState>) as <bool>;
+          //sensitive: bind $playbin_active (template.playbin as <$AudreyPlaybin>.state as <$AudreyPlaybinState>) as <bool>;
 
           clicked => $seek_forward () swapped;
         }
@@ -135,7 +136,7 @@ template $AudreyUiPlaybar: Adw.Bin {
         Button {
           icon-name: "media-skip-forward";
           valign: center;
-          sensitive: bind $playbin_active (template.playbin as <$AudreyPlaybin>.state as <$AudreyPlaybinState>) as <bool>;
+          //sensitive: bind $playbin_active (template.playbin as <$AudreyPlaybin>.state as <$AudreyPlaybinState>) as <bool>;
 
           clicked => $on_skip_forward_clicked () swapped;
         }

src/mpv/error.rs

@@ -3,7 +3,7 @@ use std::ffi::{c_int, CStr};
 use std::fmt;
 
 #[derive(Copy, Clone)]
-pub struct Error(c_int);
+pub struct Error(pub(super) c_int);
 
 impl Error {
     pub(super) fn from_return_code(rc: c_int) -> Result<(), Self> {

src/mpv/format.rs

@@ -122,6 +122,19 @@ impl GetProperty for i64 {
     }
 }
 
+impl GetProperty for f64 {
+    unsafe fn get_property(ctx: *mut ffi::mpv_handle, name: *const c_char) -> Result<Self, Error> {
+        let mut value: f64 = 0.0;
+        Error::from_return_code(ffi::mpv_get_property(
+            ctx,
+            name,
+            ffi::mpv_format_MPV_FORMAT_DOUBLE,
+            std::ptr::from_mut::<f64>(&mut value) as *mut c_void,
+        ))?;
+        Ok(value)
+    }
+}
+
 impl GetProperty for String {
     unsafe fn get_property(ctx: *mut ffi::mpv_handle, name: *const c_char) -> Result<Self, Error> {
         let mut value: *mut c_char = std::ptr::null_mut();
@@ -136,3 +149,13 @@ impl GetProperty for String {
         Ok(result)
     }
 }
+
+impl<T: GetProperty> GetProperty for Option<T> {
+    unsafe fn get_property(ctx: *mut ffi::mpv_handle, name: *const c_char) -> Result<Self, Error> {
+        match T::get_property(ctx, name) {
+            Ok(t) => Ok(Some(t)),
+            Err(Error(ffi::mpv_error_MPV_ERROR_PROPERTY_UNAVAILABLE)) => Ok(None),
+            Err(err) => Err(err),
+        }
+    }
+}

src/playbin2.rs

@@ -95,7 +95,7 @@ where
     }
 
     pub fn position(&self) -> Option<f64> {
-        todo!()
+        self.mpv.get_property("time-pos").unwrap()
     }
 
     pub fn current_entry(&self) -> Option<usize> {

src/ui/playbar.rs

@@ -23,6 +23,10 @@ mod imp {
         volume: Cell<i32>,
         #[property(get, set)]
         mute: Cell<bool>,
+        #[property(get, set)]
+        position: Cell<f64>,
+        #[property(get, set)]
+        duration: Cell<f64>,
     }
 
     #[glib::object_subclass]

src/ui/window.rs

@@ -188,6 +188,7 @@ mod imp {
 use adw::prelude::*;
 use adw::subclass::prelude::*;
 use gtk::{gdk, gio, glib};
+use std::rc::Rc;
 
 glib::wrapper! {
     pub struct Window(ObjectSubclass<imp::Window>)
@@ -237,6 +238,33 @@ impl Window {
             ),
         );
 
+        window.imp().playbin2.file_started().connect_object(
+            &*window.imp().playbar,
+            |playbin, playbar, ()| {
+                let entry = &playbin.entries()[playbin.current_entry().unwrap()];
+                playbar.set_duration(entry.duration() as f64);
+                true
+            },
+        );
+
+        // update position every 100 ms
+        glib::source::timeout_add_local(std::time::Duration::from_millis(100), {
+            let playbar = window.imp().playbar.downgrade();
+            let playbin = Rc::downgrade(&window.imp().playbin2);
+            move || {
+                let playbar = match playbar.upgrade() {
+                    None => return glib::ControlFlow::Break,
+                    Some(playbar) => playbar,
+                };
+                let playbin = match playbin.upgrade() {
+                    None => return glib::ControlFlow::Break,
+                    Some(playbin) => playbin,
+                };
+                playbar.set_position(playbin.position().unwrap_or(0.0));
+                glib::ControlFlow::Continue
+            }
+        });
+
         window
             .imp()
             .setup