On 27/11/2012 13:27, Amit Sengupta wrote:
hey! It's about the bug

http://sourceforge.net/tracker/?func=detail&aid=3587465&group_id=153444&atid=787791

i tried to debug it by changing the buttons to YES No buttons, I figured
out that the document.c is the file associated with this bug.

So in the detect/reload dialog we would have buttons:
[_Close] [_No] [_Yes]

instead of:
[_Close] [_Cancel] [_Reload]

to fix the _c clashing mnemonic. This seems ok to me, but personally I wonder if:

[_Close] [_No] [_Reload]

is better, as it avoids listing no,yes which is a little odd in order (albeit the Gnome HIG order), and having Reload instead of Yes is clearer IMO.

What does everyone think?

Can anyone please help me with this so that I can test this, also how to
submit the code (ie if I get it right)

I don't mind making the change myself as it's easy (once we all agree), so in this case it's not that necessary. But the usual way is to create a patch file in unified diff format. Google 'unified diff patch'.

_______________________________________________
Devel mailing list
Devel@lists.geany.org
https://lists.geany.org/cgi-bin/mailman/listinfo/devel

Reply via email to