[haiku-commits] Re: r40747 - haiku/trunk/src/apps/mail

  • From: Clemens <clemens.zeidler@xxxxxxxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Fri, 04 Mar 2011 11:53:18 +1300

On Wed, 02 Mar 2011 02:47:31 +1300, Axel Dörfler <axeld@xxxxxxxxxxxxxxxx> wrote:


Stephan Aßmus <superstippi@xxxxxx> wrote:
>> BTW, is it possible to get information about the current sorting
> > of
>> the Tracker window? I have usually set up the sorting by date, but
>> in reverse order, i.e. newest mails on top. I can't get to the
>> "logical" next mail via Mail, unless I press the "previous mail"
>> button. :-)
> good point, but as far as I can see from http://haiku.it.su.se:
> 8180/source/xref/src/kits/tracker/PoseViewScripting.cpp
>  there is no support for this at the moment...
If Mail just sends commands like "Next" or "Previous" pose, it could
be fixed in Tracker, but I have no idea how it works behind the scene.

Before anyone does changes like this: I am not sure it's really a good
idea. How could Tracker know that the reverse sorting order also means
that the Next/Previous logic should be reversed? That might be
convenient for Mail, but it would be pretty confusing for any other
piece of software, like ShowImage.

Therefore, I would just extend the scripting feature, and use that
extra information within Mail to decide what to do (ie. if a reversed
"MAIL:when" or "Modified time" is part of the sort order, reverse the
logic of next/previous).


I thought about it and I would just keep it as it is. You simply don't know what the user want. If somebody sort mails by subject it would be odd if the next mail would be selected by date... Taking the sort direction into account would also be more confusing then helping IMHO. Pressing next button and you getting the prev message in tracker is strange.

This two buttons are tightly connected to a special tracker window so they do exactly their job: navigating in the tracker window. Maybe the label could be improved or the icon could reflect that it means next/prev in tracker...

cheers,
        Clemens

Other related posts: