[#6832] Warn to user while move message to Notepad/Memo
Summary Warn to user while move message to Notepad/Memo
Queue IMP
Queue Version Git master
Type Enhancement
State Resolved
Priority 1. Low
Owners slusarz@horde.org
Requester david@tmv.gov.tw
Created 2008-06-04 (4413 days ago)
Due
Updated 2010-07-14 (3643 days ago)
Assigned 2010-04-05 (3743 days ago)
Resolved 2010-07-07 (3650 days ago)
Milestone
Patch No

Comments
david@tmv.gov.tw 2008-06-04 04:56:45
Warn to user while move message to Notepad/Memo,if message with attachments.

Thus, can prevend attachments lost permanently.

goncalo.queiros@portugalmail.net 2009-09-21 18:16:27
Hope this patch fixes this :)

Had to move some js to top of page and include a inline part, if they 
are not in their correct places, please say so.

Best regards,

Gonçalo Queirós

Michael Slusarz <slusarz@horde.org> 2009-09-25 16:48:28
This patch only deals with the mailbox page, not the message page.   
And this code will show a confirmation screen for every move action.   
That is most definitely not desired - a confirmation screen (or better 
yet, a confirmation javascript dialog box) should only be shown for 
moves to the notepad and/or memo app.

goncalo.queiros@portugalmail.net 2009-09-26 22:28:00
Tottally forgot about checking if the user is moving to nag or mnome.. 
:-/ Will fix that and the confirmation on message screen also... I did 
this with a confirmation screen and not with a javascript box, because 
Jan told me it would be better to do this way :-)

Will check it monday!

goncalo.queiros@portugalmail.net 2009-09-28 10:18:22
Ok, think it fixed now. I have kept with the middle confirmation 
screen instead of javascript box, because Jan prefers one and you 
prefer another :-) If you guys want, i can change this to javascript ;)



Git Commit <commits@lists.horde.org> 2010-07-07 05:22:42

Michael Slusarz <slusarz@horde.org> 2010-07-07 05:24:14
This is best handled with a RedBox dialog display rather than the 
extreme overhead of displaying an additional page, which would become 
quite tedious every time this action is to be performed.  Added to IMP 
5.

Chuck Hagenbuch <chuck@horde.org> 2010-07-13 23:08:38
I'm not sure that displaying another page is really extreme overhead 
given that this isn't a constant action like deleting. No problem with 
using a dialog in general, but what about when javascript is off? Do 
we still care about that case?

Michael Slusarz <slusarz@horde.org> 2010-07-14 00:22:43
> I'm not sure that displaying another page is really extreme overhead 
> given that this isn't a constant action like deleting. No problem 
> with using a dialog in general, but what about when javascript is 
> off? Do we still care about that case?

No - this would be no different than the delete without trash.  We 
don't want a confirm page every time we delete a message.  That is 
simply way too much overhead.

I agree with an intermediate page when it is something that is 
potentially EXTREMELY destructive - i.e. emptying an entire mailbox.   
But not at a message level.

And IIRC, at this point we are assuming that people have javascript 
enabled if using IMP (or any of the standard views of apps).  If not, 
they should be using mimp instead.