Open Bug 319528 Opened 19 years ago Updated 2 years ago

File->Quit, "Confirm close" shows in other workspace

Categories

(Firefox :: Tabbed Browser, defect)

x86
Linux
defect

Tracking

()

People

(Reporter: ginnchen+exoracle, Unassigned)

References

Details

Attachments

(1 obsolete file)

A customer complained about this.

1)Open a firefox window with 2 tabs in Workspace 1
2)Open a new firefox window in Workspace 2
3)Press Ctrl+Q in firefox in Workspace 2
4)Nothing happens in Workspace 2 (few people can notice that "Confirm close" is opened in Workspace 1)
Attached patch patch (obsolete) — Splinter Review
It can bring "Confirm close" dialog to current workspace.
But the dialog says "You can about to close 2 open tabs", I think it's still some kind of confusing.
Any idea?
Change the summary, since Ctrl+Q is gone.
Summary: Press Ctrl+Q, "Confirm close" shows in other workspace → File->Quit, "Confirm close" shows in other workspace
Attachment #205291 - Flags: review?(mconnor)
This seems like a strange hack.  I think I have a patch that addresses this better without mucking with focus.  Basically, it pops a dialog from the window that quit is called from, with something like "you are about to close 2 windows with 3 open tabs, do you wish to quit?"  Its a better and less-confusing dialog (if the dialog appears on the window with one tab, it doesn't make sense) and it would be called if you have seven windows instead of seven tabs.
Comment on attachment 205291 [details] [diff] [review]
patch

Agree
Attachment #205291 - Attachment is obsolete: true
Attachment #205291 - Flags: review?(mconnor)
Blocks: 326400
*** Bug 292342 has been marked as a duplicate of this bug. ***
Actually, this is probably just us not setting the right transient hint on the window when we do the popup.  The window manager will switch workspaces for you if the hint is set right.
That came across wrong.  The window manager will switch the workspace the window is on (it will move it to the current workspace) if the hint is properly set.
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: