Skip to content

Commit c772e6f

Browse files
committed
Track File Export: add 'Apply to all' checkbox, remove ".. All" buttons
1 parent 8790b5f commit c772e6f

1 file changed

Lines changed: 18 additions & 13 deletions

File tree

src/library/export/trackexportdlg.cpp

Lines changed: 18 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
#include "library/export/trackexportdlg.h"
22

3+
#include <QCheckBox>
34
#include <QMessageBox>
45

56
#include "moc_trackexportdlg.cpp"
@@ -66,30 +67,34 @@ void TrackExportDlg::slotAskOverwriteMode(
6667
std::promise<TrackExportWorker::OverwriteAnswer>* promise) {
6768
QMessageBox question_box(
6869
QMessageBox::Warning,
69-
tr("Overwrite Existing File?"),
70-
tr("\"%1\" already exists, overwrite?").arg(filename),
70+
tr("Replace Existing File?"),
71+
tr("\"%1\" already exists, replace?").arg(filename),
7172
QMessageBox::Cancel);
7273

7374
QPushButton* pSkip = question_box.addButton(
7475
tr("&Skip"), QMessageBox::NoRole);
75-
QPushButton* pSkipAll = question_box.addButton(
76-
tr("Skip &All"), QMessageBox::NoRole);
7776
QPushButton* pOverwrite = question_box.addButton(
78-
tr("&Overwrite"), QMessageBox::YesRole);
79-
QPushButton* pOverwriteAll = question_box.addButton(
80-
tr("Over&write All"), QMessageBox::YesRole);
77+
tr("&Replace"), QMessageBox::YesRole);
8178
question_box.setDefaultButton(pSkip);
8279

80+
QCheckBox* pApplyToAll = new QCheckBox("Apply to all files");
81+
pApplyToAll->setChecked(false);
82+
question_box.setCheckBox(pApplyToAll);
83+
8384
question_box.exec();
8485
auto* pBtn = question_box.clickedButton();
8586
if (pBtn == pSkip) {
86-
promise->set_value(TrackExportWorker::OverwriteAnswer::SKIP);
87-
} else if (pBtn == pSkipAll) {
88-
promise->set_value(TrackExportWorker::OverwriteAnswer::SKIP_ALL);
87+
if (pApplyToAll->isChecked()) {
88+
promise->set_value(TrackExportWorker::OverwriteAnswer::SKIP_ALL);
89+
} else {
90+
promise->set_value(TrackExportWorker::OverwriteAnswer::SKIP);
91+
}
8992
} else if (pBtn == pOverwrite) {
90-
promise->set_value(TrackExportWorker::OverwriteAnswer::OVERWRITE);
91-
} else if (pBtn == pOverwriteAll) {
92-
promise->set_value(TrackExportWorker::OverwriteAnswer::OVERWRITE_ALL);
93+
if (pApplyToAll->isChecked()) {
94+
promise->set_value(TrackExportWorker::OverwriteAnswer::OVERWRITE_ALL);
95+
} else {
96+
promise->set_value(TrackExportWorker::OverwriteAnswer::OVERWRITE);
97+
}
9398
} else {
9499
// Cancel
95100
promise->set_value(TrackExportWorker::OverwriteAnswer::CANCEL);

0 commit comments

Comments
 (0)