Hi Alec,
thanks for comments.
I can't contribute with patches due lack of knowledge (Windows fuzzy). I will
make it to disable links if mailbox is empty.
As soon as 'invert' is part of core I'll strip 'invert' of the plugin code.
One question ... Is there any reason not to provide a default command case in
app.js? Such like ...
case 'delete-folder':
this.delete_folder(props);
break;
// Find me: API: Javascript command,
default:
if(typeof eval(command) == 'function'){
return eval(command)();
}
... would make it easier to jump in - at least for me.
Regards,
Roland
----- Original Message -----
From: "A.L.E.C" <[email protected]>
To: <[email protected]>
Sent: Saturday, May 30, 2009 7:31 PM
Subject: Re: [RCD] Feature Discussion
> Roland Liebl wrote:
>> I have coded #4 (Invert Selection and Select Range) as plugin (see
>> attachment). Perhaps not all will like this features. Any criticism is
>> welcome. I'm not the most skilled programmer!
>
> As I said previously, this should be added into core. Two points from me:
> 1. Create a patch not a plugin. Since 'range' is targeted for IE users,
> maybe we should split this: 'invert' as core feature and 'range' as plugin.
> 3. If mailbox is empty links should be disabled.
>
> --
> Aleksander 'A.L.E.C' Machniak http://alec.pl gg:2275252
> LAN Management System Developer http://lms.org.pl
> Roundcube Webmail Developer http://roundcube.net
> _______________________________________________
> List info: http://lists.roundcube.net/dev/
>_______________________________________________
List info: http://lists.roundcube.net/dev/
