"Move local tasks to menu" leaves empty UL

This should work as it is an extremely simple 1 line fix (untested):

    // Move page tabs into administration menu.
    if (Drupal.settings.admin_menu.tweak_tabs) {
      $('ul.tabs.primary li').each(function() {
        $(this).addClass('admin-menu-tab').appendTo('#admin-menu > ul');
      });
      $('ul.tabs.secondary').appendTo('#admin-menu > ul > li.admin-menu-tab.active');
      $('ul.tabs.primary').remove(); // <------ Added
    }

1. I know how to make a patch. I was not at a dev machine when I tried to help you with a fix off the top of my head (notice it says untested).

2. It is a 1 line fix. Quicker for you to just do manually than bother with another patronising and unhelpful response instead of a little thanks.

3. Best of luck in the future, you've done enough to discourage me from fixing any more of your bugs.

Index: admin_menu.js
===================================================================
--- admin_menu.js	(revision 262)
+++ admin_menu.js	(working copy)
@@ -15,6 +15,7 @@
         $(this).addClass('admin-menu-tab').appendTo('#admin-menu > ul');
       });
       $('ul.tabs.secondary').appendTo('#admin-menu > ul > li.admin-menu-tab.active');
+      $('ul.tabs.primary').remove();
     }
   }

Ok then, no thanks for the fix. Cheers

I appreciate you have a lot to do, we all do .. but one word answers and pointing to patch instructions for the simplest of fixes without a hint of appreciation or thanks don't encourage people to participate much.

As for the error in the patch I corrected a minute after posting, good catch but what good is a patch-formatted fix if there is an error in it anyway? Not sure how you could have made the same mistake by copy+pasting a solitary line that I posted correctly originally (and if you think about it your insistence on a patch potentially led to the creation of an error).