Funkwhale/Clementine-audio-player-Mac-Linux-Windows
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Force the position of the checkbox for the confirmation prompt (looked bad on Windows) + update an old, not up to date comment

Browse Source
This commit is contained in:
Arnaud Bienner 2013-08-11 16:36:07 +02:00
parent 1c2f4be185
commit 379b8743f9
1 changed files with 20 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
src/playlist/playlisttabbar.cpp
View File

@ -28,6 +28,8 @@
#include <QCheckBox> #include <QCheckBox>
#include <QContextMenuEvent> #include <QContextMenuEvent>
#include <QDialogButtonBox>
#include <QGridLayout>
#include <QInputDialog> #include <QInputDialog>
#include <QMenu> #include <QMenu>
#include <QMessageBox> #include <QMessageBox>
@ -181,7 +183,20 @@ void PlaylistTabBar::Close() {
dont_prompt_again.blockSignals(true); dont_prompt_again.blockSignals(true);
dont_prompt_again.setToolTip( dont_prompt_again.setToolTip(
tr("This option can be changed in the \"Behavior\" preferences")); tr("This option can be changed in the \"Behavior\" preferences"));
confirmation_box.addButton(&dont_prompt_again, QMessageBox::ActionRole);
QGridLayout* grid = qobject_cast<QGridLayout*>(confirmation_box.layout());
QDialogButtonBox* buttons = confirmation_box.findChild<QDialogButtonBox*>();
if (grid && buttons) {
const int index = grid->indexOf(buttons);
int row, column, row_span, column_span = 0;
grid->getItemPosition(index, &row, &column, &row_span, &column_span);
QLayoutItem* buttonsItem = grid->takeAt(index);
grid->addWidget(&dont_prompt_again, row, column, row_span, column_span,
Qt::AlignLeft | Qt::AlignTop);
grid->addItem(buttonsItem, ++row, column, row_span, column_span);
} else {
confirmation_box.addButton(&dont_prompt_again, QMessageBox::ActionRole);
}
if (confirmation_box.exec() != QMessageBox::Yes) { if (confirmation_box.exec() != QMessageBox::Yes) {
return; return;
@ -193,8 +208,10 @@ void PlaylistTabBar::Close() {
} }
} }
// Just hide the tab from the UI - don't delete it completely (it can still // Close the playlist. If the playlist is not a favorite playlist, it will be
// be resurrected from the Playlists tab). // deleted, as it will not be visible after being closed. Otherwise, the tab
// is closed but the playlist still exists and can be resurrected from the
// "Playlists" tab.
emit Close(playlist_id); emit Close(playlist_id);
// Select the nearest tab. // Select the nearest tab.