Commit 9d00b2f9 authored by Friedrich W. H. Kossebau's avatar Friedrich W. H. Kossebau
Browse files

Open >5 terminals dialog: use action terms instead of Yes/No buttons

Yes/No buttons in dialogs are discouraged (not only) by KDE HIG in favour
of actions terms.

(cherry picked from commit c6f82aa3)
parent 93b71e84
Pipeline #204344 passed with stage
in 10 minutes and 59 seconds
......@@ -1119,7 +1119,10 @@ void DolphinMainWindow::openTerminalHere()
if (urls.count() > 5) {
QString question = i18np("Are you sure you want to open 1 terminal window?",
"Are you sure you want to open %1 terminal windows?", urls.count());
const int answer = KMessageBox::warningYesNo(this, question);
const int answer = KMessageBox::warningYesNo(this, question, {},
KGuiItem(i18ncp("@action:button", "Open %1 Terminal", "Open %1 Terminals", urls.count()),
QStringLiteral("utilities-terminal")),
KStandardGuiItem::cancel());
if (answer != KMessageBox::Yes) {
return;
}
......
Supports Markdown
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