Commit 83cb227e authored by Michael Pyne's avatar Michael Pyne
Browse files

playqueue: Use an appropriate icon for rebadged "remove playlist" action.

A JuK user pointed out that the Play Queue context menu shows "Hide"
instead of "Remove Playlist" on the context menu, but that the icon
still looks like a trash can, which is not appropriate for "Hide".

I wasn't sure of a great icon to use either, but "list-remove" at least
does not imply that the Play Queue is being tossed in the rubbish.

BUG:336250
FIXED-IN:21.04
parent 1c698b3a
Pipeline #55269 passed with stage
in 10 minutes and 29 seconds
......@@ -88,14 +88,14 @@ PlaylistBox::PlaylistBox(PlayerManager *player, QWidget *parent, QStackedWidget
m_contextMenu = new QMenu(this);
m_contextMenu->addAction( action("file_new") );
m_contextMenu->addAction( action("renamePlaylist") );
m_contextMenu->addAction( action("editSearch") );
m_contextMenu->addAction( action("duplicatePlaylist") );
m_contextMenu->addAction( action("reloadPlaylist") );
m_contextMenu->addAction( action("deleteItemPlaylist") );
m_contextMenu->addAction( action("file_save") );
m_contextMenu->addAction( action("file_save_as") );
m_contextMenu->addAction(action("file_new"));
m_contextMenu->addAction(action("renamePlaylist"));
m_contextMenu->addAction(action("editSearch"));
m_contextMenu->addAction(action("duplicatePlaylist"));
m_contextMenu->addAction(action("reloadPlaylist"));
m_contextMenu->addAction(action("deleteItemPlaylist"));
m_contextMenu->addAction(action("file_save"));
m_contextMenu->addAction(action("file_save_as"));
m_contextMenu->addSeparator();
......@@ -609,10 +609,14 @@ void PlaylistBox::slotPlaylistChanged()
if(singlePlaylist) {
PlaylistCollection::raise(playlists.front());
if(playlists.front() == upcomingPlaylist())
if(playlists.front() == upcomingPlaylist()) {
action("deleteItemPlaylist")->setText(i18n("Hid&e"));
else
action("deleteItemPlaylist")->setIcon(QIcon::fromTheme("list-remove"));
}
else {
action("deleteItemPlaylist")->setText(i18n("R&emove"));
action("deleteItemPlaylist")->setIcon(QIcon::fromTheme("user-trash"));
}
}
else if(!playlists.isEmpty())
createDynamicPlaylist(playlists);
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment