diff options
| author |  Dirk Hohndel <dirk@hohndel.org> | 2014-11-23 07:41:25 -0800 |
|---|---|---|
| committer | Dirk Hohndel <dirk@hohndel.org> | 2014-11-23 07:45:17 -0800 |
| commit | bf14d3180427a8197e7dfbcb29fed1002efbbb80 (patch) | |
| tree | eea7ec3b53b0d4e3b0eeb9fe4702cdee976d360b | |
| parent | 13934b0f02a812cc5073ab21f2cbee754ecaee14 (diff) | |
| download | subsurface-bf14d3180427a8197e7dfbcb29fed1002efbbb80.tar.gz | |
Make the button labels for apply / cancel of edits more consistent
Before you could 'Cancel' the edit and then would be asked if you wanted
to 'Discard' your changes or 'Cancel'. So clicking 'Cancel' cancelled the
action of having clicked 'Cancel'. That's so confusing, it's even hard to
explain. Yes, it uses "typical" language for user interaction and kind of
makes sense, but it's not easy to understand for the non-technical user.
With the new labels the user is asked to whether they want to 'Apply' the
changes or 'Discard' them. And when they choose 'Discard' the verification
question is still 'Discard' or 'Cancel'. That seems much more consistent.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
| -rw-r--r-- | qt-ui/maintab.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/qt-ui/maintab.cpp b/qt-ui/maintab.cpp index 3c7489f73..68775c8c1 100644 --- a/qt-ui/maintab.cpp +++ b/qt-ui/maintab.cpp @@ -51,11 +51,11 @@ MainTab::MainTab(QWidget *parent) : QTabWidget(parent), ui.extraData->setModel(extraDataModel); closeMessage(); - QAction *action = new QAction(tr("Save"), this); + QAction *action = new QAction(tr("Apply changes"), this); connect(action, SIGNAL(triggered(bool)), this, SLOT(acceptChanges())); addMessageAction(action); - action = new QAction(tr("Cancel"), this); + action = new QAction(tr("Discard changes"), this); connect(action, SIGNAL(triggered(bool)), this, SLOT(rejectChanges())); QShortcut *closeKey = new QShortcut(QKeySequence(Qt::Key_Escape), this); |