[Geany-Devel] 'shiftcolumn' plugin - keyboard shortcut conflict

[Steven Blatnick]

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>