script.aculo.us

The following is not working properly in IE7 with the latest versions of Prototype and Script.aculo.us:

<ul>
<li id="test1"><a href="#" onclick="Effect.BlindUp('test1', {duration: .5}); return false;">BlindUp Test Link 1</a></li>
<li id="test2"><a href="#" onclick="Effect.BlindUp('test2', {duration: .5}); return false;">BlindUP Test Link 2</a></li>
<li id="test3"><a href="#" onclick="Effect.SlideUp('test3', {duration: .5}); return false;">SlideUp Test Link 1</a></li>
<li id="test4"><a href="#" onclick="Effect.SlideUp('test4', {duration: .5}); return false;">SlideUp Test Link 2</a></li>
</ul>

The list items move down instead of up.

The following is a fix I implemented which I'm not sure is the best solution (since it also happens with Effect.SlideUp):

Effect.BlindUp = function(element) {
  element = $(element);
  var originalElement;
  if (Prototype.Browser.IE && element.nodeName == 'LI') {
    originalElement = element;
    element = element.wrap('div');
  }
  element.makeClipping();
  return new Effect.Scale(element, 0,
    Object.extend({ scaleContent: false, 
      scaleX: false, 
      restoreAfterFinish: true,
      afterFinishInternal: function(effect) {
        effect.element.hide().undoClipping();
        if (originalElement) element.parentNode.replaceChild(originalElement, element);
      } 
    }, arguments[1] || { })
  );
};