Re: Accelerator keys and popup menus? (really solved)

[Stefan Kost <ensonic hora-obscura de>]

Its a gtk+ bug!
Files as http://bugzilla.gnome.org/show_bug.cgi?id=469374 with a patch.

After adding a g_signal_emit() as a workaround it works too.
    gtk_window_add_accel_group(GTK_WINDOW(window),GTK_ACCEL_GROUP(user_data));
    g_signal_emit_by_name (window, "keys-changed", 0);

Stefan


Stefan Kost wrote:
> hi,
>
> more research and a standalone test (attached and via web [1]). The good news
> are that gtk+ supports multiple accel maps and that those can be even
> dynamically switched (activated/deactivated). The issue seems that a new
> accel_map needs to be added to the window before adding any accelerators. If one
> only adds it later, it has no effect. I have not yet managed to figure out what
> is causing the difference.
>
> The problem for me is that when I construct my ui I use lots of g_objects which
> init children while they are initialized. As such one cannot yet query the
> window from deep within the hierarchy :/ and passing the window down the call
> hierarchy is out of question.
>
> Stefan
>
>
> [1]
> http://buzztard.cvs.sourceforge.net/buzztard/buzztard/design/gui/accelpopup.c?view=markup
>
> Stefan Kost wrote:
> > Hi,
> >
> > Dunno if this is a bug/limmitation in gtk+. I now moved the
> > GtkAccelGroup to my ui-resource singleton and it works. That means I use
> > only *one* accel_group.
> > I have a toplevel window with views (tabs). Before I was creating a
> > separate one for my popup menu and adding/removing to the window
> > depending on which tab was visible. Seems that gtk+ is only supporting
> > one accel_group.
> >
> > Stefan
> >
> > Quoting Stefan Kost <ensonic hora-obscura de>:
> >
> > > hi,
> > >
> > > Gabriel Schulhof wrote:
> > > > Hi!
> > > >
> > > > On Sun, 2007-08-19 at 23:48 +0300, Stefan Kost wrote:
> > > > > I read about the differences. I really want "accelerators".
> > > > I was able to hack it:
> > > >
> > > > Create a plain old regular menubar-type menu with accelerators. Then ref
> > > > the submenu you want to make into a popup menu and detach it from its
> > > > parent menu item[0].
> > > >
> > > > [0]http://library.gnome.org/devel/gtk/unstable/GtkMenuItem.html#id3734460
> > > >
> > > >
> > > > HTH,
> > > >
> > > > Gabriel
> > > like this?
> > >
> > > // generate the context menu
> > > GtkWidget *mb=gtk_menu_bar_new();
> > > GtkWidget *mi=gtk_menu_item_new();
> > > gtk_container_add(GTK_CONTAINER(mb),mi);
> > >
> > > self->priv->context_menu=GTK_MENU(gtk_menu_new());
> > > gtk_menu_set_accel_group(GTK_MENU(self->priv->context_menu),
> > >   self->priv->accel_group);
> > >
> > > gtk_menu_set_accel_path(GTK_MENU(self->priv->context_menu),"<Buzztard-Main>/PatternView/PatternContext");
> > >
> > >
> > > gtk_menu_item_set_submenu(GTK_MENU_ITEM(mi),GTK_WIDGET(self->priv->context_menu));
> > >
> > > g_object_ref(self->priv->context_menu);
> > > gtk_menu_detach(self->priv->context_menu);
> > >
> > >
> > > make no difference for me :/
> > >
> > > I also tried
> > > gtk_menu_attach_to_widget(self->priv->context_menu,
> > >   GTK_WIDGET (main_window),NULL);
> > >
> > >
> > > Stefan
> > > _______________________________________________
> > > gtk-app-devel-list mailing list
> > > gtk-app-devel-list gnome org
> > > http://mail.gnome.org/mailman/listinfo/gtk-app-devel-list
>
>
> ------------------------------------------------------------------------
>
> /** $Id: accelpopup.c,v 1.1 2007/08/21 19:55:32 ensonic Exp $
>  * test popup menus with accelerator keys
>  *
>  * gcc -Wall -g `pkg-config gtk+-2.0 --cflags --libs` accelpopup.c -o accelpopup
>  */
>
> #include <stdio.h>
> #include <stdlib.h>
> #include <string.h>
>
> #include <gtk/gtk.h>
> #include <gdk/gdk.h>
> #include <gdk/gdkkeysyms.h>
> #include <glib.h>
>
> static GtkWidget *window=NULL;
>
>
> static void destroy(GtkWidget *widget,gpointer data) {
>   gtk_main_quit();
> }
>
> static void on_menu_activate(GtkMenuItem *menuitem,gpointer user_data) {
>   gtk_label_set_text(GTK_LABEL(user_data),"main menu");
> }
>
> static void on_context_menu1_activate(GtkMenuItem *menuitem,gpointer user_data) {
>   gtk_label_set_text(GTK_LABEL(user_data),"context menu 1");
> }
>
> static void on_context_menu2_activate(GtkMenuItem *menuitem,gpointer user_data) {
>   gtk_label_set_text(GTK_LABEL(user_data),"context menu 2");
> }
>
> static void on_popup_button_clicked(GtkButton *widget,gpointer user_data) {
>   gtk_menu_popup(GTK_MENU(user_data),NULL,NULL,NULL,NULL,3,gtk_get_current_event_time());
> }
>
> static void on_check_toggled(GtkToggleButton *togglebutton,gpointer user_data) {
>   if(gtk_toggle_button_get_active(togglebutton)) {
>     gtk_window_add_accel_group(GTK_WINDOW(window),GTK_ACCEL_GROUP(user_data));
>     puts("2nd accel group added");
>   }
>   else {
>     gtk_window_remove_accel_group(GTK_WINDOW(window),GTK_ACCEL_GROUP(user_data));
>     puts("2nd accel group removed");
>   }
> }
>
> static void init() {
>   GtkWidget *vbox,*hbox;
>   GtkWidget *button;
>   GtkWidget *mb,*pm,*sm,*mi;
>   GtkAccelGroup *accel_group;
>   GtkWidget *label,*ck;
>
>   window=gtk_window_new(GTK_WINDOW_TOPLEVEL);
>   gtk_window_set_title(GTK_WINDOW(window), "Popup with accelerator");
>   g_signal_connect(G_OBJECT(window), "destroy",       G_CALLBACK (destroy), NULL);
>
>   accel_group=gtk_accel_group_new();
>   gtk_window_add_accel_group(GTK_WINDOW(window),accel_group);
>
>   vbox=gtk_vbox_new(FALSE,0);
>   gtk_container_add(GTK_CONTAINER(window),vbox);
>
>   // message area
>   label=gtk_label_new("");
>
>   // regular menu
>   mb=gtk_menu_bar_new();
>   gtk_container_add(GTK_CONTAINER(vbox),mb);
>   mi=gtk_menu_item_new_with_label("Menu");
>   gtk_container_add(GTK_CONTAINER(mb),mi);
>
>   sm=gtk_menu_new();
>   gtk_menu_item_set_submenu(GTK_MENU_ITEM(mi),sm);
>
>   mi=gtk_image_menu_item_new_from_stock(GTK_STOCK_NEW,accel_group);
>   gtk_container_add(GTK_CONTAINER(sm),mi);
>   g_signal_connect(G_OBJECT(mi),"activate",G_CALLBACK(on_menu_activate),(gpointer)label);
>
>   hbox=gtk_hbox_new(FALSE,6);
>   gtk_container_add(GTK_CONTAINER(vbox),hbox);
>
>   // popup menu 1
>   pm=gtk_menu_new();
>   gtk_menu_set_accel_group(GTK_MENU(pm), accel_group);
>   gtk_menu_set_accel_path(GTK_MENU(pm),"<Main>/Context1");
>
>   mi=gtk_menu_item_new_with_label("popup item");
>   gtk_menu_item_set_accel_path (GTK_MENU_ITEM (mi), "<Main>/Context1/PopupItem");
>   gtk_accel_map_add_entry ("<Main>/Context1/PopupItem", GDK_P, GDK_CONTROL_MASK);
>   gtk_menu_shell_append(GTK_MENU_SHELL(pm),mi);
>   gtk_widget_show(mi);
>   g_signal_connect(G_OBJECT(mi),"activate",G_CALLBACK(on_context_menu1_activate),(gpointer)label);
>
>   button=gtk_button_new_with_label("Popup 1");
>   g_signal_connect(G_OBJECT(button), "clicked", G_CALLBACK(on_popup_button_clicked), (gpointer)pm);
>   gtk_container_add(GTK_CONTAINER(hbox),button);
>
>   // popup menu 2
>   accel_group=gtk_accel_group_new();
>   // if I only add it later, all accelerators get ignored
>   //gtk_window_add_accel_group(GTK_WINDOW(window),accel_group);
>
>   pm=gtk_menu_new();
>   gtk_menu_set_accel_group(GTK_MENU(pm), accel_group);
>   gtk_menu_set_accel_path(GTK_MENU(pm),"<Main>/Context2");
>
>   mi=gtk_menu_item_new_with_label("popup item");
>   gtk_menu_item_set_accel_path (GTK_MENU_ITEM (mi), "<Main>/Context2/PopupItem");
>   gtk_accel_map_add_entry ("<Main>/Context2/PopupItem", GDK_O, GDK_CONTROL_MASK);
>   gtk_menu_shell_append(GTK_MENU_SHELL(pm),mi);
>   gtk_widget_show(mi);
>   g_signal_connect(G_OBJECT(mi),"activate",G_CALLBACK(on_context_menu2_activate),(gpointer)label);
>
>   button=gtk_button_new_with_label("Popup 2");
>   g_signal_connect(G_OBJECT(button), "clicked", G_CALLBACK(on_popup_button_clicked), (gpointer)pm);
>   gtk_container_add(GTK_CONTAINER(hbox),button);
>
>   ck=gtk_check_button_new_with_label("<- enable accel");
>   gtk_container_add(GTK_CONTAINER(hbox),ck);
>   g_signal_connect (G_OBJECT(ck), "toggled", G_CALLBACK (on_check_toggled), (gpointer)accel_group);
>
>   // message area
>   gtk_container_add(GTK_CONTAINER(vbox),label);
>
>   gtk_widget_show_all(window);
> }
>
> static void done() {
>
> }
>
> int main(int argc, char **argv) {
>   gtk_init(&argc,&argv);
>
>   init();
>   gtk_main();
>   done();
>
>   return(0);
> }