On Oct 17, 2007, at 6:58 PM, Thomas Bruederli wrote:

> Your patch looks good so far. The only thing I'd like to add is to set
> the UI busy while submitting the rename request. That simply means to
> set the third parameter of this.http_post('rename-folder') to true.
> Please add this to your patch and feel free to commit it.

Will do.


> One more thing to mention is the warning/information to display when
> adding a new folder and having one folder selected. I'm not sure if
> the user is aware that the new folder will be created as a subfolder
> of the selected one. Please correct me if I'm wrong with this
> assumption.

I thought of that. I think, as a user, I'd rather have a bit of help  
text somewhere, than a popup coming every time I do something.


Robin

_______________________________________________
List info: http://lists.roundcube.net/dev/

Reply via email to