Branch: refs/heads/master Author: Colomban Wendling ban@herbesfolles.org Committer: Colomban Wendling ban@herbesfolles.org Date: Thu, 21 Feb 2013 00:31:38 UTC Commit: a0dc8af9c4238fe435c7f45ef4c39c358b6803d2 https://github.com/geany/geany/commit/a0dc8af9c4238fe435c7f45ef4c39c358b6803...
Log Message: ----------- Fix GTK default keybindings being still active after being unbound
If we provide an AccelGroup when creating a menu item using a sock ID, it installs the GTK default accelerator, accelerator we can't remove since we don't know about it. So, don't give an AccelGroup so GTK don't install it's own accelerator.
This fix also required to properly update the accelerator on some item we used to ignore since the update didn't work anyway (since the GTK accelerator was displayed instead).
Note that this doesn't fix the fact the editor popup menu accelerators are never updated after startup so they don't get updated before restart after changing a keybinding in the preferences. This is a separate (and less problematic) issue due to a simple lack of update.
Closes #1912683 and #3599251.
Modified Paths: -------------- data/geany.glade doc/geany.txt src/keybindings.c
Modified: data/geany.glade 27 files changed, 0 insertions(+), 27 deletions(-) =================================================================== @@ -99,7 +99,6 @@ <property name="can_focus">False</property> <property name="use_underline">True</property> <property name="use_stock">True</property> - <property name="accel_group">accelgroup1</property> <signal name="activate" handler="on_undo1_activate" swapped="no"/> </object> </child> @@ -111,7 +110,6 @@ <property name="can_focus">False</property> <property name="use_underline">True</property> <property name="use_stock">True</property> - <property name="accel_group">accelgroup1</property> <signal name="activate" handler="on_redo1_activate" swapped="no"/> </object> </child> @@ -129,7 +127,6 @@ <property name="can_focus">False</property> <property name="use_underline">True</property> <property name="use_stock">True</property> - <property name="accel_group">accelgroup1</property> <signal name="activate" handler="on_cut1_activate" swapped="no"/> </object> </child> @@ -141,7 +138,6 @@ <property name="can_focus">False</property> <property name="use_underline">True</property> <property name="use_stock">True</property> - <property name="accel_group">accelgroup1</property> <signal name="activate" handler="on_copy1_activate" swapped="no"/> </object> </child> @@ -153,7 +149,6 @@ <property name="can_focus">False</property> <property name="use_underline">True</property> <property name="use_stock">True</property> - <property name="accel_group">accelgroup1</property> <signal name="activate" handler="on_paste1_activate" swapped="no"/> </object> </child> @@ -165,7 +160,6 @@ <property name="can_focus">False</property> <property name="use_underline">True</property> <property name="use_stock">True</property> - <property name="accel_group">accelgroup1</property> <signal name="activate" handler="on_delete1_activate" swapped="no"/> </object> </child> @@ -183,7 +177,6 @@ <property name="can_focus">False</property> <property name="use_underline">True</property> <property name="use_stock">True</property> - <property name="accel_group">accelgroup1</property> <signal name="activate" handler="on_menu_select_all1_activate" swapped="no"/> </object> </child> @@ -7333,7 +7326,6 @@ <property name="can_focus">False</property> <property name="use_underline">True</property> <property name="use_stock">True</property> - <property name="accel_group">accelgroup1</property> <signal name="activate" handler="on_new1_activate" swapped="no"/> </object> </child> @@ -7362,7 +7354,6 @@ <property name="can_focus">False</property> <property name="use_underline">True</property> <property name="use_stock">True</property> - <property name="accel_group">accelgroup1</property> <signal name="activate" handler="on_open1_activate" swapped="no"/> </object> </child> @@ -7399,7 +7390,6 @@ <property name="can_focus">False</property> <property name="use_underline">True</property> <property name="use_stock">True</property> - <property name="accel_group">accelgroup1</property> <signal name="activate" handler="on_save1_activate" swapped="no"/> </object> </child> @@ -7411,7 +7401,6 @@ <property name="can_focus">False</property> <property name="use_underline">True</property> <property name="use_stock">True</property> - <property name="accel_group">accelgroup1</property> <signal name="activate" handler="on_save_as1_activate" swapped="no"/> </object> </child> @@ -7477,7 +7466,6 @@ <property name="can_focus">False</property> <property name="use_underline">True</property> <property name="use_stock">True</property> - <property name="accel_group">accelgroup1</property> <signal name="activate" handler="on_file_properties_activate" swapped="no"/> </object> </child> @@ -7505,7 +7493,6 @@ <property name="can_focus">False</property> <property name="use_underline">True</property> <property name="use_stock">True</property> - <property name="accel_group">accelgroup1</property> <signal name="activate" handler="on_print1_activate" swapped="no"/> </object> </child> @@ -7523,7 +7510,6 @@ <property name="can_focus">False</property> <property name="use_underline">True</property> <property name="use_stock">True</property> - <property name="accel_group">accelgroup1</property> <signal name="activate" handler="on_close1_activate" swapped="no"/> </object> </child> @@ -7592,7 +7578,6 @@ <property name="can_focus">False</property> <property name="use_underline">True</property> <property name="use_stock">True</property> - <property name="accel_group">accelgroup1</property> <signal name="activate" handler="on_undo1_activate" swapped="no"/> </object> </child> @@ -7604,7 +7589,6 @@ <property name="can_focus">False</property> <property name="use_underline">True</property> <property name="use_stock">True</property> - <property name="accel_group">accelgroup1</property> <signal name="activate" handler="on_redo1_activate" swapped="no"/> </object> </child> @@ -7622,7 +7606,6 @@ <property name="can_focus">False</property> <property name="use_underline">True</property> <property name="use_stock">True</property> - <property name="accel_group">accelgroup1</property> <signal name="activate" handler="on_cut1_activate" swapped="no"/> </object> </child> @@ -7634,7 +7617,6 @@ <property name="can_focus">False</property> <property name="use_underline">True</property> <property name="use_stock">True</property> - <property name="accel_group">accelgroup1</property> <signal name="activate" handler="on_copy1_activate" swapped="no"/> </object> </child> @@ -7646,7 +7628,6 @@ <property name="can_focus">False</property> <property name="use_underline">True</property> <property name="use_stock">True</property> - <property name="accel_group">accelgroup1</property> <signal name="activate" handler="on_paste1_activate" swapped="no"/> </object> </child> @@ -7658,7 +7639,6 @@ <property name="can_focus">False</property> <property name="use_underline">True</property> <property name="use_stock">True</property> - <property name="accel_group">accelgroup1</property> <signal name="activate" handler="on_delete1_activate" swapped="no"/> </object> </child> @@ -7676,7 +7656,6 @@ <property name="can_focus">False</property> <property name="use_underline">True</property> <property name="use_stock">True</property> - <property name="accel_group">accelgroup1</property> <signal name="activate" handler="on_menu_select_all1_activate" swapped="no"/> </object> </child> @@ -8149,7 +8128,6 @@ <property name="can_focus">False</property> <property name="use_underline">True</property> <property name="use_stock">True</property> - <property name="accel_group">accelgroup1</property> <signal name="activate" handler="on_find1_activate" swapped="no"/> </object> </child> @@ -8563,7 +8541,6 @@ <property name="can_focus">False</property> <property name="use_underline">True</property> <property name="use_stock">True</property> - <property name="accel_group">accelgroup1</property> <signal name="activate" handler="on_zoom_in1_activate" swapped="no"/> </object> </child> @@ -8575,7 +8552,6 @@ <property name="can_focus">False</property> <property name="use_underline">True</property> <property name="use_stock">True</property> - <property name="accel_group">accelgroup1</property> <signal name="activate" handler="on_zoom_out1_activate" swapped="no"/> </object> </child> @@ -8587,7 +8563,6 @@ <property name="can_focus">False</property> <property name="use_underline">True</property> <property name="use_stock">True</property> - <property name="accel_group">accelgroup1</property> <signal name="activate" handler="on_normal_size1_activate" swapped="no"/> </object> </child> @@ -9130,7 +9105,6 @@ <property name="can_focus">False</property> <property name="use_underline">True</property> <property name="use_stock">True</property> - <property name="accel_group">accelgroup1</property> <signal name="activate" handler="on_project_properties1_activate" swapped="no"/> </object> </child> @@ -9325,7 +9299,6 @@ <property name="can_focus">False</property> <property name="use_underline">True</property> <property name="use_stock">True</property> - <property name="accel_group">accelgroup1</property> <signal name="activate" handler="on_info1_activate" swapped="no"/> </object> </child>
Modified: doc/geany.txt 8 files changed, 0 insertions(+), 8 deletions(-) =================================================================== @@ -3204,14 +3204,6 @@ want to assign to the action and it will be saved when you press OK. You can define only one key combination for each action and each key combination can only be defined for one action.
-Some of the default key combinations are common across many -applications, for example *Ctrl-N* for New and *Ctrl-O* for Open. -Because they are so common it is not advisable to change these, but -you can add other key combinations for these actions. For example -*Ctrl-O* is set to execute menu_open by default, but you can also -define *Alt-O*, so that the file open dialog is shown by pressing -either *Ctrl-O* or *Alt-O*. - The following tables list all customizable keyboard shortcuts, those which are common to many applications are marked with (C) after the shortcut.
Modified: src/keybindings.c 18 files changed, 12 insertions(+), 6 deletions(-) =================================================================== @@ -248,14 +248,14 @@ static void init_default_kb(void) group = keybindings_get_core_group(GEANY_KEY_GROUP_FILE);
add_kb(group, GEANY_KEYS_FILE_NEW, NULL, - GDK_n, GDK_CONTROL_MASK, "menu_new", _("New"), NULL); + GDK_n, GDK_CONTROL_MASK, "menu_new", _("New"), "menu_new1"); add_kb(group, GEANY_KEYS_FILE_OPEN, NULL, - GDK_o, GDK_CONTROL_MASK, "menu_open", _("Open"), NULL); + GDK_o, GDK_CONTROL_MASK, "menu_open", _("Open"), "menu_open1"); add_kb(group, GEANY_KEYS_FILE_OPENSELECTED, NULL, GDK_o, GDK_SHIFT_MASK | GDK_CONTROL_MASK, "menu_open_selected", _("Open selected file"), "menu_open_selected_file1"); add_kb(group, GEANY_KEYS_FILE_SAVE, NULL, - GDK_s, GDK_CONTROL_MASK, "menu_save", _("Save"), NULL); + GDK_s, GDK_CONTROL_MASK, "menu_save", _("Save"), "menu_save1"); add_kb(group, GEANY_KEYS_FILE_SAVEAS, NULL, 0, 0, "menu_saveas", _("Save as"), "menu_save_as1"); add_kb(group, GEANY_KEYS_FILE_SAVEALL, NULL, @@ -335,11 +335,11 @@ static void init_default_kb(void) group = keybindings_get_core_group(GEANY_KEY_GROUP_CLIPBOARD);
add_kb(group, GEANY_KEYS_CLIPBOARD_CUT, NULL, - GDK_x, GDK_CONTROL_MASK, "menu_cut", _("Cut"), NULL); + GDK_x, GDK_CONTROL_MASK, "menu_cut", _("Cut"), "menu_cut1"); add_kb(group, GEANY_KEYS_CLIPBOARD_COPY, NULL, - GDK_c, GDK_CONTROL_MASK, "menu_copy", _("Copy"), NULL); + GDK_c, GDK_CONTROL_MASK, "menu_copy", _("Copy"), "menu_copy1"); add_kb(group, GEANY_KEYS_CLIPBOARD_PASTE, NULL, - GDK_v, GDK_CONTROL_MASK, "menu_paste", _("Paste"), NULL); + GDK_v, GDK_CONTROL_MASK, "menu_paste", _("Paste"), "menu_paste1"); add_kb(group, GEANY_KEYS_CLIPBOARD_COPYLINE, NULL, GDK_c, GDK_CONTROL_MASK | GDK_SHIFT_MASK, "edit_copyline", _("_Copy Current Line(s)"), "copy_current_lines1"); @@ -729,6 +729,7 @@ static void add_menu_accel(GeanyKeyGroup *group, guint kb_id, GtkWidget *menuite add_menu_accel(group, kb_id, ui_lookup_widget(main_widgets.editor_menu, G_STRINGIFY(wid)))
/* set the menu item accelerator shortcuts (just for visibility, they are handled anyway) */ +/* FIXME: update those during runtime */ static void add_popup_menu_accels(void) { GeanyKeyGroup *group; @@ -738,6 +739,11 @@ static void add_popup_menu_accels(void) GEANY_ADD_POPUP_ACCEL(GEANY_KEYS_EDITOR_REDO, redo1); GEANY_ADD_POPUP_ACCEL(GEANY_KEYS_EDITOR_CONTEXTACTION, context_action1);
+ group = keybindings_get_core_group(GEANY_KEY_GROUP_CLIPBOARD); + GEANY_ADD_POPUP_ACCEL(GEANY_KEYS_CLIPBOARD_CUT, cut1); + GEANY_ADD_POPUP_ACCEL(GEANY_KEYS_CLIPBOARD_COPY, copy1); + GEANY_ADD_POPUP_ACCEL(GEANY_KEYS_CLIPBOARD_PASTE, paste1); + group = keybindings_get_core_group(GEANY_KEY_GROUP_SELECT); GEANY_ADD_POPUP_ACCEL(GEANY_KEYS_SELECT_ALL, menu_select_all2);
-------------- This E-Mail was brought to you by github_commit_mail.py (Source: https://github.com/geany/infrastructure).