resources/play_queue_song.blp

@@ -3,10 +3,10 @@ using Gtk 4.0;
 template $AudreyUiPlayQueueSong: Box {
   height-request: 48;
   spacing: 12;
+  margin-start: 6;
+  margin-end: 6;
 
   Box {
-    margin-start: 6;
-
     width-request: 36;
     focusable: false;
     homogeneous: true;
@@ -39,7 +39,7 @@ template $AudreyUiPlayQueueSong: Box {
     margin-top: 1;
     margin-bottom: 1;
     pixel-size: 50;
-    // paintable: bind template.song as <$AudreyModelSong>.thumbnail;
+    // paintable: bind template.song as <$AudreyPlaybinSong>.thumbnail;
   }
 
   Box title_box {
@@ -70,7 +70,7 @@ template $AudreyUiPlayQueueSong: Box {
         ellipsize: end;
         max-width-chars: 90;
         justify: fill;
-        label: bind template.song as <$AudreyModelSong>.title;
+        label: bind template.song as <$AudreyPlaybinSong>.title;
       }
 
       Label {
@@ -85,7 +85,7 @@ template $AudreyUiPlayQueueSong: Box {
         ellipsize: end;
         max-width-chars: 90;
         justify: fill;
-        label: bind template.song as <$AudreyModelSong>.artist;
+        label: bind template.song as <$AudreyPlaybinSong>.artist;
       }
     }
   }
@@ -101,12 +101,12 @@ template $AudreyUiPlayQueueSong: Box {
       "dim-label"
     ]
 
-    label: bind $format_duration(template.song as <$AudreyModelSong>.duration) as <string>;
+    label: bind $format_duration(template.song as <$AudreyPlaybinSong>.duration) as <string>;
   }
 
   Button {
     focusable: true;
-    // TODO icon-name: bind $star_button_icon_name (template.song as <$AudreyModelSong>.starred) as <string>;
+    // TODO icon-name: bind $star_button_icon_name (template.song as <$AudreyPlaybinSong>.starred) as <string>;
     icon-name: bind $star_button_icon_name() as <string>;
 
     styles [
@@ -116,24 +116,20 @@ template $AudreyUiPlayQueueSong: Box {
     valign: center;
   }
 
-  Box {
-    MenuButton {
-      // visible: false;
-      focusable: true;
-      icon-name: "view-more";
+  MenuButton {
+    // visible: false;
+    focusable: true;
+    icon-name: "view-more";
 
-      styles [
-        "flat"
-      ]
+    styles [
+      "flat"
+    ]
 
-      valign: center;
+    valign: center;
 
-      popover: PopoverMenu {
-        menu-model: song-menu;
-      };
-    }
-
-    margin-end: 6;
+    popover: PopoverMenu {
+      menu-model: song-menu;
+    };
   }
 
   DragSource {

resources/style.css

@@ -19,11 +19,6 @@
   min-height: 15px;
 }
 
-/* make drag and drop indicator take up entire perimeter */
-#play-queue listview row {
-  padding: 0;
-}
-
 #play-queue .playing label.title {
   font-weight: bold;
 }

src/main.rs

@@ -11,7 +11,8 @@ pub mod ui;
 pub mod mpris;
 pub use mpris::Mpris;
 
-pub mod model;
+pub mod playbin_song;
+pub use playbin_song::Song as PlaybinSong;
 
 pub mod subsonic;
 

src/model.rs

@@ -1,2 +0,0 @@
-mod song;
-pub use song::Song;

src/playbin_song.rs

@@ -35,7 +35,7 @@ mod imp {
 
     #[glib::object_subclass]
     impl ObjectSubclass for Song {
-        const NAME: &'static str = "AudreyModelSong";
+        const NAME: &'static str = "AudreyPlaybinSong";
         type Type = super::Song;
     }
 

src/ui/play_queue/song.rs

@@ -1,5 +1,5 @@
 mod imp {
-    use crate::model::Song as ModelSong;
+    use crate::PlaybinSong;
     use glib::subclass::InitializingObject;
     use gtk::{gdk, gio, glib, prelude::*, subclass::prelude::*};
     use std::cell::{Cell, RefCell};
@@ -28,7 +28,7 @@ mod imp {
         #[property(set, get)]
         displayed_position: Cell<u32>,
         #[property(get, set)]
-        song: RefCell<Option<ModelSong>>,
+        song: RefCell<Option<PlaybinSong>>,
 
         drag_pos: Cell<(i32, i32)>,
         drag_widget: Cell<Option<gtk::ListBox>>,
@@ -112,7 +112,8 @@ mod imp {
         fn on_drag_prepare(&self, x: f64, y: f64) -> Option<gdk::ContentProvider> {
             if self.draggable.get() {
                 self.drag_pos.replace((x as i32, y as i32));
-                Some(gdk::ContentProvider::for_value(&self.obj().to_value()))
+                let value = self.obj().to_value();
+                Some(gdk::ContentProvider::for_value(&value))
             } else {
                 None
             }
@@ -121,7 +122,6 @@ mod imp {
         #[template_callback]
         fn on_drag_begin(&self, drag: &gdk::Drag) {
             let drag_widget = gtk::ListBox::new();
-            drag_widget.add_css_class("boxed-list");
 
             let drag_row: super::Song = glib::Object::new();
             drag_row.set_draggable(false);
@@ -148,23 +148,19 @@ mod imp {
         }
 
         #[template_callback]
-        // why BoxedValue? see https://discourse.gnome.org/t/gtk-rs-passing-widget-as-gvalue-through-drag-and-drop-with-composite-template/16449/2
-        fn on_drop(&self, value: glib::BoxedValue, _x: f64, _y: f64) -> bool {
+        fn on_drop(&self, _value: glib::Value, _x: f64, _y: f64) -> bool {
+            /* FIXME: WrongValueType(ValueTypeMismatchError { actual: GValue, requested: AudreyUiPlayQueueSong })
             let source: super::Song = value.get().unwrap();
 
             source.imp().drag_widget.set(None);
 
-            let from = source.position();
-            let to = self.obj().position();
+            self.obj().playbin().move_track(
+                source.displayed_position() - 1,
+                self.obj().displayed_position() - 1,
+            );
 
-            // see playlist_move for justification
-            if from < to {
-                self.obj().window().playlist_move(from, to + 1);
-            } else {
-                self.obj().window().playlist_move(from, to);
-            }
-
-            true
+            true */
+            false
         }
 
         fn set_playlist_pos(&self, playlist_pos: i32) {
@@ -174,7 +170,7 @@ mod imp {
     }
 }
 
-use crate::model::Song as ModelSong;
+use crate::PlaybinSong;
 use adw::prelude::*;
 use gtk::glib;
 
@@ -190,7 +186,7 @@ impl Song {
     }
 
     pub fn bind(&self, position: u32, window: &crate::ui::Window) {
-        let song: ModelSong = window
+        let song: PlaybinSong = window
             .playlist_model()
             .item(position)
             .unwrap()

src/ui/playbar.rs

@@ -1,5 +1,5 @@
 mod imp {
-    use crate::model::Song;
+    use crate::PlaybinSong;
     use adw::prelude::*;
     use adw::subclass::prelude::*;
     use glib::subclass::InitializingObject;
@@ -16,7 +16,7 @@ mod imp {
         pulse_bar: TemplateChild<gtk::ProgressBar>,
 
         #[property(get, set, nullable)]
-        song: RefCell<Option<Song>>,
+        song: RefCell<Option<PlaybinSong>>,
         #[property(get, set)]
         playing_cover_art: RefCell<Option<gdk::Paintable>>,
         #[property(get, set, default = true)]
@@ -168,18 +168,18 @@ mod imp {
         // these are nedeed because with regular bindings, if song becomes None, the labes are not
         // updated
         #[template_callback]
-        fn song_title(&self, song: Option<&Song>) -> String {
-            song.map(Song::title).unwrap_or("".to_string())
+        fn song_title(&self, song: Option<&PlaybinSong>) -> String {
+            song.map(PlaybinSong::title).unwrap_or("".to_string())
         }
 
         #[template_callback]
-        fn song_artist(&self, song: Option<&Song>) -> String {
-            song.map(Song::artist).unwrap_or("".to_string())
+        fn song_artist(&self, song: Option<&PlaybinSong>) -> String {
+            song.map(PlaybinSong::artist).unwrap_or("".to_string())
         }
 
         #[template_callback]
-        fn song_album(&self, song: Option<&Song>) -> String {
-            song.map(Song::album).unwrap_or("".to_string())
+        fn song_album(&self, song: Option<&PlaybinSong>) -> String {
+            song.map(PlaybinSong::album).unwrap_or("".to_string())
         }
     }
 

src/ui/window.rs

@@ -1,6 +1,6 @@
 mod imp {
-    use crate::model::Song;
     use crate::mpv;
+    use crate::PlaybinSong;
     use adw::prelude::*;
     use adw::subclass::prelude::*;
     use glib::subclass::InitializingObject;
@@ -26,7 +26,7 @@ mod imp {
         #[property(get, set, nullable)]
         playing_cover_art: RefCell<Option<gdk::Paintable>>,
 
-        #[property(type = Option<Song>, get = Self::song, nullable)]
+        #[property(type = Option<PlaybinSong>, get = Self::song, nullable)]
         _song: (),
 
         pub(super) setup: crate::ui::Setup,
@@ -95,7 +95,7 @@ mod imp {
                 setup: Default::default(),
                 api: Default::default(),
                 mpv,
-                playlist_model: gio::ListStore::new::<Song>(),
+                playlist_model: gio::ListStore::new::<PlaybinSong>(),
 
                 _volume: (),
                 _mute: (),
@@ -398,7 +398,7 @@ mod imp {
                 Rc::clone(api.as_ref().unwrap())
             };
             for song in api.get_random_songs(10).await.unwrap().into_iter() {
-                let song = Song::from_child(&api, &song);
+                let song = PlaybinSong::from_child(&api, &song);
                 self.mpv
                     .command(["loadfile", &song.stream_url(), "append-play"])
                     .unwrap();
@@ -474,7 +474,7 @@ mod imp {
 
             {
                 let left = duration.map(|f| f as i64);
-                let right = self.song().as_ref().map(crate::model::Song::duration);
+                let right = self.song().as_ref().map(crate::PlaybinSong::duration);
                 if left != right {
                     event!(
                         Level::WARN,
@@ -498,11 +498,11 @@ mod imp {
                 .unwrap()
         }
 
-        fn song(&self) -> Option<Song> {
+        fn song(&self) -> Option<PlaybinSong> {
             if self.obj().playlist_pos() < 0 {
                 None
             } else {
-                let song: Song = self
+                let song: PlaybinSong = self
                     .obj()
                     .playlist_model()
                     .item(self.obj().playlist_pos() as u32)
@@ -627,39 +627,4 @@ impl Window {
             }
         }
     }
-
-    pub fn playlist_move(&self, from: u32, to: u32) {
-        // NOTE: for mpv, to refers to the "gap" right before i
-        // so playlist-move i 0 makes a track the first
-        // playlist-move i 1 makes a track the second
-        // and playlist-move i playlist-count makes a track the last
-        // (so if to is the position of another track, from is left behind to)
-        self.imp()
-            .mpv
-            .command(["playlist-move", &from.to_string(), &to.to_string()])
-            .unwrap();
-
-        if from < to {
-            // F1234T -> 1234FT
-            let mut spliced = Vec::with_capacity((to - from) as usize);
-            for i in from + 1..to {
-                spliced.push(self.playlist_model().item(i).unwrap());
-            }
-            spliced.push(self.playlist_model().item(from).unwrap());
-
-            self.playlist_model()
-                .splice(from, spliced.len() as u32, &spliced);
-        } else if to < from {
-            // T1234F -> FT1234
-            let mut spliced = Vec::with_capacity((from - to + 1) as usize);
-            spliced.push(self.playlist_model().item(from).unwrap());
-            spliced.push(self.playlist_model().item(to).unwrap());
-            for i in to + 1..from {
-                spliced.push(self.playlist_model().item(i).unwrap());
-            }
-
-            self.playlist_model()
-                .splice(to, spliced.len() as u32, &spliced);
-        }
-    }
 }