Tickets :: [#9511] Compose popup not opening in IE7

Summary	Compose popup not opening in IE7
Queue	IMP
Queue Version	Git master
Type	Bug
State	Resolved
Priority	1. Low
Owners	slusarz (at) horde (dot) org
Requester	goncalo.queiros (at) portugalmail (dot) net
Created	01/14/2011 (5287 days ago)
Due
Updated	01/14/2011 (5287 days ago)
Assigned
Resolved	01/14/2011 (5287 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	No

01/14/2011 06:55:28 PM	Michael Slusarz	Comment #3 Assigned to Michael Slusarz State ⇒ Resolved	Reply to this comment
The problem is that when composeDimp.fillForm is executed, the elements are not present on the page, so focus on the $('to') field fails. I also noticed that in Firefox, any onDomLoaded function present on other .js files get executed and only then, composeDimp.fillForm is called. In IE7 this happens the other way around, which can help explain the problem. This is all known an expected. Which is why fillForm() doesn't run until document.loaded is true (http://api.prototypejs.org/dom/document/loaded/). However, fillForm() now relies on things initialized by onDomLoad(), and it is possible that onDomLoad() is run after fillForm(). So looks like we need a check for that.

01/14/2011 06:55:16 PM	Git Commit	Comment #2	Reply to this comment
Changes have been made in Git for this ticket: ~~Bug #9511~~: Make sure DimpCompose.onDomLoad() has run before doing other actions http://git.horde.org/horde-git/-/commit/e2ceee21f7bed099b2d8af8872ecc762e4fc72cc

01/14/2011 10:19:38 AM	goncalo (dot) queiros (at) portugalmail (dot) net	Comment #1 Priority ⇒ 1. Low Patch ⇒ No Milestone ⇒ Queue ⇒ IMP Summary ⇒ Compose popup not opening in IE7 Type ⇒ Bug State ⇒ Unconfirmed	Reply to this comment
The problem is that when composeDimp.fillForm is executed, the elements are not present on the page, so focus on the $('to') field fails. I also noticed that in Firefox, any onDomLoaded function present on other .js files get executed and only then, composeDimp.fillForm is called. In IE7 this happens the other way around, which can help explain the problem.