Closed Bug 814630 Opened 12 years ago Closed 12 years ago

Thunderbird 17.0 closes all windows if multiple are opened

Categories

(Thunderbird :: Mail Window Front End, defect)

17 Branch
x86_64
Windows 7
defect
Not set
normal

Tracking

(thunderbird18 fixed, thunderbird19 fixed, thunderbird-esr1718+ fixed)

VERIFIED FIXED
Thunderbird 20.0
Tracking Status
thunderbird18 --- fixed
thunderbird19 --- fixed
thunderbird-esr17 18+ fixed

People

(Reporter: chke85, Assigned: Paenglab)

References

()

Details

(Keywords: regression, Whiteboard: [gs][regression:tb17])

Attachments

(1 file)

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:17.0) Gecko/17.0 Firefox/17.0
Build ID: 20121119183901

Steps to reproduce:

Reproduced with Windows 7 Ultimate x64 SP1 and Thunderbird 17.0
Steps:
1) open Thunderbird
2) open a Mail (or something else) in a new tab within the current window
3) drag the tab out of the current window scope
4) now close either the initial window or the newly created one

Additional:
After closing Thunderbird, open it anew and the multiple windows should start with the process. Now the steps 2-4 can be repeated to open more and more windows for the current instance and every time Thunderbird closes all windows.
Only the manual deletion of the window-JSONs in the profile's "session.json" reverts the instance to one window.


Actual results:

Thunderbird closes ALL windows, not only the selected one.


Expected results:

Only the selected window should close and the remaining windows should stay opened.
Thus draging tabs outer the current window scope should open stand-alone windows.
Probably related to the problem recently reported in gsfn:

https://getsatisfaction.com/mozilla_messaging/topics/two_message_windows_are_displayed_when_thunderbird_launches

Solution is the same, to delete session.json.
the might be related. but does deleting session.json really prevent all windows from being closed?  I wouldn't think so.

However, https://getsatisfaction.com/mozilla_messaging/topics/when_thunderbird_17_0_starts_on_xp_it_runs_two_or_three_copies#reply_10632085 is quite correct, setting mail.tabs.drawInTitlebar=false resolves the problem.

see https://getsatisfaction.com/mozilla_messaging/topics/new_window_close_behavior_in_tb_17_0
Status: UNCONFIRMED → NEW
Ever confirmed: true
Flags: in-testsuite?
Flags: in-moztrap?
Keywords: regression
See Also: → 805185
Whiteboard: [gs][regression:tb17]
The same happens when you use Exit from menu. With draw in title bar the close button is using "cmd_quit" like Exit in menu. This is on every Windows system with draw in title bar except Vista/Win7 Aero. On Aero the native buttons with their system function is used. It exists also "cmd_close" but this closes only one tab after the other until the last tab is closed.

Mike, do you know a function we can use?
Richard, FWIW, alt+F4 closes a single window just fine under these conditions
(In reply to Wayne Mery (:wsmwk) from comment #2)
> the might be related. but does deleting session.json really prevent all
> windows from being closed?  I wouldn't think so.
> 
> However,
> https://getsatisfaction.com/mozilla_messaging/topics/
> when_thunderbird_17_0_starts_on_xp_it_runs_two_or_three_copies#reply_10632085
> is quite correct, setting mail.tabs.drawInTitlebar=false resolves the
> problem.
> 
> see
> https://getsatisfaction.com/mozilla_messaging/topics/
> new_window_close_behavior_in_tb_17_0

I'll try this workaround in about ~ 36 hours, then I'll sit in front of the PC on which the error was reproduced.
And to note: I did not delete the session.json at all, I only changed the array property "windows" to one element thus resetting the amount of windows which are opened at the start of Thunderbird.
With due respect, changing the mail.tabs.drawInTitlebar value is only a workaround, no fix! If I call a close-window-function (it does not matter how) I expect that the window I selected is closed and not some other windows in the background. Then they should not appear in a separate window in the first place...
Alright, changing Thunderbird to the more "classical" window layout does change the close window behaviour. Now it does what I expect, it only closes the current window and not all open windows.

But the bug with the default window layout is still reproducible.
(In reply to Richard Marti [:Paenglab] from comment #3)
> Mike, do you know a function we can use?

window.close() might be sufficient - that's my first instinct on the matter.
Attached patch proposed fixSplinter Review
Yess Mike, window.close() is what we need.
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #685204 - Flags: review?(mconley)
FYI 
a) https://getsatisfaction.com/mozilla_messaging/topics/thunderbird_inbox_opens_twice describes bug 805185 - I just merged about 10 new gsfn topics created in the last 12 hours.
b) but not seeing nearly as many reports about clicking close terminates multiple TB  windows

