Drag-and-Drop behavior in the menu editor is inconsistent and confusing

Hi,
Here is a patch, which insert the menu items right after dropping.
This is my first patch. I hope my patch can help to fix the bug.

@Richard: I like the immediate feedback that your patch adds to the menu editor. It's a definite improvement, but It doesn't really solve the problem of where the items get placed once you drop them. If you open the Menu Editor and drag an item up or down, you'll notice a black line that appears when you hover between any two items. The placement of the line implies that when you drop the item, it will be placed in between those other two items, but if you're dragging downward the item directly below gets pushed up and the item you were dragging is placed below the line. When you drag an item down, the item should be placed above the black line, not below it. Dragging an item up behaves as expected, though. I'm attaching an image that demonstrates what I'm talking about just to clear up any confusion.

@Wes M: My patch fixes exactly what you have explained. :) Here the other bug report with a similar patch: http://bugzilla.gnome.org/show_bug.cgi?id=585807

@Richard: I must have done something wrong then because I'm still having the same problem in Jaunty 64-bit. I downloaded the source code for alacarte, moved the patch file into the source code folder, and applied the patch (patch -p0 < MenuEdit.patch). It said that the patch was successfully applied, so I compiled and installed it (./configure && make && sudo make install). After that I logged out, then logged back in and opened up the Menu Editor to test the patch. Is there something else I needed to do?

@Wes M: "make install" installs in /usr/local/ please use ./configure --prefix=/usr && make && sudo make install (don't worked for me).
Better is to patch the file directly. It is located in /usr/share/pyshare/Alacarte.

@Richard: I patched the file in /usr/share/pyshared/Alacarte and it works exactly as expected now. Thank you for your help, and hopefully the patch will make its way into the next release.

I can confirm that Richard's patch fixes the issue on Lucid with alacarte 0.12.4-1ubuntu1.

Reassigning to bgo 608221 since the previous (bgo 585807) seems to have been marked Resolved even though we can clearly confirm it wasn't (or that there is a regression).

The attached debdiff applies Richard's patch which corrects the issue.

Robert: thanks for your work here!

I've provided a (non-Alacarte developer) review of the patch on the bug upstream; the patch looks correct and fixes a real bug.

I've also updated the debdiff to apply to the current Natty package, and attached it here.

This bug was fixed in the package alacarte - 0.13.2-1ubuntu2

---------------
alacarte (0.13.2-1ubuntu2) natty; urgency=low

  * Correct off-by-one error when dragging an item downward (LP: #395692)
    - added debian/patches/04_lp395692_downward_dragging.patch
 -- Mathieu Trudel <email address hidden> Tue, 26 Jan 2010 23:07:18 -0500

Unsubscribing sponsors team, nothing for us here.