hmm very short code. Cool.

notes:

*1:*

myTree.addEvent('focus', function(){ if(myTree.blur) myTree.blur(); });

Why are you doing this?


*2:*

if( $chk( myTree.getNext('ul') ) ){

you dont need $chk here, a simple if(myTree.getNext('ul')) is fine.

*3:*

myTree.set('href','javascript: void(0);');

you dont need this. Use e.preventDefault() into the click event that you
define some lines after it and it will be fine.

*4:

*This is one of those things that should really use event delegation.

http://mootools.net/docs/more/Element/Element.Delegation

Using event delegation is more memory efficient and scales well in this
case. Still it is perfect if you are loading new folders and files, cause
you wont need to add the click event on them.


--
Fábio Miranda Costa
Solucione Sistemas
Engenheiro de interfaces


On Sat, Dec 12, 2009 at 4:25 PM, Sanford Whiteman <
[email protected]> wrote:

> > kist
>
> I'm going to pretend that this is just-coined jargon for "enjoyed."
>
>

Reply via email to