3 messages in net.sourceforge.lists.courier-sqwebmailRe: [sqwebmail] "Add/Remove attachmen...
FromSent OnAttachments
Pawel TeczaJun 5, 2006 2:52 am.gz
Sam VarshavchikJun 6, 2006 6:08 pm 
Pawel TeczaJun 7, 2006 1:52 am 
Actions with this message:
Paste this link in email or IM:
Paste this link in email or IM:
Atom feed for this thread
Paste this URL into your reader:
Subject:Re: [sqwebmail] "Add/Remove attachment" button at "Create Message" pageActions...
From:Sam Varshavchik (mrs@courier-mta.com)
Date:Jun 6, 2006 6:08:15 pm
List:net.sourceforge.lists.courier-sqwebmail

Pawel Tecza writes:

Hello Sam,

This patch improves a bit the button which uploads attachments. It no longer displays a number of attachments on the button, but like below:

Attachments: X [ Add/Remove ]

I think this button should has a label with action phrase, the same like rest buttons. Moreover, it's probably easier to explain to a new user how to work with attachments when he can see what a button does.

Sigh…

Clearly you don't even bother to test your own patches:

-[#$ATTACHMENTS=%d attachments#] +[#$ATTACHMENTS=Attachments:#] +[#UPLOAD=Add/Remove#] [#$SALUTATION=%s writes:#]

Now, I just wasted ten or so minutes chasing down this typo. Granted, it's not much, but I already must find available time to allocate for this, so I want to put it into as much of a good use as I can, and this kind of thing does not help. Especially when all you had to do was apply your own patch, and see that the attachment button stops working.