diff options
| author |  Dirk Hohndel <dirk@hohndel.org> | 2013-05-02 14:05:53 -0700 |
|---|---|---|
| committer | Dirk Hohndel <dirk@hohndel.org> | 2013-05-02 14:49:56 -0700 |
| commit | c4e2c322a3f6505880b9330e577129e0110ed2ba (patch) | |
| tree | 3405495f6788522929f251e0e6ffdd74fedd3c4c /qt-ui/models.cpp | |
| parent | f9c97ff97d072d6a4cb934a50427dc69382281e0 (diff) | |
| download | subsurface-c4e2c322a3f6505880b9330e577129e0110ed2ba.tar.gz | |
Set better column widths in the dive list
This code seems rather crude to me. I'm sure this could be done better.
This also makes the column alignment work again.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
Diffstat (limited to 'qt-ui/models.cpp')
| -rw-r--r-- | qt-ui/models.cpp | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/qt-ui/models.cpp b/qt-ui/models.cpp index 68f575ec8..5c7fc7a8b 100644 --- a/qt-ui/models.cpp +++ b/qt-ui/models.cpp @@ -434,7 +434,8 @@ QVariant DiveItem::data(int column, int role) const { QVariant retVal; - if (role == Qt::TextAlignmentRole) { + switch (role) { + case Qt::TextAlignmentRole: switch (column) { case DATE: /* fall through */ case SUIT: /* fall through */ @@ -445,9 +446,8 @@ QVariant DiveItem::data(int column, int role) const retVal = Qt::AlignRight; break; } - } - - if (role == Qt::DisplayRole) { + break; + case Qt::DisplayRole: switch (column) { case NR: retVal = dive->number; @@ -492,6 +492,7 @@ QVariant DiveItem::data(int column, int role) const retVal = dive->rating; break; } + break; } return retVal; } @@ -598,8 +599,6 @@ QVariant DiveTripModel::data(const QModelIndex& index, int role) const if (!index.isValid()) return QVariant(); - if (role != Qt::DisplayRole) - return QVariant(); TreeItemDT* item = static_cast<TreeItemDT*>(index.internalPointer()); |