213922 - Pages rendered via a HREF TARGET= tag lose keyboard navigation focus

Status: RESOLVED FIXED

(Core :: DOM: UI Events & Focus Handling, defect)

Description

[Ron Higgins]

User-Agent:       Mozilla/4.0 (compatible; MSIE 6.0; Windows NT 5.1; .NET CLR 1.0.3705)
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.4) Gecko/20030624

A page rendered in a new browser window via a TARGET= tag in an HREF loses the 
ability to use keyboard navigation.  Page-Up/Page-Down/Arrow keys, etc....  
The focus appears to go to the address/location bar instead of the browser 
window contents.

Reproducible: Always

Steps to Reproduce:
1. Load this page http://www.bigscreen.com/Shop/index.php
2. Click the Charlies Angles pic which spawns a new window
3. Try to scroll up and down the page using the keyboard arrow keys

Actual Results:  
No ability to scroll page using the keyboard.  (mouse wheel works though)

Expected Results:  
Focus should be on the document instead of the address bar.  It didn't used to 
be this way in previous versions to the best of my knowledge.

Browser theme doesn't affect this problem.  (modern and classic behaviour is 
the same)

Comment 1

[Boris Zbarsky [:bzbarsky]]

didn't we fix this once, jag?

Comment 2

[Ron Higgins]

Attachment: small html file to demonstrate problem

Click the link, try to scroll

Comment 3

[Asa Dotzler [:asa]]

Ugh. We fail this testcase with Mozilla 1.7 Alpha and Firefox 0.8

Comment 4

[Johnny Stenback (:jst)]

Looks like this is seamonkey only now, I can't reproduce in FireFox, but I can
in SeaMonkey.

Comment 5

[Johnny Stenback (:jst)]

Attachment: This fixes the focus problem in this case.

This fixes the problem in this case, but it might break other cases. Still
investigating if more is needed...

Comment 6

[Ben Goodger (use ben at mozilla dot org for email)]

Comment on attachment 143524 [details] [diff] [review]
This fixes the focus problem in this case.

It might be worth just adding a brief mention to the comment describing the
case you mentioned about window.open("");

sr=[e-mail]

Comment 7

[Johnny Stenback (:jst)]

Comment on attachment 143524 [details] [diff] [review]
This fixes the focus problem in this case.

After thinking more about this, and testing more, and talking to people about
this fix, it's looking like this is the way to go. I've added the comment Ben
requested, now requesting approval to land this in 1.7b (once I have r=).

Comment 8

[Brendan Eich [:brendan]]

Comment on attachment 143524 [details] [diff] [review]
This fixes the focus problem in this case.

a=[e-mail] for 1.7b.

/be

Comment 9

[Johnny Stenback (:jst)]

Fix checked in.

Comment 10

[Boris Zbarsky [:bzbarsky]]

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