306235 - keyboard stops working on window restore from minimized state

Status: RESOLVED FIXED

(Thunderbird :: Mail Window Front End, defect)

Description

[Jason Ellis]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8b4) Gecko/20050825 Firefox/1.0+
Build Identifier: 20050826 branch

If you minimize a Thunderbird window and restore it, the that window stops
accepting input from the keyboard of any kind -- menu shortcuts, arrow keys, etc.  

Bringing another Thunderbird window to the front (using the mouse) and then
going back to the problem window fixes the issue.

Reproducible: Always

Steps to Reproduce:
1. Minimize the main Thunderbird window
2. Restore the main Thunderbird window
3. Press Ctrl-M
4. Press Alt-F

Actual Results:  
Nothing happens

Expected Results:  
New composition window should appear, then file menu should appear

I've been using Thunderbird Alpha 2 and just switched to nightlies with the
20050826 branch build, so I'm not sure when this regressed (assuming others see it).

Comment 1

[Jason Ellis]

Regression date info: I see this issue in the 20050817 branch build but not in
the 20050816 branch build.

Comment 2

[Joe Sabash [:JoeS1]]

I see this also in Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8b4)
Gecko/20050828 Thunderbird/1.0+ ID:2005082804
Further clarification/symptoms:
Restore down (small window) restore up (large window) restores functionality.

If you open the indox, and select a message, then minimize and restore, you lose
focus on the message. Restore down, then up re-focuses the selected message.

Could not verify the regression date, but possibly bug #302120 checkin on 08/16
cc to Neil on the possible regression

Comment 3

[[e-mail]]

Bug 302120 doesn't relate to the main mail window.

I can reproduce this in a Suite nightly, so it might be a Windows widget bug.

However, it would be helpful if you could get an accurate regression range.

Comment 4

[Joe Sabash [:JoeS1]]

Confirming original posters regression range
tb version 1.0+ (20050816) works fine

Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8b4) Gecko/20050817
Thunderbird/1.0+ ID:2005081707
Shows the symtoms described in the original bug report.

Comment 5

[Ere Maijala (slow)]

I see this in my branch build too. Needs to be fixed before next beta. 

Comment 6

[Scott MacGregor]

agree this bug needs to block 1.8b4. triage team: we need to plus this bug.

This regression was caused by aaronl's change for Bug 258285. Backing that
change out of the branch makes keyboard inputs work again for me. 

Comment 7

[Aaron Leventhal]

Attachment: Very low risk. Partial backout of bug 258285 -- only changes to nsWebShellWindow.cpp

This is very low risk because it is patching out a change that was made for
only one of the testcases which caused bug 258285.

The testcase was:
1) Make google.com your home page
2) Minimze all copies of Firefox
3) Launch Firefox from an icon on your desktop

Apparently this part of the fix wasn't necessary to fix that anymore. After
some of the other pieces of the fix for bug 258285 were developed, they also
took care of this testcase.

Comment 8

[Aaron Leventhal]

Comment on attachment 194456 [details] [diff] [review]
Very low risk. Partial backout of bug 258285 -- only changes to nsWebShellWindow.cpp

Brian, you might as well give me r+sr=. This is just backing out an atomic
change that we don't need.

Comment 9

[Louie Zhao]

*** Bug 305587 has been marked as a duplicate of this bug. ***

Comment 10

[Aaron Leventhal]

Comment on attachment 194456 [details] [diff] [review]
Very low risk. Partial backout of bug 258285 -- only changes to nsWebShellWindow.cpp

Very low risk -- backout of atomic change from recent fix for bug 258285.

Comment 11

[Aaron Leventhal]

This change caused bug 305032 (launch from new icon after minimize) to break again.

We never had all that working in the first place, but fixing it caused this bug.

Right now we have to choose between fixing that bug and this one.

Comment 12

[Aaron Leventhal]

Reopening. We need one fix that simultaneously solves both this and bug 305032.

Comment 13

[Mats Palmgren (inactive)]

Aaron, is there a way to reproduce this bug using only Firefox?
or SeaMonkey Mail? (given that I reverse the attached patch of course)

Comment 14

[Scott MacGregor]

you should be able to see this in seamonkey mail too. Make sure you have a
message selected and focused int eh thread pane. then minimize, maximize. 

Comment 15

[Adam Guthrie]

*** Bug 305601 has been marked as a duplicate of this bug. ***

Comment 16

[Ria Klaassen (not reading all bugmail)]

Yes, I see also this problem in Firefox. 
Regression range 1.8b4_2005081606 (works) and 1.8b4_2005081615 (fails).

Comment 17

[Ria Klaassen (not reading all bugmail)]

The problem in Firefox is only reproducible for me with middle sized windows,
not with maximized windows.  

Comment 18

[Aaron Leventhal]

Fixed by checkin to bug 305032.

Comment 19

[Jay Patel [:jay]]

v.fixed with today's branch respin: Mozilla/5.0 (Windows; U; Windows NT 5.1;
en-US; rv:1.8b4) Gecko/20050906 Firefox/1.4