Re: [evolution-patches] fix for 57653 : toolbar visibility

[William Jon McCann <mccannwj pha jhu edu>]

Anna M Dirks wrote:
> I don't have Evo built from CVS (and hence can't apply the patch),
> but...
>
> however, I can say that the HIG does indeed specify that toolbars should
> be hideable. As long as all of the options which are accessible through
> the toolbars are accessible through the menus too (which obviously they
> should be), then this option is fine (and appropriate) to adopt. 
>
> BTW, you should give the new menu item a keyboard shortcut.

Hi Anna,

One thing I just noticed about my patch is that is uses the mnemonic "T" 
for the menu item.  This is kind of the de facto standard for this item 
but I don't think it is required.

The problem is that "T" is used in the View menu for:

Mail
  _Threaded Message List Ctrl+T

Calendars
  Select _Today Ctrl+Alt+T


What do you recommend?  I suppose the most important thing is that the 
mnemonic be consistent across components.

On a related note, I would like to submit patches that add View menu 
items to toggle the visibility of the sidebar and the search bar too. 
They would have similar issues.

Thanks,
Jon