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 84462 - Confirmation for removal of a drawer
Confirmation for removal of a drawer
Status: RESOLVED FIXED
Product: gnome-panel
Classification: Other
Component: panel
1.5.x
Other other
: Normal enhancement
: ---
Assigned To: Panel Maintainers
Panel Maintainers
Depends on:
Blocks:
 
 
Reported: 2002-06-07 07:21 UTC by Christian Marillat
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: Unversioned Enhancement



Description Christian Marillat 2002-06-07 07:21:21 UTC
Hi,

today I missed while clicking and accidently removed a drawer full of
stuff, including several sub-drawers.  It would be nice to have a
dialog, "Are you sure you want to remove this drawer?"
Comment 1 Vincent Untz 2002-10-26 21:46:32 UTC
same problem for applets: bug #95007.
Comment 2 Arvind S N 2002-10-28 03:55:39 UTC
Mark:

Deleting the drawer has an inconsistency.
You could delete the drawer in 2 ways.
1)Right click on the panel and do a "Delete this panel"
2)Right click on the drawer button and "Remove from panel"

In the first case you get a confirmation dialog because that's linked
with the generic panel code but not in the case of second one because
that's on the drawer button . Also just being off the topic here, the
drawer's gconf is not cleaned up when you delete it from "Remove from
panel" since it is not following the "Delete panel" code path. 
Comment 3 Arvind S N 2003-02-14 14:06:11 UTC
Fixed in HEAD and gnome-2-2 branch.

2003-02-14  Arvind Samptur <arvind.samptur@wipro.com>
        * applet.c: (applet_idle_remove) Deleting the drawer panel
        using the drawer button queries the user.

        * panel.[ch]: (panel_delete_query) (remove_panel_accept):
        When deleting the panel, clean up drawers if they exist.

        *panel-widget.[ch]: (panel_widget_remove_drawers): Cleanup 
        the gconf of the drawers recursively, if they each drawer
        holds another drawer.