Hi Eric, Hi all,
i forgott the video here is the missing file sorry i have this only as wmv.
*Detachments* -The following attachments have been detached and are available for viewing.
* http://detached.gigo.com/rc/wR/wBRA9pUu/07122007-1.wmv
I'm pretty sure that I'm able to reproduce this on ALL recent roundcube versions (at least from 0.3-beta) WITHOUT thread patch applied.
How to reproduce (I tried to nail it down to the simplest case that shows error):
a single page (f.e. 41 message with 40 rows limit). 2) Select and delete any number of messages, but not one and not all. You may do it with either method (mouse or keyboard). 3) Try to select all messages on a page after remaining messages are loaded to list. Artifact is shown. You are unable to select one of messages (which has duplicate in another row).
Thus, I'd say that bug shown in Eric's video is not related to a threaded-view patch and is a separate long-standing issue.
Hope this will help to fix this.
Should I open a ticket in trac? Or someone can find erroneous code quite quickly?
Eric, would you try to check it again on a clean RC installation without threading patch?
Vladislav.
BTW, deleted message rows are not removed from a DOM model, they only switched to a hidden state. I'm not sure if this is an expected behavior. Can they ever be removed from the inside of JS?
List info: http://lists.roundcube.net/dev/