On 03 September, 2014 - Tomaz Canabrava wrote: > Em 03/09/2014 17:24, "Anton Lundin" <[email protected]> escreveu: > > > > Some columns in the dive list makes more sense to have right aligned > > than left aligned. This switches the numeric columns to right alignment > > so they is more easily compared visually. > > > > Signed-off-by: Anton Lundin <[email protected]> > > --- > > qt-ui/models.cpp | 21 ++++++++++++++++++++- > > 1 file changed, 20 insertions(+), 1 deletion(-) > > > > diff --git a/qt-ui/models.cpp b/qt-ui/models.cpp > > index 5266130..872f395 100644 > > --- a/qt-ui/models.cpp > > +++ b/qt-ui/models.cpp > > @@ -1070,7 +1070,26 @@ QVariant DiveItem::data(int column, int role) const > > > > switch (role) { > > case Qt::TextAlignmentRole: > > - retVal = int(Qt::AlignLeft | Qt::AlignVCenter); > > + switch (column) { > > + case NR: > > + case DEPTH: > > + case DURATION: > > + case TEMPERATURE: > > + case TOTALWEIGHT: > > + case SAC: > > + case OTU: > > + case MAXCNS: > > + retVal = int(Qt::AlignRight | Qt::AlignVCenter); > > + break; > > + case DATE: > > + case RATING: > > + case SUIT: > > + case CYLINDER: > > + case GAS: > > + case LOCATION: > > + retVal = int(Qt::AlignLeft | Qt::AlignVCenter); > > + break; > > + } > > break; > > case DiveTripModel::SORT_ROLE: > > Q_ASSERT(dive != NULL); > > -- > > 1.9.1 > > > > > > Nope. > NR is one of the columns that this doesn't works because of the way that > the treeview works. There is actually a commit from me reverting this a > while ago. >
On my system i have a visual tab on all the trip-child-dives, and right-aligning the NR column makes the non-trip-dive-numbers align with the trip-dive-numbers. I would prefer if we could get rid of that implicit tab on trip dives. It just makes things look inconsistent if you lower the with for the NR column. //Anton -- Anton Lundin +46702-161604 _______________________________________________ subsurface mailing list [email protected] http://lists.hohndel.org/cgi-bin/mailman/listinfo/subsurface
