Hi Raymond,

Following my discovery that _assignElement is looping on a layer children to do the same, I realized what if in
_assignElement, we would add if(this.isInline) DynLayer._updateValues(this), and add it as well to the loop

for (var i=0; i<dlyr.children.length; i++) {
var iChild = dlyr.children[i];
DynLayer._assignElement(iChild);
if(this.isInline) DynLayer._updateValues(iChild);
}

It would save the cost to iterate over the children array again in _createInline

Benoit

On Tuesday, March 4, 2003, at 08:50 PM, Raymond Irving wrote:

Benoit,

I've added the inlineID argument to
addChild(c,alias,inlineID) as you've originally
suggested. This worked out great for create a DynLayer
in one line of code. When inlineID is passed the
addChild() will cal setID()

--
Raymond Irving

--- Benoit Marchant <[EMAIL PROTECTED]> wrote:
On Tuesday, March 4, 2003, at 01:42 PM, Raymond
Irving wrote:

--- Benoit Marchant <[EMAIL PROTECTED]> wrote:
It looks like the _ is used for different files
for
different browsers,
and the . is used to name an extension of an
existing file, like
functions.color.js, or ioelement.soda.js, so I
would
say 2 is the
perfect candidate.

I agree with that. Then #2 it is

By the way, I did upload 2 files, the inline, and
the dragevent fixes.

I've downloaded the files and will take a look
them
later as my mouse is giving me some problems and
I'll
have to either fix or change it.

As for the NS4 drag updates. What about the
following
lines?

e.defaultValue = false;
e.bubble = de.bubble;

You said that these were the fixed. Or is it that
these lines works better?

e.preventDefault();
I found out that e.preventDefault(); does
e.defaultValue = false;

e.preventBubble();
and preventBubble set e.bubble to false , which is
I believe is
required. I did re test NS4 after i changed these 2
lines and it was
fine. These 2 lines are used in the onmousemove, as
their equivallent
were in dynapi2.
Please test it on your machine.

Benoit
--
Raymond Irving

Benoit

On Tuesday, March 4, 2003, at 01:26 PM, Raymond
Irving wrote:

--- Benoit Marchant <[EMAIL PROTECTED]> wrote:
So will you rename the file dynlayerinline.js ?

Well, since most of the files inside dynapi uses
the
"." extension I would assume it's best to just
follow
through with.

What do you think?

1) dynlayer_inline.js
2) dynlayer.inline.js
3) inline.js
4) dynlayerinline.js

I'm more for #2 or #1.

--
Raymond Irving

Benoit

On Monday, March 3, 2003, at 11:48 AM, Raymond
Irving wrote:

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








-------------------------------------------------------
This SF.net email is sponsored by: Etnus,
makers
of
TotalView, The debugger
for complex code. Debugging C/C++ programs can
leave
you feeling lost and
disoriented. TotalView can help you find your
way.
Available on major UNIX
and Linux platforms. Try it free. www.etnus.com
_______________________________________________
Dynapi-Dev mailing list
[EMAIL PROTECTED]





http://www.mail-archive.com/[EMAIL PROTECTED]/
__________________________________________________
Do you Yahoo!?
Yahoo! Tax Center - forms, calculators, tips,
more
http://taxes.yahoo.com/







-------------------------------------------------------
This SF.net email is sponsored by: Etnus, makers
of
TotalView, The debugger
for complex code. Debugging C/C++ programs can
leave
you feeling lost and
disoriented. TotalView can help you find your
way.
Available on major UNIX
and Linux platforms. Try it free. www.etnus.com
_______________________________________________
Dynapi-Dev mailing list
[EMAIL PROTECTED]



http://www.mail-archive.com/[EMAIL PROTECTED]/
__________________________________________________
Do you Yahoo!?
Yahoo! Tax Center - forms, calculators, tips, more
http://taxes.yahoo.com/




-------------------------------------------------------
This SF.net email is sponsored by: Etnus, makers
of TotalView, The
debugger
for complex code. Debugging C/C++ programs can
leave you feeling lost
and
disoriented. TotalView can help you find your way.
Available on major
UNIX
and Linux platforms. Try it free. www.etnus.com
_______________________________________________
Dynapi-Dev mailing list
[EMAIL PROTECTED]


http://www.mail-archive.com/[EMAIL PROTECTED]/



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


-------------------------------------------------------
This SF.net email is sponsored by: Etnus, makers of TotalView, The debugger
for complex code. Debugging C/C++ programs can leave you feeling lost and
disoriented. TotalView can help you find your way. Available on major UNIX
and Linux platforms. Try it free. www.etnus.com
_______________________________________________
Dynapi-Dev mailing list
[EMAIL PROTECTED]
http://www.mail-archive.com/[EMAIL PROTECTED]/

Reply via email to