In IE7, Popup window sticks to mouse point when trying to move the popup window

changlin - August 4, 2007 - 11:41
Project:Drupal ajaxIM
Version:5.x-1.x-dev
Component:User interface
Category:bug report
Priority:critical
Assigned:Unassigned
Status:needs review
Description

IE version: 7.0.5730.11
Every time when I tried to move the popup dialog window, this window will follow the mouse point all the way. Double click the left key, the window becomes bigger, but doesn't stop chasing the mouse point. At the same time, the IE status bar shows "error on page", the detail error message is like this:
Line: 1183
Char: 9
Error: Object expected
Code: 0
URL: my website

I mark this problem as critical because everybody will try to move the popup window to more comfortable place. Then they will find that they can only close the IE to stop the funny chasing (in fact, for the common users, it is not easy for them to close the window when there is a window chasing their mouse).

I tested the original ajaxIM by this link : http://www.ajaxim.net/ . It has no such problem in IE7.

Wish I could get help.

Thanks.

#1

changlin - August 4, 2007 - 11:44

By the way, another friend who uses IE7 also found the same problem. There is no such problem in FireFox 2.0.0.6 that I used in the same computer.

Changlin.

#2

yngens - August 23, 2007 - 07:10

same problem

#3

rpfilomeno - August 29, 2007 - 13:56
Component:Code» User interface

Same problem here. IE7 v7.0.5730.11

Also there is a bug on the join chatroom dialog where the buttons covers the text field (IE 7 and FireFox 2.0.0.6).

Fixed it by editing line 153:
height: 170

and line 166:
$ID('newroom_buttons').style.top = '150px';

#4

rpfilomeno - August 29, 2007 - 14:57
Status:active» needs review

OK. Found the fix

on (MODULE_ROOT)\ajaxim\ajaxim\js\window.js

LINE 1162-1165:

if (contentId && isIE) {
       if($ID('select')=='null') $ID('select').each(function(element) {element.style.visibility = "hidden"});
       if($ID('#'+contentId+' select')=='null') $ID('#'+contentId+' select').each(function(element) {element.style.visibility = "visible"});
}

LINE 1181-1183:

         if (isIE) {
if($ID('select')=='null') $ID('select').each(function(element) {element.style.visibility = "visible"});
         }

on (MODULE_ROOT)\ajaxim\ajax_im.js

LINE 125:

                        {className: "dialog", width: 240, height: 170, resizable: false, zIndex: 1,

LINE 138:

  $ID('newim_buttons').style.top      = '150px';

LINE 153:

                          {className: "dialog", width: 240, height: 170, resizable: false, zIndex: 1,

LINE 166:

  $ID('newroom_buttons').style.top      = '150px';

#5

alakon - October 11, 2007 - 21:32

Submitting rpfilomeno's work as .patch file

Download to your modules/ajaxim directory
From that directory, apply as:

patch -p0 < 20071110ajaxim.patch

The patch seems to fix the problem (but I didn't test extensively). Page (obviously) requires a refresh before the updated files are loaded.

AttachmentSize
20071110ajaxim.patch 4.33 KB
 
 

Drupal is a registered trademark of Dries Buytaert.