@b4n requested changes on this pull request.
LGTM (obviously) but for the tiny style issue.
> @@ -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)
```suggestion
GtkWidget *symbol_list_scrollwin =
ui_lookup_widget(main_widgets.window, "scrolledwindow2");
```
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3681#pullrequestreview-1738758343
You are receiving this because you are subscribed to this thread.
Message ID: <geany/geany/pull/3681/review/[email protected]>
