[prev in list] [next in list] [prev in thread] [next in thread] 

List:       horde-dev
Subject:    Re: [dev] Re: [cvs] commit: ansel ecard.php
From:       Jan Schneider <jan () horde ! org>
Date:       2004-08-30 18:57:40
Message-ID: 20040830205740.skggk4w8oo8s4ks0 () jan ! dip ! ammma ! net
[Download RAW message or body]

Zitat von Michael M Slusarz <slusarz@bigworm.curecanti.org>:

> Quoting Jan Schneider <jan@horde.org>:
>
>> jan         2004-08-30 03:17:02 PDT
>>
>>   Modified files:
>>     .                    ecard.php
>>   Log:
>>   Set charset of HTML mime part and don't reload opener window (was there a
>>   reason for this?)
>
> I think I stuck it in there because I was going to add a success notification
> message, and we need the reload to display it in the opener window.  
> Never got
> around to creating the success notification obviously, and am not sure
> if it is
> necessary and/or desired.

I removed it because I was testing over a slow line, and reloading the page
was kind of annoying in that situation. A notification might make sense,
but at least we have one already if anything went wrong. And even in IMP we
didn't have success notifications for a long time if this is comparable.

Jan.

--
Do you need professional PHP or Horde consulting?
http://horde.org/consulting/

-- 
Horde developers mailing list - Join the hunt: http://horde.org/bounties/
Frequently Asked Questions: http://horde.org/faq/
To unsubscribe, mail: dev-unsubscribe@lists.horde.org
[prev in list] [next in list] [prev in thread] [next in thread]