6.0.0-beta1
7/8/25

[#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 (at) horde (dot) org
Requester david (at) tmv (dot) gov (dot) tw
Created 06/04/2008 (6243 days ago)
Due
Updated 07/14/2010 (5473 days ago)
Assigned 04/05/2010 (5573 days ago)
Resolved 07/07/2010 (5480 days ago)
Milestone
Patch No

History
07/14/2010 12:22:43 AM Michael Slusarz Comment #9 Reply to this comment
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.
07/13/2010 11:08:38 PM Chuck Hagenbuch Comment #8 Reply to this comment
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?
07/07/2010 05:24:14 AM Michael Slusarz Comment #7
State ⇒ Resolved
Reply to this comment
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.
04/05/2010 01:20:53 PM Jan Schneider State ⇒ Assigned
 
04/05/2010 01:20:39 PM Jan Schneider Deleted Original Message
 
09/28/2009 10:18:22 AM goncalo (dot) queiros (at) portugalmail (dot) net Comment #5
New Attachment: bug6832[1].diff Download
Reply to this comment
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 ;)


09/28/2009 04:13:12 AM Michael Slusarz Priority ⇒ 1. Low
 
09/26/2009 10:28:00 PM goncalo (dot) queiros (at) portugalmail (dot) net Comment #4 Reply to this comment
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!
09/25/2009 04:48:28 PM Michael Slusarz Comment #3
State ⇒ Feedback
Reply to this comment
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.
09/25/2009 12:34:47 PM Jan Schneider Assigned to Michael Slusarz
State ⇒ Assigned
Version ⇒ Git master
 
09/21/2009 06:16:27 PM goncalo (dot) queiros (at) portugalmail (dot) net Comment #2
New Attachment: bug6832.diff
Reply to this comment
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
06/04/2008 09:26:14 AM Jan Schneider State ⇒ Accepted
Priority ⇒ 3. High
 
06/04/2008 04:56:45 AM david (at) tmv (dot) gov (dot) tw Comment #1
Priority ⇒ 1. Low
State ⇒ New
Patch ⇒ No
Milestone ⇒
Summary ⇒ Warn to user while move message to Notepad/Memo
Type ⇒ Enhancement
Queue ⇒ IMP
Reply to this comment
Warn to user while move message to Notepad/Memo,if message with attachments.

Thus, can prevend attachments lost permanently.

Saved Queries