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 |
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?
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.
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?
State ⇒ Resolved
extreme overhead of displaying an additional page, which would become
quite tedious every time this action is to be performed. Added to IMP
5.
Bug #6832: Add confirm dialog when moving to notepad/tasklisthttp://git.horde.org/diff.php/imp/js/mailbox.js?rt=horde-git&r1=4b600ba8f7806b634cb6a0596e69839f5009f765&r2=25e4e9f441d22b3df2b6f6396ff8b4be3063695c
http://git.horde.org/diff.php/imp/js/message.js?rt=horde-git&r1=4ff6763fddf80477f6e7d5c169bedf33cbf8e6ec&r2=25e4e9f441d22b3df2b6f6396ff8b4be3063695c
http://git.horde.org/diff.php/imp/mailbox.php?rt=horde-git&r1=4b600ba8f7806b634cb6a0596e69839f5009f765&r2=25e4e9f441d22b3df2b6f6396ff8b4be3063695c
http://git.horde.org/diff.php/imp/message.php?rt=horde-git&r1=5f6adf20fe681b1d040cbaa824292993daae303d&r2=25e4e9f441d22b3df2b6f6396ff8b4be3063695c
http://git.horde.org/diff.php/imp/templates/imp/javascript_defs.php?rt=horde-git&r1=9c078bc25ad56f126dafab18d31b8bfc594fa44a&r2=25e4e9f441d22b3df2b6f6396ff8b4be3063695c
New Attachment: bug6832[1].diff
screen instead of javascript box, because Jan prefers one and you
prefer another :-) If you guys want, i can change this to javascript ;)
:-/ 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!
State ⇒ Feedback
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.
State ⇒ Assigned
Version ⇒ Git master
New Attachment: bug6832.diff
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
Priority ⇒ 3. High
Priority ⇒ 1. Low
State ⇒ New
Patch ⇒ No
Milestone ⇒
Summary ⇒ Warn to user while move message to Notepad/Memo
Type ⇒ Enhancement
Queue ⇒ IMP
Thus, can prevend attachments lost permanently.