Display shuffle order

src/commands.rs

@@ -120,7 +120,13 @@ impl CommandManager {
                         self.queue.get_current_index()
                     );
                     s.queuestate.queue = queue.clone();
-                    s.queuestate.random_order = self.queue.get_random_order();
+                    s.queuestate.random_order = self
+                        .queue
+                        .get_random_order()
+                        .read()
+                        .unwrap()
+                        .as_ref()
+                        .cloned();
                     s.queuestate.current_track = self.queue.get_current_index();
                     s.queuestate.track_progress = self.spotify.get_current_progress();
                 });

src/queue.rs

@@ -43,7 +43,7 @@ pub struct Queue {
     /// the raw data only.
     pub queue: Arc<RwLock<Vec<Playable>>>,
     /// The playback order of the queue, as indices into `self.queue`.
-    random_order: RwLock<Option<Vec<usize>>>,
+    random_order: Arc<RwLock<Option<Vec<usize>>>>,
     current_track: RwLock<Option<usize>>,
     spotify: Spotify,
     cfg: Arc<Config>,
@@ -62,7 +62,7 @@ impl Queue {
             queue: Arc::new(RwLock::new(queue_state.queue)),
             spotify: spotify.clone(),
             current_track: RwLock::new(queue_state.current_track),
-            random_order: RwLock::new(queue_state.random_order),
+            random_order: Arc::new(RwLock::new(queue_state.random_order)),
             cfg,
             #[cfg(feature = "notify")]
             notification_id: Arc::new(AtomicU32::new(0)),
@@ -442,8 +442,8 @@ impl Queue {
     }
 
     /// Get the current order that is used to shuffle.
-    pub fn get_random_order(&self) -> Option<Vec<usize>> {
-        self.random_order.read().unwrap().clone()
+    pub fn get_random_order(&self) -> Arc<RwLock<Option<Vec<usize>>>> {
+        self.random_order.clone()
     }
 
     /// (Re)generate the random shuffle order.

src/ui/listview.rs

@@ -34,6 +34,7 @@ use crate::ui::pagination::Pagination;
 
 pub struct ListView<I: ListItem> {
     content: Arc<RwLock<Vec<I>>>,
+    order: Arc<RwLock<Option<Vec<usize>>>>,
     last_content_len: usize,
     selected: usize,
     search_query: String,
@@ -61,6 +62,7 @@ impl<I: ListItem + Clone> ListView<I> {
     pub fn new(content: Arc<RwLock<Vec<I>>>, queue: Arc<Queue>, library: Arc<Library>) -> Self {
         let result = Self {
             content,
+            order: Arc::new(RwLock::new(None)),
             last_content_len: 0,
             selected: 0,
             search_query: String::new(),
@@ -77,6 +79,11 @@ impl<I: ListItem + Clone> ListView<I> {
         result
     }
 
+    pub fn with_order(mut self, order: Arc<RwLock<Option<Vec<usize>>>>) -> Self {
+        self.order = order;
+        self
+    }
+
     pub fn with_title(mut self, title: &str) -> Self {
         self.title = title.to_string();
         self
@@ -193,11 +200,19 @@ impl<I: ListItem + Clone> View for ListView<I> {
                     printer.print((0, 0), &buf);
                 });
             } else if i < content.len() {
-                let item = &content[i];
+                let current_index = if self.order.read().unwrap().is_some() {
+                    self.order.read().unwrap().as_ref().unwrap()[i]
+                } else {
+                    i
+                };
+
+                let item = &content[current_index];
+
                 let currently_playing = item.is_playing(self.queue.clone())
-                    && self.queue.get_current_index() == Some(i);
+                    && self.queue.get_current_index() == Some(current_index);
 
                 let style = if self.selected == i {
+                    // Highlight the currently selected item.
                     if currently_playing {
                         ColorStyle::new(
                             *printer.theme.palette.custom("playing_selected").unwrap(),
@@ -207,6 +222,7 @@ impl<I: ListItem + Clone> View for ListView<I> {
                         ColorStyle::highlight()
                     }
                 } else if currently_playing {
+                    // Apply a different color to the currently playing item.
                     ColorStyle::new(
                         ColorType::Color(*printer.theme.palette.custom("playing").unwrap()),
                         ColorType::Color(*printer.theme.palette.custom("playing_bg").unwrap()),

src/ui/queue.rs

@@ -23,7 +23,8 @@ pub struct QueueView {
 
 impl QueueView {
     pub fn new(queue: Arc<Queue>, library: Arc<Library>) -> QueueView {
-        let list = ListView::new(queue.queue.clone(), queue.clone(), library.clone());
+        let list = ListView::new(queue.queue.clone(), queue.clone(), library.clone())
+            .with_order(queue.get_random_order());
 
         QueueView {
             list,