@b4n could you just merge your branch with the resolved conflicts and remove
the defaults for the new keybindings? I guess this would least work and we
would get this one done.
---
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/858#issuecomment-184441388
> +
> + if (!tag_goto_popup)
> + create_goto_popup();
> +
> + model = gtk_tree_view_get_model(GTK_TREE_VIEW(tag_goto_tree_view));
> + gtk_list_store_clear(GTK_LIST_STORE(model));
> +
> + foreach_ptr_array(tmtag, i, tags)
> + {
> + gchar *fname = g_path_ge
> +
> + model = gtk_tree_view_get_model(GTK_TREE_VIEW(tag_goto_tree_view));
> + gtk_list_store_clear(GTK_LIST_STORE(model));
> +
> + foreach_ptr_array(tmtag, i, tags)
> + {
> + gchar *fname = g_path_get_basename(tmtag->file->file_name);
> + gchar *text;
> +
>
Current situation (and USERLIST one suggested above too) has bad usability with
the mouse (ctrl+click) as it requires a double click. As there is a primary
mouse-based trigger, I think it's important to fix this.
(It probably could be nice to also directly select on single click from
Scintilla'
> +
> + if (!tag_goto_popup)
> + create_goto_popup();
> +
> + model = gtk_tree_view_get_model(GTK_TREE_VIEW(tag_goto_tree_view));
> + gtk_list_store_clear(GTK_LIST_STORE(model));
> +
> + foreach_ptr_array(tmtag, i, tags)
> + {
> + gchar *fname = g_path_ge
> +
> + model = gtk_tree_view_get_model(GTK_TREE_VIEW(tag_goto_tree_view));
> + gtk_list_store_clear(GTK_LIST_STORE(model));
> +
> + foreach_ptr_array(tmtag, i, tags)
> + {
> + gchar *fname = g_path_get_basename(tmtag->file->file_name);
> + gchar *text;
> +
>
Sorry about the typo. Have made changes according to suggestions made by @b4n .
---
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany-plugins/pull/342#issuecomment-184218627
