After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 540160 - Deletion of N multiple emails re-sets cursor down N emails in list
Deletion of N multiple emails re-sets cursor down N emails in list
Status: RESOLVED FIXED
Product: evolution
Classification: Applications
Component: Mailer
2.24.x (obsolete)
Other All
: Normal minor
: ---
Assigned To: Milan Crha
Evolution QA team
: 269854 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2008-06-25 14:30 UTC by Reid Thompson
Modified: 2013-09-13 00:59 UTC
See Also:
GNOME target: ---
GNOME version: 2.21/2.22


Attachments
proposed evo patch (4.50 KB, patch)
2008-06-25 15:24 UTC, Milan Crha
committed Details | Review

Description Reid Thompson 2008-06-25 14:30:04 UTC
Please describe the problem:
Select multiple emails for deletion where there are both older and newer
emails around the selection.  Delete the emails.  Note that instead of
just moving to the next newest email, the 'cursor' moves down multiple
emails ( i've not actually done a count so this may be wrong, but it
looks like the cursor may be jumping down n emails where n is the number
of emails just deleted)

I did some testing, and it does appear that for me, the next selected
email is N down the tree where N is the number of selected emails
deleted.

Steps to reproduce:
1. See description
2. 
3. 


Actual results:
See description

Expected results:
Next email in list is selected, rather than N down email selected

Does this happen every time?
pretty much

Other information:
building from svn head
Comment 1 Milan Crha 2008-06-25 14:35:51 UTC
Really my fault, when reverting patch from bug #337160 then it works like before.
Comment 2 Milan Crha 2008-06-25 15:24:39 UTC
Created attachment 113403 [details] [review]
proposed evo patch

for evolution;

I would never expect that this change can cause such behavior. Anyway, it does.
I also noticed that sometimes the cursor message has been forgotten between starts, it was probably also because of that patch.

Also read my comment in the code.
Comment 3 Srinivasa Ragavan 2008-06-25 17:02:56 UTC
I haven't tested it. Just put it trunk.
Comment 4 Milan Crha 2008-06-26 08:40:39 UTC
Committed to trunk. Committed revision 35684.

Reid, please reopen is you see the problem even with this revision (ideally you should not). Thanks.
Comment 5 Milan Crha 2008-06-26 08:41:20 UTC
s/is/if
Comment 6 Milan Crha 2010-05-13 10:39:19 UTC
*** Bug 269854 has been marked as a duplicate of this bug. ***