From 46b1c2d609f46c3d248695d3ec4fc9f0d4fa3a60 Mon Sep 17 00:00:00 2001 From: Elyse Date: Mon, 1 Nov 2021 19:32:35 -0600 Subject: [PATCH] SoundPlayer: Add a keyboard shortcut 'M' to mute the player This shortcut let us mute/unmute the player, but it still doesn't update the volume slider because the actual volume widget can't display a muted state. --- .../SoundPlayer/SoundPlayerWidgetAdvancedView.cpp | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/Userland/Applications/SoundPlayer/SoundPlayerWidgetAdvancedView.cpp b/Userland/Applications/SoundPlayer/SoundPlayerWidgetAdvancedView.cpp index b5d8ce353a..4f94fcca07 100644 --- a/Userland/Applications/SoundPlayer/SoundPlayerWidgetAdvancedView.cpp +++ b/Userland/Applications/SoundPlayer/SoundPlayerWidgetAdvancedView.cpp @@ -142,6 +142,9 @@ void SoundPlayerWidgetAdvancedView::keydown_event(GUI::KeyEvent& event) if (event.key() == Key_Space) m_play_button->click(); + if (event.key() == Key_M) + toggle_mute(); + if (event.key() == Key_S) m_stop_button->click(); @@ -182,6 +185,7 @@ void SoundPlayerWidgetAdvancedView::loop_mode_changed(Player::LoopMode) void SoundPlayerWidgetAdvancedView::mute_changed(bool) { + // FIXME: Update the volume slider when player is muted } void SoundPlayerWidgetAdvancedView::sync_previous_next_buttons()