GNOME Bugzilla – Bug 139633
WM_DELETE_WINDOW deletes a panel w/o confirmation
Last modified: 2004-12-22 21:47:04 UTC
Description of Problem: When a panel receives a "close" command (X event WM_DELETE_WINDOW, afaik), it gets deleted without asking the user for confirmation. Steps to reproduce the problem: 1. focus the panel by clicking on it or "cycling" through the set of open windows. The panel is focused when e.g. a dotted line appears along the edge of a "main menu" button inside the panel. 2. Hit the keystroke defined for "close window". Actual Results: The panels disappears immediately without asking the user for confirmation. Expected Results: The user should be asked to confirm the operation, as is is the case if the panel is closed via "Delete this Panel..." from the context menu. How often does this happen? always Additional Information: According to various user reports, the following configurations are affected by this problem: - Gnome 2.4 / Sawfish - Gnome 2.4 / xfwm4 - Gnome 2.6 / Sawfish The following configurations are apparently *not* affected: - Gnome 2.4 / Metacity - Gnome 2.6 / Metacity
confirming.
Same here; I'd like to add that the panel is not actually deleted. It's just "hidden". Restarting (killall) gnome-panel makes it reappear.
BTW, I don't think the panel should display a confirmation dialog when Alt+F4 is invoked on a panel. When this happens, it is almost certainly a user mistake, and it would be quite confusing to pop up scary dialogs. IMO gnome-panel should just ignore this event (but prevent the deletion).
Created attachment 29563 [details] [review] Patch: Ignore the delete events.
Sounds reasonable, but is it really a show stopper for 2.8.0?
It may not be a show stopper for you, but it is an unnerving bug with a trivial fix. I don't think it needs a large amount of thinking about.
The problem is that we should have applied it before the hard code freeze. I don't think that this bug is important enough to break the freeze for 2.8.0. But the patch should probably be applied for 2.8.1.
Sorry, I didn't know that you were frozen already.
Chris: Why did you change the Gnome Milestone from 2.8.x to 2.10.0? Vincent said he thought the patch should go in for 2.8.1...
Yes, sorry for that.
Comment on attachment 29563 [details] [review] Patch: Ignore the delete events. Please commit to gnome-2-8 branch and HEAD
I committed the patch. Thanks