[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kmail-devel
Subject:    Re: New action: move to next unread message or folder
From:       Ingo =?iso-8859-1?q?Kl=F6cker?= <kloecker () kde ! org>
Date:       2003-07-18 21:16:48
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


On Friday 18 July 2003 01:59, David Smith wrote:
> On Thursday 17 July 2003 00.20, Ingo Klöcker wrote:
> > On Wednesday 16 July 2003 21:09, David Smith wrote:
> > > I just started to use KMail and this is a feature I was missing.
> > > An action that sort of combines "Next unread message" and "Next
> > > unread text", that is, goes to the next unread message but
> > > continues to the next folder.
> >
> > Instead of adding another action we should simply make the strategy
> > for "Next unread message" configurable. We already have the option
> > [ ] Loop in the current folder when trying to find unread messages
> >
> > This could be replaced with a combo box with the options:
> >
> > When trying to find unread messages:
> > - Don't loop
> > - Loop in the current folder
> > - Loop over all folders
>
> Hmm, ok, how about this patch then? It's a bit more intrusive, but it
> does exactly that. It applies to previous and next unread message and
> also to "next unread text" (i.e. if you anything but loop over all
> folders, "next unread text" will be different. I wasn't sure about
> that one, but it seemed right to me.)

Thanks for the patch. It looks good. In the last line of the QWhatsThis 
is a typo (depeding -> depending). Furthermore please add "diff -u3 -p" 
to your ~/.cvsrc. The -p will make your diffs slightly better readable. 
Do you have a cvs account? Or should I commit the patch?

A little thing is still missing. We'll have to write a kconf_update 
script which upgrades the boolean value of LoopOnGotoUnread to an 
integer.

Regards,
Ingo

[Attachment #5 (application/pgp-signature)]

_______________________________________________
KMail Developers mailing list
kmail@mail.kde.org
http://mail.kde.org/mailman/listinfo/kmail


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic