Config option to sort artists albums by type (#654)

Co-authored-by: John Oberhauser <[email protected]>
Co-authored-by: Thang Pham <[email protected]>

.gitignore

@@ -20,3 +20,5 @@ NOTES*
 
 # Generated html files for live previewing markdown
 *.html
+
+.idea

docs/config.md

@@ -55,6 +55,7 @@ All configuration files should be placed inside the application's configuration
 | `cover_img_length`                | the length of the cover image (`image` feature only)                                     | `9`                                                     |
 | `cover_img_scale`                 | the scale of the cover image (`image` feature only)                                      | `1.0`                                                   |
 | `seek_duration_secs`              | the duration (in seconds) to seek when using `SeekForward` and `SeekBackward` commands   | `5`                                                     |
+| `sort_artist_albums_by_type`      | sort albums on artist's pages by type, i.e. album or single                              | `false`                                                 |
 
 ### Notes
 

spotify_player/src/config/mod.rs

@@ -114,6 +114,8 @@ pub struct AppConfig {
     pub notify_streaming_only: bool,
 
     pub seek_duration_secs: u16,
+
+    pub sort_artist_albums_by_type: bool,
 }
 
 #[derive(Debug, Deserialize, Serialize, Clone)]
@@ -324,6 +326,8 @@ impl Default for AppConfig {
             notify_streaming_only: false,
 
             seek_duration_secs: 5,
+
+            sort_artist_albums_by_type: false,
         }
     }
 }

spotify_player/src/ui/page.rs

@@ -796,8 +796,9 @@ fn render_artist_context_page_windows(
     rect: Rect,
     artist_data: (&[Track], &[Album], &[Artist]),
 ) {
+    let configs = config::get_config();
     // 1. Get data
-    let (tracks, albums, artists) = (
+    let (tracks, mut albums, artists) = (
         ui.search_filtered_items(artist_data.0),
         ui.search_filtered_items(artist_data.1),
         ui.search_filtered_items(artist_data.2),
@@ -830,6 +831,19 @@ fn render_artist_context_page_windows(
 
     // 3. Construct the page's widgets
     // album table
+    if configs.app_config.sort_artist_albums_by_type {
+        fn get_priority(album_type: &str) -> usize {
+            match album_type {
+                "album" => 0,
+                "single" => 1,
+                "appears_on" => 2,
+                "compilation" => 3,
+                _ => 4,
+            }
+        }
+        albums.sort_by_key(|a| get_priority(&a.album_type()));
+    }
+
     let is_albums_active = is_active && focus_state == ArtistFocusState::Albums;
     let n_albums = albums.len();
     let album_rows = albums