Can,<br><br>BTW I have never even seen this dialog, so for finding it, and noticing the inconsistency, and trying to do something about it you deserve thanks for the effort.<br><br>Maybe you should be promoted to user interface quality manager :-)<br>
<br>Cheers<br>Lex<br><br><div class="gmail_quote">On 2 March 2010 09:53, 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;">
OK, you send a patch then.<br>
<br>
--- On Mon, 3/1/10, Lex Trotman <<a href="mailto:elextr@gmail.com">elextr@gmail.com</a>> wrote:<br>
<br>
> From: Lex Trotman <<a href="mailto:elextr@gmail.com">elextr@gmail.com</a>><br>
> Subject: Re: [Geany-devel] [Patch] - Fix keyboard accelerator on Resave Missing File dialog.<br>
> To: "Geany development list" <<a href="mailto:geany-devel@uvena.de">geany-devel@uvena.de</a>><br>
> Date: Monday, March 1, 2010, 9:55 PM<br>
<div><div></div><div class="h5">><br>
><br>
> On 2 March 2010 05:25, Can Koy<br>
> <<a href="mailto:cankoy@ymail.com">cankoy@ymail.com</a>><br>
> wrote:<br>
><br>
> This patch fixes keyboard accelerator on Resave Missing<br>
> File dialog.<br>
><br>
><br>
><br>
><br>
><br>
> I have a couple of problems with this patch, sorry if they<br>
> applied to some of your previous ones but I didn't have<br>
> time to look at them before.<br>
><br>
> 1. Terminology only, this is changing the mnemonic, not the<br>
> keyboard accelerator, although the GTK docs are not totally<br>
> consistent about it :-)<br>
><br>
><br>
> A mnemonic is the character you type (usually with Alt) to<br>
> activate a visible menu or dialog item and must be unique<br>
> only within a single menu or dialog, a keyboard accelerator<br>
> is a keycode (like ctrl-shift-w) that activates<br>
> functionality directly, visible or not and must be globally<br>
> unique within Geany.<br>
><br>
><br>
> 2. The standard mnemonic for close is always 'c'<br>
> which is adhered to in most applications and other Geany<br>
> dialogs, but this patch is changing this one from the<br>
> standard, this is not good user interface practice, if<br>
> 'c' clashes then the other should be changed, its<br>
> 'closes' mnemonic<br>
><br>
><br>
> 3. by changing it from the stock item there is no icon<br>
> associated with the button, again non-standard and won't<br>
> match the other buttons<br>
><br>
> Cheers<br>
> Lex<br>
><br>
><br>
>       <br>
> _______________________________________________<br>
><br>
> Geany-devel mailing list<br>
><br>
> <a href="mailto:Geany-devel@uvena.de">Geany-devel@uvena.de</a><br>
><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>
><br>
><br>
><br>
><br>
</div></div>> -----Inline Attachment Follows-----<br>
<div><div></div><div class="h5">><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>
<br>
<br>
<br>
<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>
</div></div></blockquote></div><br>