[wx-dev] Re: new GTK assert dialog

[Vadim Zeitlin]

On Sun, 03 Dec 2006 16:16:19 +0100 Francesco Montorsi <f18m_cpp217828 at yahoo.it> wrote:

FM> Francesco Montorsi ha scritto:
FM> > Index: src/gtk/assertdlg_gtk.c
FM> > ===================================================================
FM> > RCS file: /pack/cvsroots/wxwidgets/wxWidgets/src/gtk/assertdlg_gtk.c,v
FM> > retrieving revision 1.3
FM> > diff -bu2 -r1.3 src/gtk/assertdlg_gtk.c
FM> > --- src/gtk/assertdlg_gtk.c
FM> > +++ src/gtk/assertdlg_gtk.c
FM> > @@ -72,5 +72,5 @@
FM> >  }
FM> >  
FM> > -void gtk_assert_dialog_add_button (GtkAssertDialog *dlg, const gchar *label,
FM> > +GtkWidget *gtk_assert_dialog_add_button (GtkAssertDialog *dlg, const gchar *label,
FM> >                                     const gchar *stock, gint response_id)
FM> cvs diff ignored the additional spaces I added to the "const gchar 
FM> *stock, gint response_id)" line (to make it indented correctly)... ;)

 This is probably because you have "-b" in your .cvsrc.

 Anyhow, I've applied the patch with a small change: I put the checkbox to
the left of the buttons, not to the right, as it looks better like this
IMHO.

 However there is still a rather big problem left: if you show the assert
dialog in the assert sample at least once, you can't exit it any more: the
Exit menu command as well as "x" button in the title bar don't do anything.
I guess this is because there is an open frame in the background but I
didn't look at it in the details. Could you please check what's going on?

 TIA,
VZ