<br><br><div class="gmail_quote">On 2 March 2010 05:25, Can Koy <span dir="ltr"><<a href="mailto:cankoy@ymail.com">cankoy@ymail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
This patch fixes keyboard accelerator on Resave Missing File dialog.<br>
<br>
<br></blockquote><div><br>I have a couple of problems with this patch, sorry if they applied to some of your previous ones but I didn't have time to look at them before.<br><br>1. Terminology only, this is changing the mnemonic, not the keyboard accelerator, although the GTK docs are not totally consistent about it :-)<br>
<br>A mnemonic is the character you type (usually with Alt) to activate a visible menu or dialog item and must be unique only within a single menu or dialog, a keyboard accelerator is a keycode (like ctrl-shift-w) that activates functionality directly, visible or not and must be globally unique within Geany.<br>
<br>2. The standard mnemonic for close is always 'c' which is adhered to in most applications and other Geany dialogs, but this patch is changing this one from the standard, this is not good user interface practice, if 'c' clashes then the other should be changed, its 'closes' mnemonic<br>
<br>3. by changing it from the stock item there is no icon associated with the button, again non-standard and won't match the other buttons<br><br>Cheers<br>Lex<br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

      <br>_______________________________________________<br>
Geany-devel mailing list<br>
<a href="mailto:Geany-devel@uvena.de">Geany-devel@uvena.de</a><br>
<a href="http://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel" target="_blank">http://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel</a><br>
<br></blockquote></div><br>