I just tried out this patch on SVN 364 - and I think it looks great.  One minor 
annoyance is that it previews the contents of the previously selected email 
when you change folders, select multiple emails etc.  

I don't know how hard it would be to update the preview pane when an email is 
no longer 'in focus', but I think it would be a good idea if possible.

Great work so far though - I really like roundcube :)

Nathan. 

On Fri, 10 Nov 2006 16:44:11 -0500, Chris Richardson <[EMAIL PROTECTED]> wrote:
> I just applied this to our test branch here and I must say I realy like
> this good job guys.
> 
> Brett Patterson wrote:
>> Robin Elfrink wrote:

>> I mean not to sound like an ass, so if I do, I apologize.  If you want
>> to include the conditional functionality (<roundcube:if />
>> <roundcube:else >) then I might suggest looking at what SMARTY has
>> already put in place.  They're dedicated to the template system, and
>> it works well.  I'm not saying convert to SMARTY, but it is an option
>> if you so choose.  But if you wanted to add the functionality, SMARTY
>> already has it in place and it works very well.  So look at how it's
>> done elsewhere, and bring it over ;)
>>
>> Just my $0.02
>>
>> ~Brett
>>
>>
> 
> 
> 
> 
> The information transmitted is intended only for the person or entity
> to which it is addressed and may contain confidential and/or
> privileged material. Any review, retransmission, dissemination or
> other use of, or taking of any action in reliance upon, this information
> by persons or entities other than the intended recipient is prohibited.
> If you received this in error, please contact the sender and delete this
> material from any computer.
> 
> In accordance with industry regulations, all messages are retained and are
> subject to monitoring.
> 
> This message has been scanned for viruses and dangerous content and is
> believed to be clean.
> 
> Securities offered through Cantella & Co., Inc., Member NASD/SIPC.
> Home Office: 2 Oliver Street, 11th Floor, Boston, MA 02109
> Telephone: (617)521-8630



Reply via email to