https://bugzilla.novell.com/show_bug.cgi?id=334550#c8 Federico Mena Quintero <federico@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |federico@novell.com Status|NEEDINFO |ASSIGNED Info Provider|federico@novell.com | Status Whiteboard| |gnome-function-does-not-work --- Comment #8 from Federico Mena Quintero <federico@novell.com> 2007-11-07 16:27:55 MST --- (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. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.