diff options
| author |  Berthold Stoeger <bstoeger@mail.tuwien.ac.at> | 2020-09-12 18:50:06 +0200 |
|---|---|---|
| committer |  Dirk Hohndel <dirk@hohndel.org> | 2020-09-12 10:36:36 -0700 |
| commit | 6ea0cc62b8c7b01c37fdfbe50e3cefde4c43d360 (patch) | |
| tree | 603db8f3ec9ec30fb47eb4064d09a9257c2e436e /qt-models | |
| parent | 72312bec2f4dcde3150a2d989bd729237c704d66 (diff) | |
| download | subsurface-6ea0cc62b8c7b01c37fdfbe50e3cefde4c43d360.tar.gz | |
desktop: refine auto-fill of weights
In a previous commit, auto-filling of weight based on type was
changed to be only performed if the user hadn't already set a
weight, by testing for weight=0.
However, when the user edited the type and tabbed back and forth,
that counted as an edit and therefore the weight would not
change anymore.
To refine this, introduce an "auto_filled" flag to the weightsystem,
which is set if the weight is automatically filled and cleared if
the weight is edited. Update the weight if it was zero *or* auto-filled.
The flag is not saved to disk, but that should be acceptable. If the
user saves and reloads, we can assume that they meant the weight
to be set to the default value.
Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
Diffstat (limited to 'qt-models')
| -rw-r--r-- | qt-models/weightmodel.cpp | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/qt-models/weightmodel.cpp b/qt-models/weightmodel.cpp index 020e14c24..7e89f30bf 100644 --- a/qt-models/weightmodel.cpp +++ b/qt-models/weightmodel.cpp @@ -94,8 +94,10 @@ void WeightModel::setTempWS(int row, weightsystem_t ws) tempWS = ws; // If the user had already set a weight, don't overwrite that - if (oldWS.weight.grams) + if (oldWS.weight.grams && !oldWS.auto_filled) tempWS.weight = oldWS.weight; + else + tempWS.auto_filled = true; } dataChanged(index(row, TYPE), index(row, WEIGHT)); } @@ -133,6 +135,7 @@ bool WeightModel::setData(const QModelIndex &index, const QVariant &value, int r switch (index.column()) { case WEIGHT: ws.weight = string_to_weight(qPrintable(vString)); + ws.auto_filled = false; int count = Command::editWeight(index.row(), ws, false); emit divesEdited(count); return true; |