Re: Patch to remove unneeded accelerators in mail-config.c

Srinivasa Ragavan <[email protected]>
Newsgroups gmane.comp.gnome.evolution.patches
Message-ID <[email protected]>
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
> [email protected]
> http://mail.gnome.org/mailman/listinfo/evolution-patches
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.