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 131073 - Every instance of weather applet should have separate instance of about dialog
Every instance of weather applet should have separate instance of about dialog
Status: RESOLVED FIXED
Product: gnome-applets
Classification: Other
Component: gweather
2.5.x
Other Linux
: Normal normal
: ---
Assigned To: gnome-applets Maintainers
gnome-applets Maintainers
Depends on:
Blocks:
 
 
Reported: 2004-01-10 09:48 UTC by Archana Shah
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Patch which fixes the bug (3.73 KB, patch)
2004-01-10 09:53 UTC, Archana Shah 		none Details | Review

Description Archana Shah 2004-01-10 09:48:31 UTC 
Steps to reproduce:

1. Add_to panel->Accessories->Weather report
2. Add multiple instances of this applet.
3. Open about dialog.

One about dialog is shared by all the instances of gweather applet.
Comment 1 Archana Shah 2004-01-10 09:53:43 UTC 
Created attachment 23195 [details] [review]
Patch which fixes the bug
Comment 2 Kevin Vandersloot 2004-01-12 14:13:08 UTC 
Please commit. Thanks :)
Comment 3 Arvind S N 2004-01-13 07:13:00 UTC 
Kevin: committed to head with addition of destroying the about dialog
when the
applet is removed from the panel.

2004-01-12 Archana Shah <archana.shah@wipro.com>

        * gweather.h: Added a field in the structure GWeatherApplet for
        about dialog.
        * gweather-applet.c: (gweather-applet-create): Initialized
        about_dialog field with NULL.
        (applet_destroy): Destroys the about dialog when the applet is
        removed from the panel.
        * gweather-about.c: (gweather-about-run): Removed the static
        variable "about" and used structure member about_dialog in
that place.
        Fixes bug #131073