Re: GtkTreeView "activate" signal patch

[Tim Janik <timj gtk org>]

On Fri, 16 Mar 2001, Mikael Hermansson wrote:

> gint
> cell_render_button_press(GtkCellRenderer *cell, GtkTreeView *tview,
> GdkEvent)
> {
>   do_the_test_to_check_if_its_ok_to_emit_the_signal()
> 
>   g_signal_emit(tree_view, "row_activated");
> }
> 
> This is also good to use for the checkbox/editable cells etc.
> 
> > > Is this patch ok to submit?
> > 
> > Of course it's okay to submit. (-: I'll read through it and apply it,
> > though...
> 
> The patch I sent to the list is broken. I forgot to store the
> tree_view_signal[ACTIVATE]  id :-/

not only that, but activate signals also have to stored in
GTK_WIDGET_GET_CLASS (widget)->activate_signal_id.
i'm not sure activation on double clicks shouldn't be left to
the tree view's user though.

> My new "local" patch also seems to be broken (sometimes coredumps on
> doubleclick).
> 
> It seems like the gtk_tree_view_button_press function needs
> some cleanups before we inserts more stuff to it :-) 
> 
> Greets
> 
> M.H.
> 

---
ciaoTJ