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):
1) Open folder that contains a bit more messages then could be shown on
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/

Reply via email to