@techee commented on this pull request.
> + gtk_container_add(GTK_CONTAINER(ui_widgets.config_files_menu), item);
+ menu = gtk_menu_new();
+ gtk_menu_item_set_submenu(GTK_MENU_ITEM(item), menu);
+ list = utils_get_config_files(GEANY_TEMPLATES_SUBDIR);
+ foreach_slist(node, list)
+ {
+ gchar *fname = node->data;
+
+ SETPTR(fname, g_build_filename(app->configdir,
GEANY_TEMPLATES_SUBDIR, fname, NULL));
+ if (!g_file_test(fname, G_FILE_TEST_IS_DIR))
+ ui_add_config_file_menu_item(fname, NULL,
GTK_CONTAINER(menu));
+ g_free(fname);
+ }
+ g_slist_free(list);
+
+ item = gtk_menu_item_new_with_mnemonic(_("Files"));
Shouldn't the Files submenu be also at the start of the list, similarly to
what's done below in compare_menu_item_labels()?
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3396#pullrequestreview-1661950210
You are receiving this because you are subscribed to this thread.
Message ID: <geany/geany/pull/3396/review/1661950...@github.com>
