After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 133885 - Add support for "use_filechooser" in GnomeFileEntry
Add support for "use_filechooser" in GnomeFileEntry
Status: RESOLVED FIXED
Product: libgnomeui
Classification: Deprecated
Component: general
CVS HEAD
Other Linux
: Low minor
: future
Assigned To: libgnomeui maintainers
Damon Chaplin
Depends on: 132043
Blocks:
 
 
Reported: 2004-02-09 12:54 UTC by Fernando Herrera
Modified: 2005-07-24 17:59 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch adding this prop to glade (2.64 KB, patch)
2004-02-09 12:56 UTC, Fernando Herrera
committed Details | Review

Description Fernando Herrera 2004-02-09 12:55:34 UTC
After the solution of bug #132043, glade should add the property
"use_filechooser" to GnomeFileEntry in order to allow apps to use new
GtkFileChooser. Attaching the patch.
Comment 1 Fernando Herrera 2004-02-09 12:56:13 UTC
Created attachment 24228 [details] [review]
Patch adding this prop to glade
Comment 2 Damon Chaplin 2004-02-21 16:46:49 UTC
Why have you got this function in glade-gnome.c in libgnomeui:

static void
file_entry_set_use_filechooser (GladeXML *xml, GtkWidget *w,
			  const char *name, const char *value)
{
    g_object_set (G_OBJECT (w), "use_filechooser", BOOL (value), NULL);
}

That isn't necessary, is it? If the widget has a working property you
don't need to register a custom property in libglade.

I suppose we should also add the property to GnomePixmapEntry in Glade.
Comment 3 Damon Chaplin 2004-02-21 17:06:37 UTC
I've committed the patch, and also added support for generating code,
and for GnomePixmapEntry.

Though I haven't tested it yet, so I'll leave this bug open until then.
Comment 4 Damon Chaplin 2004-02-23 15:22:12 UTC
Glade now works.

Moving to libgnomeui because of the unnecessary code mentioned above.
Comment 5 Fernando Herrera 2004-02-24 23:22:53 UTC
Humm, all the props in libgnomeui have this kind of code on
gnome-glade.c because of that I added it.