[PATCH] Switch some columns to right alignment in divelist
Davide DB
dbdavide at gmail.com
Wed Sep 3 14:54:23 PDT 2014
Anton I see what you mean..
Regarding the first point you are right. Numbers should be left aligned.
Most of my dives are > 1h.
Actually it's the same for depth. I have some three digit dives and it's wrong:
100
52
Supposed we are using a monospaced font, it should be:
100
52
Just remember that Column heading must always follow data alignment.
Regarding the http://trac.hohndel.org/ticket/561 I opened three months
ago. Look at my dive list with one trip before/after the patch. The
dive #171 is not part of the trip but on windows and Mac had the same
indentation. I hope this not break the fix.
Best
On Wed, Sep 3, 2014 at 11:12 PM, Anton Lundin <glance at acc.umu.se> wrote:
> On 03 September, 2014 - Tomaz Canabrava wrote:
>
>> Em 03/09/2014 17:34, "Anton Lundin" <glance at acc.umu.se> escreveu:
>> >
>> > On 03 September, 2014 - Tomaz Canabrava wrote:
>> >
>> > > Em 03/09/2014 17:24, "Anton Lundin" <glance at acc.umu.se> 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 <glance at acc.umu.se>
>> > > > ---
>> > > > 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.
>>
>> Can you send a picture só I can better visualize this?
>>
>> The issue on changing the NR to right Align is that its hard to say whats
>> on a trip and whats not.
>
> Here is two pictures that show what i mean.
>
> For example, look at the alignment of the duration column. When you have
> two dives besides each other where one is above 1h and one below, it
> looks really wierd:
> 35:50
> 1:03:20
>
> I say it looks way better right aligned:
> 35:50
> 1:03:20
>
> The other one is that the initial tab in the trip-number-column doesn't
> fold away, so if you lowers that column to the with of non-trip-numbers
> the trip numbers gets folded away.
>
> //Anton
>
> --
> Anton Lundin +46702-161604
>
> _______________________________________________
> subsurface mailing list
> subsurface at hohndel.org
> http://lists.hohndel.org/cgi-bin/mailman/listinfo/subsurface
>
--
Davide
https://vimeo.com/bocio/videos
-------------- next part --------------
A non-text attachment was scrubbed...
Name: trip-wrong-indent.PNG
Type: image/png
Size: 49384 bytes
Desc: not available
URL: <http://lists.hohndel.org/pipermail/subsurface/attachments/20140903/712d88df/attachment-0002.png>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: trip-indent-ok.PNG
Type: image/png
Size: 28361 bytes
Desc: not available
URL: <http://lists.hohndel.org/pipermail/subsurface/attachments/20140903/712d88df/attachment-0003.png>
More information about the subsurface
mailing list