[evolution-patches] Re: seek review for bug 47024: implement set/get_caret_offset method of AtkText interface

[Radek Doulík <rodo ximian com>]

Hi York,

On Tue, 2003-07-29 at 04:00, Yuedong Du wrote:
> Hi Radek,
> 
> Seems you do not notice this mail.
> Change the subject to the 'nomarlize form'.  And resend.
> 
> Regards
> York
> 
> Hi Radek,
> 
> The patch mainly implemented get/set_caret_offset method. After this
> patch
> gnopernicus can report current character when you move cursor, and can
> set
> caret to some text using at-poke. 
> 
> Some explain of the patch below,
> 
> I introduced a method to get/set current gtkhtml widget in html_utils,
> as below:
> 
> 
>         +static GtkHTML * gtkhtml = NULL;
>         +                                                                                 
>         +void
>         +html_utils_set_gtk_html(GtkHTML * p)
>         +{
>         +        gtkhtml = p;
>         +}
>         +                                                                                 
>         +GtkHTML *
>         +html_utils_get_gtk_html(void)
>         +{
>         +        return gtkhtml;
>          }
>         
>         
> I notice original code seems try to using the
> html_a11y_get_gtkhtml_parent() to reach the same goal.
> But the function do not work. We cannot get the gtkhtml widget from a
> html object.

do you now why does this function not work? remembering widget in static
variable will not work for multiple gtkhtml instances in one process. I
will prefer to fix html_a11y_get_gtkhtml_parent rather.

Cheers
Radek