Re: [evolution-patches] Patch to remove unneeded accelerators in mail-config.c



KJartan,

Agreed, but, these are used in the context menu as well and not having
accels in them will kill a lot. I wont agree to fix this bug, this way.

The DEFAULT should have accels in the r-click menu. Just while adding to
config data, remove the underscores. Btw, this patch will work only for
new users as, this struct is uploaded to gconf on first load, and never
updated from the code. So any change in the code, wont reflect gconf and
so the UI, which is a different issue(?) altogether.

If possible, just remove the UNDERSCORES in the place where we fill the
entry in the config, and leave it as it is in the menu. When the user
renames a label (Dont know the real use case though), let him decide to
have a new "label" and a underscore if he wants in the context menu.

-Srini

On Thu, 2006-08-17 at 15:44 +0200, Kjartan Maraas wrote:
> These look horrible in the user interface and have no meaning at all.
> 
> Cheers
> Kjartan
> 
> _______________________________________________
> Evolution-patches mailing list
> Evolution-patches gnome org
> http://mail.gnome.org/mailman/listinfo/evolution-patches




[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]