Re: [evolution-patches] [shell]patch to fix focus issue of Account Assistant

[Not Zed <notzed ximian com>]

Probably got ignored since the subject says shell, and its got nothing
to do with the shell.

Instead of commenting it out, just remove the line entirely.

Still, what it does is supposed to work, without this line, what happens
to the keyboard focus when you change the transport type?


On Mon, 2004-03-29 at 15:44 +0800, Eric Zhao wrote:
> Hi,
>    I posted this patch a month ago, but has no response. Would you
> please spend little time reviewing it? Thanks!
> 
> Atttached is a patch to fix the Tab navigation bug of Account
> Assistant, which can be reproduced by the following steps:
> (1) Invoke  your evo,
> (2) Select Tool->Settings...
> (3) Select mail Account->Add to open Account Assistant
> After the Account Assistant is initiated, you can find that using Tab 
> key to navigate around "Cancel", "Forward" buttons is impossible.
> 
>    This bug is caused by the following code in
> evolution/mail/mail-account-gui.c:618 
>         gtk_widget_grab_focus (GTK_WIDGET (gui->transport.hostname));
> This sentence transfers the focus to the hostname edit, but when
> this function is called, the wizard is just on the start page, and
> the hostname edit is unvisible, so Tab navigation failed.
> 
>    Also I found that the code can't get the expected result -- set focus
> to the host edit when the wizard is on the "Sending Email" page. So I
> just commented this sentence.
> 
> 
> -- 
> Eric Zhao <eric zhao sun com>
>