Signal editor (was Re: [Glade-devel] small patch)
bighead@users.sourceforge.net
bighead@users.sourceforge.net
13 Jan 2004 17:09:59 -0500
eww (at the theme, or is that Windows) :D
Archit
"Joaquin Cuenca Abela" <e98cuenc@free.fr> writes:
> Hi!
>
> I've been hacking a bit on the signal editor.
>
> What do you think of an interface like that one:
>
> http://e98cuenc.free.fr/signals.png
>
> You add a new signal handler just typing the name of the handler, and you
> remove it just erasing the name of the handler.
>
> I want to also add the name of the class that each signal belongs (as with
> the current "add a signal handler dialog").
>
> I've just done the interface of the screenshot, but I've not yet made it to
> work (that will be for tomorrow).
>
> It looks to me much simpler to use than the current one (for those that
> don't know, the current one is a clone of the glade2 one).
>
> Cheers,
>
> Paolo wrote:
> >
> > On Tue, 2004-01-13 at 14:10, Sridhar R wrote:
> > > Hi,
> > >
> >
> > Hi Sridhar,
> > I've not tried the patch (yet), but if I understand it
> > correctly the result is that, e.g. for a GtkButton, the
> > GtkButton signals list node in the tree is expanded.
> > Did I get it right? If yes, I like it :) [if someone else
> > doesn't feel free to voice your opinion]
> >
> > Some comments (nitpicks mostly) on the patch follow:
> >
> > 1) it seems to me that your mailer ate the patch a bit (it
> > breaks some lines). If you use Evolution you can send the
> > patch as an attachment and it will be sent in plain text,
> > don't know if other mail apps do the same...
> >
> > > I'm all new to collabarative developments. Here is my
> > > (initial) small patch to CVS glade-signal-editor.c, regarding
> > > usability. That is, it will now expand the first row of
> > signal-list
> > > dialog, which is what any user would do more often.
> > >
> > > Here goes the diff (diff --context=2 . .)
> >
> > please use diff -pu (the p option tells in which function the
> > changes are made, making the patch more readable. The default
> > context (3 iirc is just fine).
> > Beside also provide a ChangeLog entry.
> >
> > > *** ../../glade3.orig/src/glade-signal-editor.c Mon Jan 12 21:38:09
> > > 2004
> > > --- glade-signal-editor.c Mon Jan 12 22:04:06
> > > 2004
> > > ***************
> > > *** 158,161 ****
> > > --- 158,162 ----
> > > GtkTreeIter *parent = NULL;
> > > GList *list = NULL;
> > > + GtkTreePath *path_first = NULL;
> > > GladeWidgetClassSignal *signal;
> > >
> >
> > No need to initialize local vars to NULL as far as I can see,
> > while you are at it remove it also from the other vars;
> > beside, not related to your code, but usually a TreeIter is
> > allocated on the stack: i.e.
> > GtkTreeIter iter;
> > gtk_tree_functio (..., &iter, ...);
> >
> > > ***************
> > > *** 174,177 ****
> > > --- 175,182 ----
> > >
> > > glade_signal_editor_dialog_append_signal (lst_model,
> > > signal->nam
> > > e, parent);
> > > }
> > > + /* Sridhar R: Expand the first row */
> > > + path_first = gtk_tree_path_new_first();
> > > + gtk_tree_view_expand_row(GTK_TREE_VIEW (view),
> > > path_first, FALSE);
> > > + gtk_tree_path_free(path_first);
> > > }
> > >
> >
> > We usually don't add comments like "paolo: did this" for each
> > patch because they would fastly clutter the code. In this
> > particular case the whole comment seems superflous, since
> > it's just stating what the 3 line under it do.
> >
> > > ***************
> > > *** 183,186 ****
> > > --- 188,193 ----
> > > gint response;
> > >
> > > + g_assert(editor);
> > > + g_assert(editor->class);
> > > g_return_if_fail (editor->class->signals != NULL);
> > >
> >
> > Use g_return_if_fail instead of assert to check function args.
> >
> > > ***************
> > > *** 663,665 ****
> > > }
> > > }
> > > -
> > > --- 670,671 ----
> > > I have also attached the diff file.
> > >
> > > Well, what are all the stuffs that I can start working on?
> > >
> >
> > If you want to continue with ui stuff on the signal editor
> > there is a detailed bug in bugzilla.gnome.org, suggesting
> > some changes that can be made. I don't have the bug number at
> > hand, but since at the moment the open bugs against glade3
> > are just 2, you should find it pretty easily :)
> >
> > Thanks for your work, I'm sure that once you get started
> > things will go more smoothly!
> >
> > ciao
> > paolo
>
>
> _______________________________________________
> Glade-devel maillist - Glade-devel@lists.ximian.com
> http://lists.ximian.com/mailman/listinfo/glade-devel