Same keyboard shortcut for 3 elements in Add account dialog
Stu Tomlinson
stu at nosnilmot.com
Thu Aug 26 16:09:07 EDT 2010
On Thu, 2010-08-26 at 12:30 -0700, Paul Aurich wrote:
> From gtkaccount.c:
> tmp = g_strconcat("_", purple_account_option_get_text(option), NULL);
> opt_entry->widget = check = gtk_check_button_new_with_mnemonic(tmp);
> g_free(tmp);
>
> This strikes me as totally wrong (although if it's possible to 'tab'
> between entries with the same mnemonic, I guess it's less wrong). Added
> in 90c2a6492b85a718acbed863733e736cea512906.
I agree it's "wrong", and I realized that when I committed it, but I
don't think it's less wrong than having no mnemonic - and it's possible
to cycle between entries both by pressing 'tab' and by repeatedly
pressing the mnemonic (which cycles only between those that share the
mnemonic).
Obviously a more correct fix would be to push the mnemonics down to the
prpls, but just doing that with '_' indicators would cause problems for
other UIs/toolkits that don't use the same notation, so...
Regards,
Stu.
More information about the Support
mailing list