As mentioned on IRC, it's not clear to me if bug 805185 is an outright duplicate because I thought I read a user report having never opened multiple windows. (which could be incorrect)
Comment on attachment 685204 [details] [diff] [review]
proposed fix

Review of attachment 685204 [details] [diff] [review]:
-----------------------------------------------------------------

Yep - this is what we want. Thanks Richard!
Attachment #685204 - Flags: review?(mconley) → review+
Keywords: checkin-needed
Comment on attachment 685204 [details] [diff] [review]
proposed fix

[Approval Request Comment]
This made a lot of GS traffic after release of TB 17. To solve this I think it would be best if this could land in the next TB 17 update.
Attachment #685204 - Flags: approval-comm-esr17?
Attachment #685204 - Flags: approval-comm-beta?
Attachment #685204 - Flags: approval-comm-aurora?
https://hg.mozilla.org/comm-central/rev/7a9edeb46df5
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 20.0
Attachment #685204 - Flags: approval-comm-esr17?
Attachment #685204 - Flags: approval-comm-esr17+
Attachment #685204 - Flags: approval-comm-beta?
Attachment #685204 - Flags: approval-comm-beta+
Attachment #685204 - Flags: approval-comm-aurora?
Attachment #685204 - Flags: approval-comm-aurora+
I would like to clear some points about this, as a user, as I have seen after using it for a while:

Probably a new feature of 'thunderbird 17', or whatever, If there where n windows when you close your thunderbird client, next time you open thunderbird, there will be n windows. If you had 2 windows when closing thunderbird, there will be 2 windows when you open it next time.
Now, what happens if thunderbird opens 2 windows each time you tell your system to open thunderbird.. let's say you open thunderbird, but you, for some reasson, even by accident, you click many times to open thunderbird: it will open many times, but by each time you clicked the thunderbird icon, it will open 2 windows. What does it mean? it means by every time you clicked the icon, you have 2 windows.
Now here comes the fun part: when you close any window, they will all close; but even funnier is that when you open it next time, all of those windows will open next time. Funny isn't it? you can only increase windows number.
Was there any point in restoring the windows number in a mail client? or was there any point that when you close any window, they will all close?
Depicco,

All you wrote is a result of this bug and should be fixed in the next update. Except what you wrote with multiple times clicking the icon. After opening the first time, and leave them open, a new icon click rises only the window/s into the front.
Yes, I had this too. I closed Thunderbird with two tabs. When I started it next time my computer was not responding (maybe busy with another task) so I impatiently clicked the TB desktop icon eight times. When TB finally started, it opened 16 inboxes and clicking the X to close any one, closed them all. Next startup 16 inboxes again.
Checked following trunk nightly build on MS Win-XP.
> Mozilla/5.0 (Windows NT 5.1; rv:20.0) Gecko/20100101 Thunderbird/20.0a1
> Application Build ID : 20121218064339
X button closed single window only, so excess windows were not saved in sessions.json, thus phenomenon of bug 805185 didn't occur any more.
Status: RESOLVED → VERIFIED
I observed this behaviour again in Thunderbird 31.1.2 on Win XP.
(In reply to markus.selve@web.de from comment #22)
> I observed this behaviour again in Thunderbird 31.1.2 on Win XP.

What do you call by your "this behaviour"?

This bug is for:
(i) Open multiple Messenger Windows(Tb's main 3 pane window) on Win-XP.
     Try to close only one window, by X button in _□X buttons at top/right corner of each window.
     => all multiple windows are closed, so multiple windows are opened again upon restart of Tb by session-restore.
This was Win-XP only problem.

This bug is never for next phenomenon.
(ii) Open multiple Messenger Windows(Tb's main 3 pane window) on any OS.
      Terminate Thunderbird by File/Exit menu.
     => all multiple windows are closed at once, except composition window, 
           and multiple windows are opened upon restart of Tb by session-restore.
This is design of  File/Exit menu of Thunderbird.

markus.selve@web.de, what was your actual operations?
On one click two instances of Thunderbird were opened.  The bug that describes this is marked as fixed by this bug.  I indeed closed via File/Exit menu.  I did not try the X button in the upper right corner.  Instead I circumvented the original bug by closing one of the instances with Alt-F4.  So maybe also the X button in the upper right corner would have closed the superfluous instance.
Do you know how I can re-create the original bug that two instances instead of one open?  I'd then try to close one of them with the X button in the upper right corner to proof that this bug is fixed.

Regards, Markus.
Right click on a folder in the left pane and select 'Open in New Window'.
(In reply to Richard Marti (:Paenglab) from comment #25)
> Right click on a folder in the left pane and select 'Open in New Window'.

I can confirm that this works and just one window closed.  So the bug described here is fixed.  I withdraw my statement that I observed the behaviour again.

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