The setID should be modified from within dynapi.js
Only add the inline argument:

p.setID = function(id,isInline) {
  if (this.id) delete DynObject.all[this.id];
  this.id = id;
  this.isInline=isInline;
  DynObject.all[this.id] = this;
};

There are other areas inside the api that would check
for isInline. I think I'll have to check this out
later. If you don't mind I can add the necessary codes
to call the _createInLine() functions into the various
areas of the api. So you could focus on the
dynlayer_inline.js file while I'll add the isInline
checks. Agree?

PS. Also remember that We'll now be using
DynLayerInline instead of DynLayer_inline

--
Raymond Irving



--- Benoit Marchant <[EMAIL PROTECTED]> wrote:
> Ok, so do we add the isInline argument to setID in
> event.js where we do 
> use it in addChild ?
> 
> p.DynObjectSetID = DynObject.setID
> p.setID = function(id,isInline) {
>       p.DynObjectSetID(id);
>       this.isInline = isInline;
> };
> 
> Make sense ?
> 
> Benoit
> 
> > isInline
> On Saturday, March 1, 2003, at 07:24  PM, Raymond
> Irving wrote:
> 
> >
> > --- Benoit Marchant <[EMAIL PROTECTED]> wrote:
> >> You didn't tell why you don't like adding an
> >> argument to addChild ?
> >
> > Well, I believe that using setID() will allows us
> to
> > better set the isInline property. This will offer
> > greater flexiblity if we need to test if a layer's
> elm
> > already exist or if we need to do other stuff with
> > inline layers. All we have to do is to check if
> isLine
> > is true and we will know if the layers has an
> inline
> > elm.
> >
> > --
> > Raymond Irving
> >
> >
> >>
> >> Benoit
> >>
> >> On Friday, February 28, 2003, at 03:09  PM,
> Raymond
> >> Irving wrote:
> >>
> >>>
> >>> --- Doug Melvin <[EMAIL PROTECTED]>
> >> wrote:
> >>>> how about .bindInline('inlineID'); ?
> >>>> thus binding yer dynlayer to an inline layer..
> >>>
> >>> Hmmm...
> >>>
> >>> setID(newID,true) if it's for an inline is more
> >>> flexible.
> >>>
> >>>
> >>>> If done before adding the layer to the document
> >> then
> >>>> when you do add, it
> >>>> would bind the inline layer..
> >>>> if done after the dynlayer has been added..
> then
> >>>> delete the dynamically
> >>>> created components and bind to the inline layer
> >>>> ----- Original Message -----
> >>>> From: "Raymond Irving" <[EMAIL PROTECTED]>
> >>>> To: "DynAPI-Dev"
> >> <[EMAIL PROTECTED]>
> >>>> Sent: Friday, February 28, 2003 2:49 PM
> >>>> Subject: Re: [Dynapi-Dev] New TemplateDesigner
> >>>> Object for 3.0
> >>>>
> >>>>
> >>>>>
> >>>>> --- Benoit Marchant <[EMAIL PROTECTED]> wrote:
> >>>>>> So what about tweaking addChild(c,alias)
> method
> >>>> to
> >>>>>> addChild(c,alias,inline). Then you can
> specify
> >>>>>> inline as true, and we
> >>>>>> would default to no, which means it would be
> >>>>>> transparent to the current
> >>>>>> api.
> >>>>>
> >>>>> I disagree with adding an inline argument to
> the
> >>>>> addChild. I would however suggest that a
> >> isInline
> >>>>> property be used:
> >>>>>
> >>>>> lyr.isInline=true;
> >>>>>
> >>>>> or we could modify the setID() function to use
> >> the
> >>>>> inline agrument.
> >>>>>
> >>>>>> We can "override"
> >>>>>> DynElement.prototype.addChild in the
> >>>>>> dynlayer_inline.js, t should work
> >>>>>> as long as you include that file, right ?
> >>>>>
> >>>>> Yes but not necessary.
> >>>>>
> >>>>>> DynLayer.prototype.assignInlineElement =
> >>>> function ()
> >>>>>> {
> >>>>>>      if (this.parent && !this.elm) {
> >>>>>>          //I'm not sure we want to call that
> as
> >>>>>> onprecreate was usually
> >>>>>> used to do setHTML which doesn't make sense
> >> here
> >>>>>> since the html already
> >>>>>> exists. Comments ?
> >>>>>> //DynElement._flagEvent(this,'precreate');
> >>>>>> DynLayer._assignElement(this);
> >>>>>> DynLayer._updateValues(this);
> >>>>>> DynElement._flagEvent(this,'create');
> >>>>>>      }
> >>>>>> }
> >>>>>
> >>>>> Use _createInline() isstead of
> >>>> assignInlineElement()
> >>>>>
> >>>>>> function(c,alias,inline) {
> >>>>>
> >>>>> function(c,alias) {
> >>>>>
> >>>>>> if (!c) return dynapi.debug.print("Error: no
> >>>> object
> >>>>>> sent to
> >>>>>> [DynLayer].addChild()");
> >>>>>> if (c.isChild) c.removeFromParent();
> >>>>>> c.isChild = true;
> >>>>>> c.parent = this;
> >>>>>> if (c._saveAnchor) {
> >>>>>> c.setAnchor(c._saveAnchor);
> >>>>>> delete c._saveAnchor;
> >>>>>> }
> >>>>>> if(alias) this[alias]=c;
> >>>>>> if (this._created) {
> >>>>>> if(!inline) c._create();
> >>>>>
> >>>>>   if(!c.isLine) c._create();
> >>>>>
> >>>>>> else {
> >>>>>>                      c.assignInlineElement();
> >>>>>
> >>>>>                       c._createInline();
> >>>>>
> >>>>>> }
> >>>>>> this.children[this.children.length] = c;
> >>>>>> return c;
> >>>>>> };
> >>>>>>
> >>>>>
> >>>>>
> >>>>> --
> >>>>> Raymond Irving
> >>>>>
> >>>>>
> >>>>>
> >>>>>
> >>>>>
> >>
> __________________________________________________
> >>>>> Do you Yahoo!?
> >>>>> Yahoo! Tax Center - forms, calculators, tips,
> >> more
> >>>>> http://taxes.yahoo.com/
> >>>>>
> >>>>>
> >>>>>
> >>>>
> >>>
> >>
> >
>
-------------------------------------------------------
> >>>>> This sf.net email is sponsored by:ThinkGeek
> >>>>> Welcome to geek heaven.
> >>>>> http://thinkgeek.com/sf
> >>>>>
> _______________________________________________
> >>>>> Dynapi-Dev mailing list
> >>>>> [EMAIL PROTECTED]
> >>>>>
> >>>>
> >>>
> >>
> >
>
http://www.mail-archive.com/[EMAIL PROTECTED]/
> >>>>>
> >>>>
> >>>>
> >>>>
> >>>>
> >>>
> >>
> 
=== message truncated ===


__________________________________________________
Do you Yahoo!?
Yahoo! Tax Center - forms, calculators, tips, more
http://taxes.yahoo.com/


-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf
_______________________________________________
Dynapi-Dev mailing list
[EMAIL PROTECTED]
http://www.mail-archive.com/[EMAIL PROTECTED]/

Reply via email to