Re: [Dynapi-Help] NS4 link/event cancelling [Was: (no subject)]

[Richard B. <ric...@sk...>]

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

ma...@ri...
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
> Dyn...@li...
> http://lists.sourceforge.net/lists/listinfo/dynapi-help
>