Improve QDialogButtonBox::Yes override text
ClosedPublic

Authored by elvisangelaccio on Apr 2 2018, 9:12 PM.

Details

Summary

We are asking the users whether they are sure they want to quit.

But the "Quit" button can be ambiguous (am I quitting dolphin or just
this dialog?). If we use a "Quit Dolphin" button instead, that clearly
shows what will happen after triggering the action.

Test Plan

Close dolphin while multiple tabs are open.

Diff Detail

Repository
R318 Dolphin
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
elvisangelaccio requested review of this revision.Apr 2 2018, 9:12 PM
elvisangelaccio created this revision.
elvisangelaccio edited the summary of this revision. (Show Details)Apr 2 2018, 9:15 PM
ngraham requested changes to this revision.EditedApr 2 2018, 9:35 PM
ngraham added a subscriber: ngraham.

Sorry, I can't approve of this change. Button text should (almost) always use verbs instead of "Yes" or "No" because many users don't read the explanatory text, just the buttons. A Button that says "Quit" makes it clear what's about to happen if you didn't read the explanatory text; a button that says "Yes" does not.

For evidence of this phenomenon and more information on the subject, see:

As for possible confusion over what's being "quit", I'm not sure that's a problem. Windows are closed; apps are quit. Our own menu items reflect this distinction and I don't think many users are confused by what's going to happen when they click a Quit button. If you think it's really a problem, we should make the button say "Quit Dolphin" instead of just "Quit."

This revision now requires changes to proceed.Apr 2 2018, 9:35 PM

Right. How about "Close All Tabs"? But I'd also be ok with "Quit Dolphin".

As for possible confusion over what' being "quit", I'm not sure that's a problem. Windows are closed; apps are quit.

As a non-native speaker who runs dolphin in english, the distinction is not so clear...

elvisangelaccio planned changes to this revision.Apr 2 2018, 9:48 PM

I'd prefer "Quit Dolphin", since then it also teaches the user that "Quit" it something that applies to apps, if they're not already familiar with this concept.

  • Yes -> Quit Dolphin
elvisangelaccio retitled this revision from Don't override QDialogButtonBox::Yes text to Improve QDialogButtonBox::Yes override.Apr 3 2018, 9:23 PM
elvisangelaccio edited the summary of this revision. (Show Details)

New "After" screenshot:

elvisangelaccio retitled this revision from Improve QDialogButtonBox::Yes override to Improve QDialogButtonBox::Yes override text.Apr 3 2018, 9:23 PM
ngraham accepted this revision.Apr 3 2018, 9:24 PM

+1 ship it!

This revision is now accepted and ready to land.Apr 3 2018, 9:24 PM
This revision was automatically updated to reflect the committed changes.