Bug 157467 - [PATCH] Use Save/Discard instead of Yes/No on exit dialog
Summary: [PATCH] Use Save/Discard instead of Yes/No on exit dialog
Status: RESOLVED FIXED
Alias: None
Product: kwordquiz
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR wishlist
Target Milestone: ---
Assignee: Peter Hedlund
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-02-09 01:02 UTC by David Benjamin
Modified: 2008-02-10 03:57 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:

Attachments
Use KStandardGuiItem::save(), etc. (1.07 KB, patch)
2008-02-09 01:05 UTC, David Benjamin 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description David Benjamin 2008-02-09 01:02:38 UTC 
Version:           0.8.9 (using KDE 4.0.0)
Installed from:    Compiled From Sources
OS:                Linux

(This behavior is actually consistent with other KDE apps this time. ;-) See KWrite.)

When exiting KWordQuiz on a modified document, a dialog appears. The buttons say "Yes/No". They should be labeled with verbs.

References:
KWrite
KOffice
http://wiki.openusability.org/guidelines/index.php/Guidelines:Buttons
Comment 1 David Benjamin 2008-02-09 01:05:11 UTC 
Created attachment 23484 [details]
Use KStandardGuiItem::save(), etc.

I wasn't sure whether to use discard() or dontSave(). Other KDE apps seem to
use discard(), so I guess that's the KDE convention... which makes me wonder
what the point of dontSave() is.

(The patch also includes a little change to avoid the

QLayout: Attempting to add QLayout "" to KWordQuizApp "MainWindow#1", which
already has a layout

warning that occurs at startup. I thought it wasn't worth it to open two bugs
for a one-liner each.)
Comment 2 Peter Hedlund 2008-02-10 03:57:33 UTC 
SVN commit 773063 by hedlund:

Use Save/Discard instead of Yes/No on exit. Patch by David Benjamin.

BUG:157467

 M  +2 -2      kwordquiz.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=773063