I don't really care either for which it is; but maybe restoring this to the previous state is the safer solution.
FWIW, I don't use that shortcut (didn't even know it existed 😄) so I don't mind.

works for me, and I too really don't understand why the same call on treeview2 (the initial correction I suggested) doesn't work...

This is more subtle :) The reason is that the actual GtkTreeView is either the default empty one, or the document's one. So the one in Glade (retrieved with the treeview2 name) is likely not the one actually attached to the symbols tab.
The correct fix is:

diff --git a/src/sidebar.c b/src/sidebar.c
index 9d004f392..435e31394 100644
--- a/src/sidebar.c
+++ b/src/sidebar.c
@@ -1649,7 +1649,7 @@ void sidebar_focus_symbols_tab(void)
 	if (ui_prefs.sidebar_visible && interface_prefs.sidebar_symbol_visible)
 	{
 		GtkNotebook *notebook = GTK_NOTEBOOK(main_widgets.sidebar_notebook);
-		GtkWidget *symbol_list_scrollwin = gtk_notebook_get_nth_page(notebook, TREEVIEW_SYMBOL);
+		GtkWidget *symbol_list_scrollwin = ui_lookup_widget(main_widgets.window, "scrolledwindow2");
 
 		gtk_notebook_set_current_page(notebook, TREEVIEW_SYMBOL);
 		gtk_widget_grab_focus(gtk_bin_get_child(GTK_BIN(symbol_list_scrollwin)));


Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: <geany/geany/pull/3681/c1817231917@github.com>