Bug 677427 - how to close the "about firefox" window in gnome-shell?
Summary: how to close the "about firefox" window in gnome-shell?
Keywords:
Status: CLOSED UPSTREAM
Alias: None
Product: Fedora
Classification: Fedora
Component: firefox
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Martin Stransky
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
: 677861 685018 702960 704084 706048 708408 716396 716581 728692 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-02-14 18:53 UTC by Thomas Meyer
Modified: 2018-04-11 18:12 UTC (History)
18 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2011-08-06 13:41:08 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Mozilla Foundation 608934 0 None None None Never
Mozilla Foundation 633812 0 None None None Never

Description Thomas Meyer 2011-02-14 18:53:46 UTC
Description of problem:
run firefox under gnome-shell and select (in firefox) help->about firefox.

try to close this windows with the mouse!

you can close the window by pressing Alt+Space and then choose "close".

Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:


Additional info:

Comment 1 Owen Taylor 2011-02-14 19:19:35 UTC
right click on titlebar, select close.

Firefox is setting _NET_WM_ALLOWED_ACTIONS and not including _NET_WM_ACTION_CLOSE, Metacity and Mutter obey that and omit the close button.

Comment 2 Thomas Meyer 2011-02-14 19:59:18 UTC
okay, thanks. escape key also does the trick. The question is: why was the window menu icon removed in gnome-shell?

Comment 3 Christopher Aillon 2011-02-14 20:18:39 UTC
(In reply to comment #1)
> Firefox is setting _NET_WM_ALLOWED_ACTIONS and not including
> _NET_WM_ACTION_CLOSE, Metacity and Mutter obey that and omit the close button.

Looks like it's because of:
http://mxr.mozilla.org/mozilla-central/source/widget/src/gtk2/nsWindow.cpp#3920
3920                     gtk_window_set_deletable(GTK_WINDOW(mShell), mBorderStyle & eBorderStyle_close);

http://library.gnome.org/devel/gtk/unstable/GtkWindow.html#gtk-window-set-deletable
By default, windows have a close button in the window frame. Some window managers allow GTK+ to disable this button. If you set the deletable property to FALSE using this function, GTK+ will do its best to convince the window manager not to show a close button. Depending on the system, this function may not have any effect when called on a window that is already visible, so you should call it before calling gtk_window_show().

Comment 4 Karl Tomlinson 2011-02-14 21:14:55 UTC
(In reply to comment #1)
> Firefox is setting _NET_WM_ALLOWED_ACTIONS and not including
> _NET_WM_ACTION_CLOSE, Metacity and Mutter obey that and omit the close button.

I hope not as _NET_WM_ALLOWED_ACTIONS is managed by the window manager.

Could gnome-shell be treating _NET_WM_WINDOW_TYPE_DIALOG windows specially?

Comment 5 Owen Taylor 2011-02-14 21:37:14 UTC
(In reply to comment #4)
> (In reply to comment #1)
> > Firefox is setting _NET_WM_ALLOWED_ACTIONS and not including
> > _NET_WM_ACTION_CLOSE, Metacity and Mutter obey that and omit the close button.
> 
> I hope not as _NET_WM_ALLOWED_ACTIONS is managed by the window manager.
> 
> Could gnome-shell be treating _NET_WM_WINDOW_TYPE_DIALOG windows specially?

Yeah, I was wrong about that - in investigation, GNOME 3 simply does not include close buttons on any dialogs. (It's hacked up in the window manager theme, in a way I still haven't quite figured out, so I didn't see the change go in, but it is intentional.)

So really, the bug here is simpler - if Firefox is going to mark the about dialog as a dialog in GNOME, it needs to be closable without window manager assistance.

Comment 6 Karl Tomlinson 2011-02-14 22:11:25 UTC
Sounds reasonable if "dialog" implies something that is requesting a response from the user.  Firefox 3.6 had a close button, so this would be a regression in Firefox 4.

Gecko seems to be using _NET_WM_WINDOW_TYPE_DIALOG for all WM_TRANSIENT_FOR windows and perhaps it should not.

Gecko may not be alone because "If _NET_WM_WINDOW_TYPE is not set, then managed windows with WM_TRANSIENT_FOR set MUST be taken as [_NET_WM_WINDOW_TYPE_DIALOG]"

Comment 9 Christopher Aillon 2011-03-15 16:45:02 UTC
*** Bug 685018 has been marked as a duplicate of this bug. ***

Comment 10 Elad Alfassa 2011-05-27 16:37:24 UTC
*** Bug 708408 has been marked as a duplicate of this bug. ***

Comment 11 Christopher Aillon 2011-05-27 18:42:38 UTC
*** Bug 706048 has been marked as a duplicate of this bug. ***

Comment 12 Matěj Cepl 2011-06-13 17:24:54 UTC
*** Bug 702960 has been marked as a duplicate of this bug. ***

Comment 13 Elad Alfassa 2011-06-25 08:15:16 UTC
*** Bug 716581 has been marked as a duplicate of this bug. ***

Comment 14 Christopher Aillon 2011-07-06 18:04:13 UTC
*** Bug 677861 has been marked as a duplicate of this bug. ***

Comment 15 Christopher Aillon 2011-07-06 18:04:25 UTC
*** Bug 704084 has been marked as a duplicate of this bug. ***

Comment 16 Christopher Aillon 2011-07-06 18:04:42 UTC
*** Bug 716396 has been marked as a duplicate of this bug. ***

Comment 17 Matěj Cepl 2011-08-06 13:38:56 UTC
*** Bug 728692 has been marked as a duplicate of this bug. ***

Comment 18 Matěj Cepl 2011-08-06 13:41:08 UTC
We filed this already bug in the upstream database (https://bugzilla.mozilla.org/show_bug.cgi?id=608934) and believe that it is more appropriate to let it be resolved upstream.

We will continue to track the issue in the centralized upstream bug tracker, and will review any bug fixes that become available for consideration in future updates.

We would strongly encourage you to subscribe to the upstream bug as well (by logging in and adding yourself to the Cc list), so that you can provide whatever information required for the successful resolving of this issue.

Thank you for helping to make free software better.


Note You need to log in before you can comment on or make changes to this bug.