Patches item #414698, was updated on 2001-04-08 06:48
You can respond by visiting: 
http://sourceforge.net/tracker/?func=detail&atid=305757&aid=414698&group_id=5757

Category: DynAPI 2 Event
Group: None
Status: Open
Priority: 5
Submitted By: Richard Bennett (richard_bennett)
Assigned to: Nobody/Anonymous (nobody)
Summary: cancelBrowserEvent update for release2.5

Initial Comment:
It seems my flawed patch for mouse.js slipped through 
into the 2.5 release.
So now line 28 in mouse.js reads like this:

DynMouseEvent.prototype.cancelBrowserEvent=function(b) 
{this.browserReturn=b};

So if we actually call cancelBrowserEvent somewhere we 
must now pass a true/false value. As far as I know 
this method is only used in dragevent.js.
On line 28 we have:

e.cancelBrowserEvent();

This line should be deleted, as it causes NS4 to stop 
default mouse events over all dynLayers, and isn't 
needed in IE as far as I can see, as we cancel the 
browser event again in the onmousemove event. That's 
on line 103, now it's:

e.cancelBrowserEvent();

This should be:

e.cancelBrowserEvent(false);

And again on line 131, now:

e.cancelBrowserEvent();

I think this should be:

e.cancelBrowserEvent(null);

as it's cleaning up the drag events.



----------------------------------------------------------------------

You can respond by visiting: 
http://sourceforge.net/tracker/?func=detail&atid=305757&aid=414698&group_id=5757

_______________________________________________
Dynapi-Dev mailing list
[EMAIL PROTECTED]
http://lists.sourceforge.net/lists/listinfo/dynapi-dev

Reply via email to