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

List:       kmail-devel
Subject:    [Bug 61891] Threading by subject and duplicates
From:       Ingo "Klöcker" <kloecker () kde ! org>
Date:       2003-07-31 11:40:53
[Download RAW message or body]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
     
http://bugs.kde.org/show_bug.cgi?id=61891     
kloecker@kde.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
      everconfirmed|0                           |1



------- Additional Comments From kloecker@kde.org  2003-07-31 13:40 -------
Currently KMail adds messages with subject "Re: foo" below the first message 
with subject "foo" (when threading by subject). I agree that it would make 
much more sense to add the "Re: foo" message below the "foo" message which has 
the most recent date compared to the date of the "Re: foo" message. 
 
Example: 
That's how it should be: 
+ foo          2003-07-30 
+-- Re: foo   2003-07-31 
... 
+ foo          2003-05-05 
+-- Re: foo   2003-05-07 
 
And that's how it's currently done (at least if one sorts by date from most 
recent to oldest): 
+ foo          2003-07-30 
... 
+ foo          2003-05-05 
+-- Re: foo   2003-07-31 
+-- Re: foo   2003-05-07 
 
The problem is that this doesn't work with the currently used simple usage of 
a QDict<md5sum_of_message_id, KMHeaderItem> because "Multiple items can have 
the same key, in which case only the last item will be accessible using 
operator[]()." (cf. http://doc.trolltech.com/3.2/qdict.html#insert). So we 
probably have to use a QDict<md5sum_of_message_id, QPtrList<KMHeaderItem> > 
instead.
_______________________________________________
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