Menu
▾
▴
Re: [Dynapi-Dev] [ dynapi-Bugs-840767 ] mouse_ie.js null error, with possible patch
|
From: Leif W <war...@us...> - 2005-07-22 16:17:16
|
> <do...@cr...> > 2005 July 22 Friday 09:52 > > Hrm.. well the bug still exists in the production code. > This is not a small bug either, as it very consistant in producing > said error > on ie 6 no matter what you are coding. > > Can we publish a new production version then? maybe we can do a small > push to > varify the current cvs version for publishing? Hi Doug, I was planning to look at it with IE6 today. I spent yesterday setting up the new list, fiddling with the mailing list and trackers. But, I'm unsure how to produce the bug? Is it a specific library that causes the problem? I can of course load them all and comment out one at a time, but it'd save time if I knew exactly what to load. ;-) Also, it really occurs just with "the" library loaded and zipping the mouse arround quickly? Do I need to test any other conditions? DynAPI documents, layers, event handlers or html forms and so on? Leif > Andrew Gillett <an...@ph...> said: > >> I've closed this bug. The patch was actually applied by Raymond quite >> a >> long time ago. >> His CVS comment was: >> revision 1.7 >> date: 2003/09/14 14:36:00; author: xwisdom; state: Exp; lines: >> +2 -3 >> Patch for dynapi.api.MouseEvent bug >> submitted by Peter Romianowski updated by Raymond >> >> I guess there are probably other items in the bug list that should be >> closed. >> >> Thanks Doug for raising this one. >> If you find any more like this, please let us know :-) >> >> >> Andrew. >> >> >> SourceForge.net wrote: >> >> >Bugs item #840767, was opened at 2003-11-12 07:31 >> >Message generated for change (Comment added) made by doug_melvin >> >You can respond by visiting: >> >>https://sourceforge.net/tracker/?func=detail&atid=105757&aid=840767&group_id=5757 >> > >> >Please note that this message will contain a full copy of the >> >comment thread, >> >including the initial issue submission, for this request, >> >not just the latest update. >> >Category: DynAPI 3 API >> >Group: None >> >Status: Open >> >Resolution: None >> >Priority: 5 >> >Submitted By: David Haggard (dhaggard) >> >Assigned to: Nobody/Anonymous (nobody) >> >Summary: mouse_ie.js null error, with possible patch >> > >> >Initial Comment: >> >Background: >> >Two objects on page (among others, but these seem to >> >be the "offenders." >> >1) A single DynLayer with a single assigned >> >eventListener for mouseover/mouseout, and using the >> >glideTo method. Used to scroll a navigation menu up and >> >down at top of page. >> >2) A form button, disabled by default, with a "title" >> >attribute set. No onmouseover trap defined for this >> >button. >> > >> >Symptom: >> > On rapid mouseover over the button, javacript error in >> >line 71 of mouse_ie.js. "null is not an object." >> > Interesting that this only happened when the mouse >> >was moving quickly. Never happened with slow mouse >> >movement. >> > >> >Lines 70-74 in mouse_ie.js -- >> >---------------------------------- >> >if (e.type=="mouseout" || e.type=="mouseover") { >> > if (r && (r==src||src.isParentOf(r))) return; //fix for >> >#15 (ie only) >> > //if (r&&(r==src.parent||r.isChildOf(src.parent))) >> >me.bubble=false; >> > if (r&&(r==src.parent||r.isChildOf(src.parent))) >> >me.bubble=false; >> >} >> >--------------------------------- >> > >> >I eliminated the problem, with no affect on correct >> >functionality, by changing to this: >> > >> >Lines 70-76 in mouse_ie.js -- >> >--------------------------------- >> >if (e.type=="mouseout" || e.type=="mouseover") { >> > if (src != null) { // error handler if src is null >> > if (r && (r==src||src.isParentOf(r))) return; //fix for >> >#15 (ie only) >> > //if (r&&(r==src.parent||r.isChildOf(src.parent))) >> >me.bubble=false; >> > if (r&&(r==src.parent||r.isChildOf(src.parent))) >> >me.bubble=false; >> > } >> >} >> >-------------------------------- >> > >> >May not be the best fix, but it worked for this strange >> >error. I searched and searched, and never could find >> >WHY this error was occurring with a form button that is >> >in no way using the DynAPI. >> > >> >David Haggard >> > >> >---------------------------------------------------------------------- >> > >> >Comment By: Doug Melvin (doug_melvin) >> >Date: 2005-07-21 10:49 >> > >> >Message: >> >Logged In: YES >> >user_id=184788 >> > >> >I was experiencing a bug in IE 6 where simply moving the >> >mouse around would random generate the above error in teh >> >same code. >> > >> >This error occurs even when the full extent of you code is >> >to load the libraries (I.E. no layers created, no event >> >handlers added.. just load the api and library) >> > >> >Adding the above mentions fix (checking for src!=null b4 >> >doing stuff to src) fixes this and breaks nothing else as >> >far as i can tell. >> > >> >I tested in IE 6 with the mouse event and drag event examples >> > >> >Could someone please add this fix and post to the cvs. >> > >> >2 years is long enough for this one imho. >> > >> >cheers >> > >> >---------------------------------------------------------------------- >> > >> >You can respond by visiting: >> >>https://sourceforge.net/tracker/?func=detail&atid=105757&aid=840767&group_id=5757 >> > >> > >> >------------------------------------------------------- >> >SF.Net email is sponsored by: Discover Easy Linux Migration >> >Strategies >> >from IBM. Find simple to follow Roadmaps, straightforward articles, >> >informative Webcasts and more! Get everything you need to get up to >> >speed, fast. http://ads.osdn.com/?ad_id=7477&alloc_id=16492&op=click >> >_______________________________________________ >> >Dynapi-Dev mailing list >> >Dyn...@li... >> >http://www.mail-archive.com/dyn...@li.../ >> > >> > >> > >> >> >> >> ------------------------------------------------------- >> SF.Net email is sponsored by: Discover Easy Linux Migration >> Strategies >> from IBM. Find simple to follow Roadmaps, straightforward articles, >> informative Webcasts and more! Get everything you need to get up to >> speed, fast. http://ads.osdn.com/?ad_id=7477&alloc_id=16492&op=click >> _______________________________________________ >> Dynapi-Dev mailing list >> Dyn...@li... >> http://www.mail-archive.com/dyn...@li.../ >> > > > > -- > > > > > > ------------------------------------------------------- > SF.Net email is sponsored by: Discover Easy Linux Migration Strategies > from IBM. Find simple to follow Roadmaps, straightforward articles, > informative Webcasts and more! Get everything you need to get up to > speed, fast. http://ads.osdn.com/?ad_id=7477&alloc_id=16492&op=click > _______________________________________________ > Dynapi-Dev mailing list > Dyn...@li... > http://www.mail-archive.com/dyn...@li.../ > > |
