Rewrite menu_minipanels.js for bugs and performance

Nice patch. Committed.

This needs to be ported to D7 so we can keep both codebases close.

Could use some review.

It appears to still work in my testing. I think it's good to go.

@codycraven: Out of interest, was there a reason to remove the $ in front of the 'target' variable? It was my understanding that was a coding standard for variables that point to jQuery objects?

Upon reading the JS coding standards I don't anything about using $, so I'm going to remove it.

Of course then I look in contextual.js and see lines like the following:

Drupal.behaviors.contextualLinks = {
  attach: function (context) {
    $('div.contextual-links-wrapper', context).once('contextual-links', function () {
      var $wrapper = $(this);
      var $region = $wrapper.closest('.contextual-links-region');
      var $links = $wrapper.find('ul.contextual-links');

This patch improves upon the one from #6 but makes the $var naming structure consistent.

This patch for the D6 branch updates the variable naming.

Committed. Thanks @codycraven.