Re: [evolution-patches] patch for review(45775 calendar keyboard navigation)

[Ettore Perazzoli <ettore ximian com>]

On Sun, 2003-09-28 at 10:01, wu yang wrote:
> I have revised it follow your advice.
> This patch is for evolution calendar keyboard navigation.
> Bug#45775
> "in DayView, WeekView, Task: Alt+o, open the editing dialog for current selected item"
> The same bug has fix in 1.4 a month ago.
> This patch is for HEAD.

Hmm I am not sure why I didn't notice this before, but this seems
broken.

Alt-letter is for widget accelerators, like menus and buttons. 

But in this case, it's a shortcut for a menu item, so it should use
Ctrl-something instead (and probably just be bound through the Bonobo
menu shortcut stuff, and not manually through the widget).

If you hardcode it to Alt-O there is also a chance that some translation
of Evolution will have a menu starting with "_O" and that will cause the
widget shortcut not to work.

Or am I missing something?

-- Ettore