Tickets :: [#12875] IE11 attachment adding problem

6.0.0-beta1

7/12/25

Summary	IE11 attachment adding problem
Queue	IMP
Queue Version	6.1.6
Type	Bug
State	Resolved
Priority	1. Low
Owners	slusarz (at) horde (dot) org
Requester	vmkari (at) cc (dot) helsinki (dot) fi
Created	12/04/2013 (4238 days ago)
Due
Updated	03/04/2014 (4148 days ago)
Assigned	12/05/2013 (4237 days ago)
Resolved	03/04/2014 (4148 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	No

03/04/2014 07:25:38 PM	Git Commit	Comment #12	Reply to this comment
Changes have been made in Git (master): commit a7ed48b91d0e10de6d4df386c1db8261c3910b39 Author: Michael M Slusarz <slusarz@horde.org> Date: Tue Mar 4 12:12:46 2014 -0700 ~~Bug #12875~~: This causes problems in IE11, and doesn't appear to be needed anymore imp/js/compose-dimp.js \| 2 -- 1 files changed, 0 insertions(+), 2 deletions(-) http://git.horde.org/horde-git/-/commit/a7ed48b91d0e10de6d4df386c1db8261c3910b39

03/04/2014 07:15:07 PM	Michael Slusarz	Comment #11	Reply to this comment
The u.clear() call was needed at some point. But it doesn't seem to effect anything anymore, and it does fix IE 11. So I will remove it.

03/04/2014 07:14:42 PM	Git Commit	Comment #10	Reply to this comment
Changes have been made in Git (FRAMEWORK_5_1): commit 5ea9f6e9776b848684f439e2b49515f7fc3b11b8 Author: Michael M Slusarz <slusarz@horde.org> Date: Tue Mar 4 12:12:46 2014 -0700 ~~Bug #12875~~: This causes problems in IE11, and doesn't appear to be needed anymore imp/js/compose-dimp.js \| 2 -- 1 files changed, 0 insertions(+), 2 deletions(-) http://git.horde.org/horde-git/-/commit/5ea9f6e9776b848684f439e2b49515f7fc3b11b8

03/04/2014 09:05:22 AM	Olbrichj (at) students (dot) uni-marburg (dot) de	Comment #9	Reply to this comment
Hi Michael, your patch is of course way cleaner than my ugly workaround for this special case. But I still get the error due to the change event from the u.clear() call. I am currently running Horde 5.1.5 and IMP 6.1.6, did you test the patch on the current git version?

03/04/2014 04:30:43 AM	Michael Slusarz	Comment #8 State ⇒ Resolved Assigned to Michael Slusarz	Reply to this comment
Horde_Core 2.12.0

03/04/2014 04:30:04 AM	Git Commit	Comment #7	Reply to this comment
Changes have been made in Git (master): commit 0db8c85f28b951e9703f37a26ac0c0c25215f0fe Author: Michael M Slusarz <slusarz@horde.org> Date: Mon Mar 3 21:29:44 2014 -0700 [mms] Fix for prototype.js to correctly recognize IE 11 as IE, not Gecko (~~Bug #12875~~). framework/Core/js/horde.js \| 12 ++++++++++++ framework/Core/package.xml \| 2 ++ 2 files changed, 14 insertions(+), 0 deletions(-) http://git.horde.org/horde-git/-/commit/0db8c85f28b951e9703f37a26ac0c0c25215f0fe

03/03/2014 10:30:35 AM	Olbrichj (at) students (dot) uni-marburg (dot) de	Comment #6 New Attachment: IE11_Attachment_Fix.patch	Reply to this comment
Captcha-Timeout killed attachment...

03/03/2014 10:29:51 AM	Olbrichj (at) students (dot) uni-marburg (dot) de	Comment #5	Reply to this comment
The current prototype.js identifies IE11 as Gecko. The attached patch is a workaround. The u.clear() seems to trigger another change-event, what is the purpose of this line?

12/07/2013 04:43:49 AM	Michael Slusarz	Comment #4	Reply to this comment
It triggers 2 events when the label element. Should read "It triggers 2 change events when the label element is clicked"

12/07/2013 04:42:50 AM	Michael Slusarz	Comment #3 Taken from Michael Slusarz	Reply to this comment
IE11 appears broken. It triggers 2 events when the label element. I can't figure out how to cancel the first event. I don't have time to track this down, so it would be great if someone can figure out a workaround.

12/05/2013 10:53:49 AM	Jan Schneider	Assigned to Michael Slusarz State ⇒ Assigned

12/05/2013 10:53:39 AM	Jan Schneider	Version ⇒ 6.1.6 Queue ⇒ IMP

12/04/2013 02:44:05 PM	vmkari (at) cc (dot) helsinki (dot) fi	Comment #2 New Attachment: rakki06.jpg	Reply to this comment
For your testing purposes, I will provide the actual rakki06.jpg test case attachment in a reply to this ticket. Here is the rakki06.jpg to be used in testing.

12/04/2013 02:43:06 PM	vmkari (at) cc (dot) helsinki (dot) fi	Comment #1 Priority ⇒ 1. Low Type ⇒ Bug Summary ⇒ IE11 attachment adding problem Queue ⇒ Horde Groupware Webmail Edition Milestone ⇒ Patch ⇒ No New Attachment: ie11_attachment_problem.jpg State ⇒ Unconfirmed	Reply to this comment
Attaching files with IE11 causes success AND error messages to be emitted at the same time. However, attaching the files works despite the error message, but the error message distracts users into thinking that attaching failed. I have attached a screenshot with this ticket. For your testing purposes, I will provide the actual rakki06.jpg test case attachment in a reply to this ticket.