Hi all,
Regarding the chrome issue described on ticket http://trac.roundcube.net/ticket/1486177. I'm pretty sure that It's a race condition in
this.sent_successfully = function(type, msg) { this.list_mailbox(); this.display_message(msg, type, true); };
If the async call that pulls the mail list loads fast enough, it will make the success message to not show up because the iframe that triggered it is gone. In some browsers like FF, it used to trigger an internal exception if the element where an event was triggered is removed from DOM. The patch is not elegant (because race condition sucks) but seems to work fine tho.
On the other hand, the opera mobile issue described in here http://trac.roundcube.net/ticket/1486777 I don't think it's something related to roundcube code. I was playing with the opera emulator + dragonfly and looks like opera breaks the json output (parser issue?) that is correctly formatted in the http get response. I attached my findings to the ticket.
By the way, I'm not sure what tickets to look from the list. I'm messing around with tickets for milestone 0.4.2 randomly.
P.S. my nick in the bug tracker is neek
Regards, Adolfo.
--- 8< --- detachments --- 8< --- The following attachments have been detached and are available for viewing. http://detached.gigo.com/rc/nh/sQRuQAwW/app.js.patch Only click these links if you trust the sender, as well as this message. --- 8< --- detachments --- 8< ---
List info: http://lists.roundcube.net/dev/ BT/aba52c80
On 05.10.2010 05:24, Adolfo Castro Menna wrote:
Regarding the chrome issue described on ticket http://trac.roundcube.net/ticket/1486177. I'm pretty sure that It's a race condition in
Please, attach patches to tickets too. Just to not forget.
By the way, I'm not sure what tickets to look from the list. I'm messing around with tickets for milestone 0.4.2 randomly.
Most bugs are assigned to 0.4.2 (after release they will be moved to next, probably 0.5-beta), so if you'd like to fix bugs, check tickets assigned to 0.4.2. If you'd like to implement new features you can do it in any order.