·  $E  is  old Moo for $$()[0] (also lots of other ways to express the
same thing)

· $clear is out, use standard clearInterval

· Element.effects --» Element.morph

· Element.extend --» Element.implement to get your methods on existing
natives

·  You were adding Element.getParents and Element.getSiblings methods,
which are now added by Moo and shouldn't be overridden unless you have
a *very* good reason

I got you to the point that there aren't any errors, but you are going
to  have  to  put  your  effect  back  in, as it seemed to be based on
requirements that I don't have the time to reverse-engineer (i.e. if I
were  doing  what you're doing, I'd do it differently from start, esp.
because  I tend to use CSS3 transitions when available and your effect
can be fully done with CSS).

    http://jsfiddle.net/sanford/J22SV/5/

You  should  always  comment your code, even when it relies on "fixed"
functionality  from  frameworks. Despite best efforts at BC, you can't
expect  methods to be truly unchanging as Moo evolves. Or you can skip
the  comments,  but  then be ready to be your sole support when people
can't remember 1.1. :)

-- S.

Reply via email to