About window needs a close button

VERIFIED FIXED in Firefox 7

Status

()

Firefox
General
VERIFIED FIXED
7 years ago
6 years ago

People

(Reporter: monreal, Assigned: pascalc)

Tracking

(Blocks: 1 bug)

Trunk
Firefox 7
All
Linux
Points:
---

Firefox Tracking Flags

(firefox5 affected, firefox6 affected)

Details

Attachments

(1 attachment, 2 obsolete attachments)

(Reporter)

Description

7 years ago
Created attachment 487535 [details]
Screenshot

The redesigned About window really needs a close button. There are system configurations (at least on Linux) which do not have a way to close this window atm:

a) some tiling window managers which do not show window bars

b) the GNOME3 default theme does not show window buttons on "about" windows (see screenshot)
There's some discussion on this at https://bugzilla.redhat.com/677427
(Assignee)

Updated

6 years ago
Duplicate of this bug: 656491

Updated

6 years ago
Attachment #487535 - Attachment is obsolete: true
Depends on: 633812
(Assignee)

Comment 3

6 years ago
Created attachment 536039 [details] [diff] [review]
patch fixing the bug

this patch fixes the bug for me on Gnome 3
Attachment #536039 - Flags: review?(margaret.leibovic)

Comment 4

6 years ago
Comment on attachment 536039 [details] [diff] [review]
patch fixing the bug

This is probably wrong for Windows, since this change would affect any non-OSX platform.

Dialog windows are supposed to have close buttons, so I think something may actually be wrong with your window manager, not the about dialog. Testing on Ubuntu 10.04, the about dialog already has a close button without this patch applied. How are dialog windows normally handled by your platform? Does esc work to close them? What do about dialogs look like for other applications?

Comment 5

6 years ago
See GNOME HIG:

  http://developer.gnome.org/hig-book/3.0/windows-dialog.html.en

"Window Controls: Minimize/Roll-up"

Note the lack of "close".

GNOME designers say that dialogs shouldn't have a close button because they require the user to take a specific action, and it may be unclear what the close button does.
Comment on attachment 536039 [details] [diff] [review]
patch fixing the bug

Yes, the about window currently provided is not a dialog (because it doesn't provide any controls for user interaction), but I agree that it is safest to leave this as is for MS XP_WIN systems (unless there is a known issue there, and it has been been tested on those systems).

One side-effect here though is that the GTK port currently doesn't correctly set WM_TRANSIENT_FOR for non-dialog windows.  That means that the window manager will not keep the about window above its "parent" window.  That should be fixed, but it's not necessarily a prerequisite for this fix.  The correctness of "dependent" here is debatable (bug 611955, and I wonder whether bug 611955 comment 3 provides a reason why this window should not be presented as a dialog on NT systems).

Fortunately centerscreen seems to still have no effect, on my system at least.
(I'm not clear what positions the window over its "parent".)

(In reply to comment #4)
> Dialog windows are supposed to have close buttons, so I think something may
> actually be wrong with your window manager, not the about dialog.

Dialogs should provide controls for users to take part in the dialogue.

When we use _NET_WM_WINDOW_TYPE to tell the window manager that it is a dialog, it is appropriate for the window manager to use this "in determining the decoration".  Dialogs should already have controls, so there is no need for the window manager to provide additional controls.

> Testing on
> Ubuntu 10.04, the about dialog already has a close button without this patch
> applied. How are dialog windows normally handled by your platform? Does esc
> work to close them? What do about dialogs look like for other applications?

On my system (with kwin) evince's about window is a dialog with credits, license, and close buttons.  Similarly okular's about window is a dialog with a close button (provided by the app).  kwin chooses to provide an additional close button in its decorations even for dialog windows.

Restoring the close button to make the about window a dialog would be an acceptable solution, but Firefox's about window already looks so different from other apps' windows that I don't think there's a need to imitate them.

Esc closes all these windows.
Attachment #536039 - Flags: feedback+
(Assignee)

Comment 7

6 years ago
Created attachment 536221 [details] [diff] [review]
updated patch

Here is an updated patch preserving the Windows options.
Assignee: nobody → pascalc
Attachment #536039 - Attachment is obsolete: true
Attachment #536221 - Flags: review?(margaret.leibovic)
Attachment #536039 - Flags: review?(margaret.leibovic)

Comment 8

6 years ago
Comment on attachment 536221 [details] [diff] [review]
updated patch

I can't officially review this, but it looks good to me.
Attachment #536221 - Flags: review?(margaret.leibovic)
Attachment #536221 - Flags: review?(gavin.sharp)
Attachment #536221 - Flags: feedback+
(Assignee)

Comment 9

6 years ago
thanks Margaret, I'll wait for Gavin's review then :)
Attachment #536221 - Flags: review?(gavin.sharp) → review+
(Assignee)

Updated

6 years ago
Keywords: checkin-needed
(Assignee)

Comment 10

6 years ago
thanks Gavin, if I understand well the terminology, should I set the flags as : 
tracking-firefox5 : ?
status-firefox5   : affected
tracking-firefox6 : ?
status-firefox6   : affected
tracking-firefox7 : ?
status-firefox7   : affected

Or should I target one release at a time? Or maybe I shouldn't touch any of these flags ? :)
status-firefox5: --- → affected
status-firefox6: --- → affected
tracking-firefox5: --- → ?
tracking-firefox6: --- → ?
Hardware: x86 → All
Keywords: checkin-needed
Whiteboard: [fixed in cedar]
No need to touch tracking flags, those are used for bugs that need to be monitored before the branch point. You can request approval on the patch, but this doesn't seem like something critical enough to warrant landing on mozilla-aurora or mozilla-beta.
tracking-firefox5: ? → ---
tracking-firefox6: ? → ---
Pushed:
http://hg.mozilla.org/mozilla-central/rev/082845fb7841
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Whiteboard: [fixed in cedar]
Target Milestone: --- → Firefox 7
Duplicate of this bug: 661346
Blocks: 633812
No longer depends on: 633812

Updated

6 years ago
Duplicate of this bug: 667208

Comment 15

6 years ago
Mozilla/5.0 (X11; Linux i686; rv:7.0a1) Gecko/20110627 Firefox/7.0a1

Verified this on Ubuntu 11.04 x86 that has Gnome 3 installed. Close button present in about window -> setting status as Verified Fixed.
Status: RESOLVED → VERIFIED
Duplicate of this bug: 677029
You need to log in before you can comment on or make changes to this bug.