Bugzilla – Bug 334550
gnome-main-menu fails to open on very first click
Last modified: 2008-01-10 21:33:42 UTC
gnome-main-menu fails to open on very first click. I'm sure we fixed this for SP1.
I'm struggling to reproduce this reliably. I've seen it at least twice today, but can't get it to happen at will.
Alright, I think I might've fixed this one. Please try the packages here ... http://w3.suse.de/~jimmyk/gnome-main-menu/openSUSE-10.3/v0.9.8-94.3b/ and see if you can replicate still. Thanks! jim
Jim, If you keep clicking on Computer (with a delay less than a second in between), it will eventually either stop closing Slab, or stop opening Slab... I *think* that ties into this bug (Happens even after updating with your new packages).
(In reply to comment #3 from Magnus Boman) > Jim, > If you keep clicking on Computer (with a delay less than a second in between), > it will eventually either stop closing Slab, or stop opening Slab... I *think* > that ties into this bug (Happens even after updating with your new packages). > That's because multi-clicks are debounced. It's always worked this way. If this is bad behavior, please file a separate bug. Thanks! jim
Ok, I've installed it, but as noted I'm note able to replicate it every time so I'll run it for a few days. If you have steps Jim or an idea how to trigger it, let me know.
Ok, I managed to replicate it on FACTORY every time, the new packages solve the problem. Please attach the patch for review.
Created attachment 182490 [details] proposed fix Here's a proposed patch for Federico to review.
(In reply to comment #7 from James Krehl) > > Here's a proposed patch for Federico to review. This makes the first click work, and it debounces the menu. For that, it's good to go. Some comments for the future: * The slab window is a GTK_WINDOW_POPUP as per the Glade file, so the window will be OverrideRedirect (i.e. not managed by the window manager). Thus, gtk_window_present_with_time() doesn't really have any special effect on it. You could simply use gtk_widget_show() for it. * I still think this needs to be done at the level of "button_press_event", not at the level of the "clicked" signal. For example, right now you can do this: 1. Click on the main-menu to open it. 2. Start clicking multiple times rapidly on the Computer button. The menu will hide, but then the Computer button will go up and down repeatedly with seemingly no action. A better implementation of debouncing would hide the main-menu, but prevent the actual visible press-and-release from happening.
Could I get a SWAMP ID for this fix? Thanks, Jim Krehl
There's a test package here with the aforementioned patch: http://download.opensuse.org/repositories/home:/federico-mena:/10.3/openSUSE_10.3 It fixes the problem for me. I'll put this patch in our main package as soon as the Main Menu bug week is over: http://en.opensuse.org/GNOME/Main_menu_bug_week
submitted to 10.3 for inclusion in the upcoming general update - test packages are in the 10.3 update test repo - http://download.opensuse.org/repositories/GNOME:/test-updates:/10.3/openSUSE_10.3/
Tested packages from this repo and confirm that this is fixed.