RGBCube/serenity
1
Fork 0
mirror of https://github.com/RGBCube/serenity synced 2025-07-25 18:37:35 +00:00
Code Issues Activity Actions

SoundPlayer: Replace two manual key event checks with action shortcut

Browse source
This commit is contained in:
Linus Groh 2022-05-29 12:20:29 +01:00 • committed by Andreas Kling
parent 1940363e0b
commit 094487b5d4
1 changed files with 2 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

10
Userland/Applications/SoundPlayer/SoundPlayerWidgetAdvancedView.cpp
View file

@ -62,13 +62,13 @@ SoundPlayerWidgetAdvancedView::SoundPlayerWidgetAdvancedView(GUI::Window& window
auto& toolbar_container = m_player_view->add<GUI::ToolbarContainer>(); auto& toolbar_container = m_player_view->add<GUI::ToolbarContainer>();
auto& menubar = toolbar_container.add<GUI::Toolbar>(); auto& menubar = toolbar_container.add<GUI::Toolbar>();
m_play_action = GUI::Action::create("Play", m_play_icon, [&](auto&) { m_play_action = GUI::Action::create("Play", { Key_Space }, m_play_icon, [&](auto&) {
toggle_pause(); toggle_pause();
}); });
m_play_action->set_enabled(false); m_play_action->set_enabled(false);
menubar.add_action(*m_play_action); menubar.add_action(*m_play_action);
m_stop_action = GUI::Action::create("Stop", m_stop_icon, [&](auto&) { m_stop_action = GUI::Action::create("Stop", { Key_S }, m_stop_icon, [&](auto&) {
stop(); stop();
}); });
m_stop_action->set_enabled(false); m_stop_action->set_enabled(false);
@ -132,15 +132,9 @@ void SoundPlayerWidgetAdvancedView::drop_event(GUI::DropEvent& event)
void SoundPlayerWidgetAdvancedView::keydown_event(GUI::KeyEvent& event) void SoundPlayerWidgetAdvancedView::keydown_event(GUI::KeyEvent& event)
{ {
if (event.key() == Key_Space)
m_play_action->activate();
if (event.key() == Key_M) if (event.key() == Key_M)
toggle_mute(); toggle_mute();
if (event.key() == Key_S)
m_stop_action->activate();
if (event.key() == Key_Up) if (event.key() == Key_Up)
m_volume_slider->increase_slider_by_page_steps(1); m_volume_slider->increase_slider_by_page_steps(1);