Allow showing actions on the current context page (#857)

As suggested in issue #427 this PR introduces a new `ShowActionsOnCurrentContext` command, this allows the user to trigger the actions menu directly when viewing a context page.

Closes #427

Author: crisco-13

README.md

@@ -374,6 +374,7 @@ List of supported commands:
 | `RestartIntegratedClient`       | restart the integrated client (`streaming` feature only)                                           | `R`                |
 | `ShowActionsOnSelectedItem`     | open a popup showing actions on a selected item                                                    | `g a`, `C-space`   |
 | `ShowActionsOnCurrentTrack`     | open a popup showing actions on the current track                                                  | `a`                |
+| `ShowActionsOnCurrentContext`   | open a popup showing actions on the current context                                                | `A`                |
 | `AddSelectedItemToQueue`        | add the selected item to queue                                                                     | `Z`, `C-z`         |
 | `FocusNextWindow`               | focus the next focusable window (if any)                                                           | `tab`              |
 | `FocusPreviousWindow`           | focus the previous focusable window (if any)                                                       | `backtab`          |

spotify_player/src/command.rs

@@ -56,6 +56,7 @@ pub enum Command {
 
     ShowActionsOnSelectedItem,
     ShowActionsOnCurrentTrack,
+    ShowActionsOnCurrentContext,
     AddSelectedItemToQueue,
     JumpToHighlightTrackInContext,
 
@@ -334,6 +335,7 @@ impl Command {
             Self::RefreshPlayback => "manually refresh the current playback",
             Self::ShowActionsOnSelectedItem => "open a popup showing actions on a selected item",
             Self::ShowActionsOnCurrentTrack => "open a popup showing actions on the current track",
+            Self::ShowActionsOnCurrentContext => "open a popup showing actions on the current context",
             Self::AddSelectedItemToQueue => "add the selected item to queue",
             Self::JumpToHighlightTrackInContext => "jump to the currently highlighted search result in the context",
             Self::FocusNextWindow => "focus the next focusable window (if any)",

spotify_player/src/config/keymap.rs

@@ -123,6 +123,10 @@ impl Default for KeymapConfig {
                     key_sequence: "a".into(),
                     command: Command::ShowActionsOnCurrentTrack,
                 },
+                Keymap {
+                    key_sequence: "A".into(),
+                    command: Command::ShowActionsOnCurrentContext,
+                },
                 #[cfg(feature = "streaming")]
                 Keymap {
                     key_sequence: "R".into(),

spotify_player/src/event/page.rs

@@ -1,6 +1,8 @@
 use anyhow::Context as _;
 use command::CommandOrAction;
 
+use crate::command::{construct_album_actions, construct_playlist_actions, construct_show_actions};
+
 use super::*;
 
 pub fn handle_key_sequence_for_page(
@@ -364,6 +366,55 @@ fn handle_command_for_context_page(
             ui.new_search_popup();
             Ok(true)
         }
+        Command::ShowActionsOnCurrentContext => {
+            let context_id = match ui.current_page() {
+                PageState::Context { id, .. } => match id {
+                    None => return Ok(false),
+                    Some(id) => id,
+                },
+                _ => anyhow::bail!("expect a context page"),
+            };
+            let data = state.data.read();
+
+            match data.caches.context.get(&context_id.uri()) {
+                Some(context) => match context {
+                    Context::Playlist { playlist, .. } => {
+                        let actions = construct_playlist_actions(playlist, &data);
+                        ui.popup = Some(PopupState::ActionList(
+                            Box::new(ActionListItem::Playlist(playlist.clone(), actions)),
+                            ListState::default(),
+                        ));
+                        Ok(true)
+                    }
+                    Context::Album { album, .. } => {
+                        let actions = construct_album_actions(album, &data);
+                        ui.popup = Some(PopupState::ActionList(
+                            Box::new(ActionListItem::Album(album.clone(), actions)),
+                            ListState::default(),
+                        ));
+                        Ok(true)
+                    }
+                    Context::Artist { artist, .. } => {
+                        let actions = construct_artist_actions(artist, &data);
+                        ui.popup = Some(PopupState::ActionList(
+                            Box::new(ActionListItem::Artist(artist.clone(), actions)),
+                            ListState::default(),
+                        ));
+                        Ok(true)
+                    }
+                    Context::Show { show, .. } => {
+                        let actions = construct_show_actions(show, &data);
+                        ui.popup = Some(PopupState::ActionList(
+                            Box::new(ActionListItem::Show(show.clone(), actions)),
+                            ListState::default(),
+                        ));
+                        Ok(true)
+                    }
+                    Context::Tracks { tracks: _, desc: _ } => Ok(false),
+                },
+                None => Ok(false),
+            }
+        }
         _ => window::handle_command_for_focused_context_window(command, client_pub, ui, state),
     }
 }