Well it looks like your right, but there are a few issues, firstly, this is
only for NS4, the other browser code seems to be missing all together.
I think the whole thing, crossbrowser should look like this:
This replaces the line nr 147 in mouse.js:
// Click on links and form elements
if (e && e.target.handleEvent && e.target!=this)evt.browserReturn =
e.target.handleEvent(e);
}else{
if(e&&e.srcElement!=this)evt.browserReturn =e.returnValue
The closing bracket is already in the code.
I tested this, and it solves the problem in IE5.5, Mozilla, and NS4.
It does however cause NS4 to fire two onclick events over a regular
dynlayer, I think this might be something Jordi could solve easily (?)
Cheers,
Richard Bennett
[EMAIL PROTECTED]
www.richardinfo.com
(Everything running on, and ported to DynAPI2.53)
visit the DynAPI homepage (and FAQ) ::
http://dynapi.sourceforge.net/dynapi/index.php?menu=1
Browse (and search) the mailinglist here:
http://www.mail-archive.com/index.php3?hunt=dynapi
> Well, mouse.js is wrong - at the end of the DynMouseEvent.EventMethod
function around line 147 it reads
>
> // Click on links and form elements
> if (e && e.target.handleEvent && e.target!=this)
e.target.handleEvent(e);
>
> It should be:
>
> // Click on links and form elements
> if(e && e.target.handleEvent && e.target!=this)
> evt.browserReturn = e.target.handleEvent(e);
>
> That change should of course be fully tested because it has global impact.
>
> /Lunna
>
>
> _______________________________________________
> Dynapi-Help mailing list
> [EMAIL PROTECTED]
> http://lists.sourceforge.net/lists/listinfo/dynapi-help
>
_______________________________________________
Dynapi-Help mailing list
[EMAIL PROTECTED]
http://lists.sourceforge.net/lists/listinfo/dynapi-help
