Menu
▾
▴
Re: [Dynapi-Dev] [ dynapi-Bugs-840767 ] mouse_ie.js null error, with possible patch
|
From: Leif W <war...@us...> - 2005-07-22 04:38:23
|
Thanks Doug for pinging this. If there are any other things you're wondering why no one responded to for 2-3 years, feel free to ping them too. One at a time we can get them all. Thanks for looking into this one too, Andrew. You beat me to it. :p I too was looking at this. I did pull a fresh CVS but did not look at the previous commit messages. I still want to test it just to make sure I understand it and verify that it's still fixed. On a related note, since pushing this message through to the Dev list, I was thinking again. Should I let all these tracker notifications flood the dev list or make a new list? Well I decided to make a new list for that, so I have been setting up a list specifically to receive all the tracker messages from now on. That should be read-only, whereas this dev list is where any subsequent discussions can take place. I believe I mentioned something about this way back on the admin list, no replies. :p Anyways it's how MinGW is doing it, and that seems to work well enough. BTW: If anyone wants to ping in a real-time chat setting, pop on to IRC /server irc.freenode.net and /join #dynapi . I leave myself idling in there. Sometimes I am away and my system might crash (or someone else powers off), and I may or may not see a message in the logs when I next connect. But generally I am on there and check the window a few times an hour. If I disconnect without responding, there's the chance I miss a message, so you can leave a system message if it's critical. So far no one but me has ever been there when I was there, so whoever is first will get a +v. :p Gaim is a nifty multi-IM (including IRC) and multi-OS client (including Linux & Windows), if you need a simple IRC client, otherwise mIRC is available for free usage in Windows. Leif ----- Original Message ----- From: "Andrew Gillett" <an...@ph...> To: <dyn...@li...> Sent: 2005 July 22 Friday 00:13 Subject: Re: [Dynapi-Dev] [ dynapi-Bugs-840767 ] mouse_ie.js null error, with possible patch > 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.../ > > |
