Hi,<br><br>Thanks very much for the input!  I've pasted an updated version here [1] which should implement your suggestions (assuming I understand them correctly).<br><br>Cheers<br>Alex<br><br>[1] <a href="http://pastebin.com/WJXYQm1N">http://pastebin.com/WJXYQm1N</a><br>
<br><div class="gmail_quote">On Tue, Nov 15, 2011 at 6:45 PM, Lex Trotman <span dir="ltr"><<a href="mailto:elextr@gmail.com">elextr@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="HOEnZb"><div class="h5">On Wed, Nov 16, 2011 at 8:29 AM, Meyer <<a href="mailto:3m.meyer@gmail.com">3m.meyer@gmail.com</a>> wrote:<br>
> Hi all,<br>
><br>
> I was looking for some programming to do recently and found this feature<br>
> request:<br>
> <a href="https://sourceforge.net/tracker/?func=detail&aid=3383201&group_id=153444&atid=787794" target="_blank">https://sourceforge.net/tracker/?func=detail&aid=3383201&group_id=153444&atid=787794</a><br>

> .  I tried writing a plugin which implements the feature, and it seems to<br>
> work, but now I'm not sure what to do with it now.  I've pasted the code<br>
> here: <a href="http://pastebin.com/549Dazj0" target="_blank">http://pastebin.com/549Dazj0</a>.  My main concerns are that it seems very<br>
> hack-ish, and seems too simple to be a full plugin.<br>
><br>
> Any advice regarding the code, what to do with it, etc. would be much<br>
> appreciated.<br>
<br>
</div></div>Hi,<br>
<br>
Nice simple plugin, but very fixed in operation, I have a couple of suggestions.<br>
<br>
1. Use keybindings rather than the editor notify. This lets you catch<br>
the key before it is inserted in Scintilla and allows the user to set<br>
the trigger keycode (for me the actual characters would not be<br>
appropriate, but <ctrl><shift>char would be, but different users would<br>
have different use-cases).  See for eg addons plugin for how to use<br>
keybindings.<br>
2. Group the changes you make into one undo group, see the scintilla docs.<br>
3. Add a preferences dialog to allow the opening and the respective<br>
closing character(s) to be set for each trigger keycode.  It would of<br>
course default to your current pairs but would allow updating for<br>
other languages and multi character sequences.  See again for eg<br>
addons.<br>
4. Have 1 and 3 allow up to say eight codes/pairs to be defined.<br>
<br>
The plugins infrastructure is about to get moved to Github and some<br>
new processes are being defined, see<br>
<a href="http://news.gmane.org/find-root.php?message_id=%3c4EC29977.1030802%40frank.uvena.de%3e" target="_blank">http://news.gmane.org/find-root.php?message_id=%3c4EC29977.1030802%40frank.uvena.de%3e</a><br>
so keep an eye on that thread and you can be the first new plugin<br>
using the new process :)<br>
<br>
Cheers<br>
Lex<br>
<br>
><br>
> Thanks,<br>
> Alex<br>
><br>
> _______________________________________________<br>
> Geany-devel mailing list<br>
> <a href="mailto:Geany-devel@uvena.de">Geany-devel@uvena.de</a><br>
> <a href="https://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel" target="_blank">https://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel</a><br>
><br>
><br>
_______________________________________________<br>
Geany-devel mailing list<br>
<a href="mailto:Geany-devel@uvena.de">Geany-devel@uvena.de</a><br>
<a href="https://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel" target="_blank">https://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel</a><br>
</blockquote></div><br>