Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
@b4n requested changes on this pull request.
LGTM (obviously) but for the tiny style issue.
In src/sidebar.c:
> @@ -1649,8 +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");
Please keep the blank line between the variable declaration block and the rest (yes, it's kind of the same here, but that's currently how we do things everywhere else)
⬇️ Suggested change- GtkWidget *symbol_list_scrollwin = ui_lookup_widget(main_widgets.window, "scrolledwindow2"); + GtkWidget *symbol_list_scrollwin = ui_lookup_widget(main_widgets.window, "scrolledwindow2");
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.