[Geany-Devel] 'shiftcolumn' plugin - keyboard shortcut conflict
Steven Blatnick
steve8track at xxxxx
Tue Mar 25 23:26:47 UTC 2014
The shortcut code would look something like this:
enum
{
KB_PLUGIN,
KB_GROUP
};
...
key_group = plugin_set_key_group(geany_plugin,
"plugin_name_keyboard_shortcut", KB_GROUP, NULL);
keybindings_set_item(key_group, KB_PLUGIN, shortcut_callback, 0, 0,
"plugin_name", _("Shortcut Name"), NULL);
Here is a simple plugin that uses that structure:
https://github.com/sblatnick/geany-plugins/blob/quick-search/quick-search/src/quick-search.c
Here is another plugin that allows dynamically adding shortcuts (as
opposed to a fixed number):
https://github.com/sblatnick/geany-plugins/blob/external-tools/external-tools/src/tool.c
line 64
I hope this helps,
Steve
On 03/25/2014 05:12 PM, Lex Trotman wrote:
> On 26 March 2014 09:33, Shankhoneer Chakrovarty <shankhoneer at gmail.com> wrote:
>> Hi,
>>
>> 'shiftcolumn' is one of the orphaned plugin which I have decided to work on.
>> shiftcolumn's default shortcut key to move the selected text to right is
>> <Primary>0 which conflicts with "Zoom reset", making the plugin useless to
>> move the text right unless for every move the user clicks Tools->Shift
>> Right. Shift left is done by <Primary>9 key which is working fine.
>>
>> Also, this plugin doesnt have its plugin preferences dialog box.
>>
>> I was planning to change the default shortcut key for the aforementioned
>> behavior of the plugin to <Primary>8 which AFAIK doesnt conflict with
>> anything.
>>
>> Do you guys think its a good idea? What should I do to prevent such problems
>> to arise in future?
> It is actually preferable that plugins do not have default keybindings.
>
> It is not known what combination plugins will be loaded and so it is
> not known what combination of keybindings might clash, and it is not
> known what keybindings a user might define themselves that can clash.
>
> So I would recommend that default keybindings be removed from plugins.
>
> Cheers
> Lex
>
>> Thanks,
>> Shankhoneer Chakrovarty
>>
>> _______________________________________________
>> Devel mailing list
>> Devel at lists.geany.org
>> https://lists.geany.org/cgi-bin/mailman/listinfo/devel
>>
> _______________________________________________
> Devel mailing list
> Devel at lists.geany.org
> https://lists.geany.org/cgi-bin/mailman/listinfo/devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.geany.org/pipermail/devel/attachments/20140325/6366619c/attachment.html>
More information about the Devel
mailing list