It can be merged into one.. see Dynacore for examples on this.. DynLayer and DynDocument can use the exact same methods for event handling.
 
But again, I think we should wait with these things until we have the current code as a stable release.
 

Pascal Bestebroer ([EMAIL PROTECTED])
Software ontwikkelaar
Oberon Informatiesystemen b.v.
http://www.oibv.com

-----Oorspronkelijk bericht-----
Van: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED]]Namens Michael Pemberton
Verzonden: vrijdag 12 januari 2001 4:47
Aan: [EMAIL PROTECTED]
Onderwerp: Re: [Dynapi-Dev] Events. not a bug, just a suggestion

oops.  I should have also said that the folowing alteration also needs to be made.

DynDocument.prototype.invokeEvent=DynLayer.prototype.invokeEvent;

Michael Pemberton wrote:

I've noticed that there is only two lines difference between the DynDocument and DynLayer invokeEvent methods.

I'd suggest we simply alter the final lines to read as follows:
    if (this.isDynLayer && is.ns && e) {
        if (e.cancelBubble) return;
        if (orig && orig.target.handleEvent && orig.target!=this.elm) orig.target.handleEvent(type,orig);
    } else if (this.isDynDocument && (i!=0 || e) && is.ns) {
        if (e.cancelBubble) return;
        if (orig && orig.target.handleEvent && !orig.target.URL) orig.target.handleEvent(orig);
    };

If I've missed something and am completly wrong, please point out the errors of my way.  If not, it might be worth making the alterations to the CVS version.
--
Michael Pemberton
[EMAIL PROTECTED]
ICQ: 12107010
  _______________________________________________ Dynapi-Dev mailing list [EMAIL PROTECTED] http://lists.sourceforge.net/lists/listinfo/dynapi-dev

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

Reply via email to