hiding/showing: right way?

[EventListener]

I have a folder/file tree that is dynamically generated from an xml
file. The way I've written it seems to work. Since I'm a fairly novice
javascript programmer, I'm concerned that there may be a hidden
downside to coding this way vs. hiding/showing using style.visibilit y.

My tree starts with only the top level folders showing. When the xml
for the tree loads, I have a recursive function that creates subtrees
for each folder, then saves them as a property of that folder.

buildTree: function( root, offset )
{
var curNode = root;
var myIndent = 20;
var myHeight = 22;

if( offset == null )
{
myIndent = 2;
}
else
{
myIndent += offset;
}

var spacer = "";
for( var i = 0; i < myIndent;i++ )
{
spacer += "-";
}

var elem = document.create Element( "div" );
elem.style.top = 0;

if( curNode.nodeNam e == "folder" )
{
elem.id = "FolderElement" ;

var children = curNode.childNo des;
var subTreeItems = new Array( );
for( var i = 0; i < children.length ; i++ )
{
if( children[i].nodeType == "1" )
{
var newChild = this.buildTree( children[i], myIndent );
subTreeItems.pu sh( newChild );
}
}

var folderName = curNode.attribu tes[0].nodeValue;
var isEmpty = subTreeItems.le ngth > 0 ? false : true ;
var line = this.makeFolder Line(folderName , myHeight, myIndent,
isEmpty );

elem.appendChil d( line );

Event.observe( line, "mouseup", this.__neh_togg leFolder_closur e,
false);

elem["subTreeDat a"]= subTreeItems;
}
else if( curNode.nodeNam e == "item" )
{
elem.id = "ItemElemen t";
elem.className = "item";
var itemObj = new this.ItemObject ( curNode );
var line = this.makeItemLi ne( itemObj, myHeight, myIndent );

elem.appendChil d( line );

Event.observe( line, "mousedown" , this.__neh_sele ctItemDown_clos ure,
false);

}

return elem;
},
When the user opens a folder, the subtree elements are appended.

Are there any dangers/concerns using this method?

(Warning for the anti-library folks -- I am using the library
prototype.js to add and remove event listeners)

Thanks,

Cathy

[Joshie Surber]

The only problem I see with it is that it would assign multiple
elements with the same ID attribute. This is a HUGE (X)HTML no no. You
could set elem.id to elem.className (or elem.className =
elem.className + newClassName) instead.

But other than that the code seems sound. I might, however, point you
to http://gazingus.org/html/DOM-Scripte...Revisited.html, which
has a hierarchal tree script that is simply attached to a list. That
means you would simply have to use basic DOM functions (or even
innerHTML) to modify the tree contents and ignore the heavy lifting
altogether. This would also have the advantage of being more usable to
people with older browsers or JavaScript turned off.

[EventListener]

Joshie Surber wrote:

The only problem I see with it is that it would assign multiple
elements with the same ID attribute. This is a HUGE (X)HTML no no. You
could set elem.id to elem.className (or elem.className =
elem.className + newClassName) instead.

Thanks Joshie. I've noticed that using elem.className is too slow,
since I have multiple class names and have to use a regex to extract
the proper one. But point taken about unique ids. I'll probablly just
add a custom property to indicate folderline vs. fileline.
But other than that the code seems sound. I might, however, point you
to http://gazingus.org/html/DOM-Scripte...Revisited.html, which
has a hierarchal tree script that is simply attached to a list. That
means you would simply have to use basic DOM functions (or even
innerHTML) to modify the tree contents and ignore the heavy lifting
altogether.
Interesting ref. It is always nice to see how others do things. At this
point, though, the tree structure is a small part of a lot of code, and
I don't want to switch it out and risk breaking stuff.
This would also have the advantage of being more usable to
people with older browsers or JavaScript turned off.

This is for an app where it is okay to have firm browser reqs. I'm
doing a lot of feature sniffing and if the features aren't found, the
app won't work.

BTW = if you are using Google Groups, click the "options" link to reply
with the original message quoted.