Closed Bug 213922 Opened 20 years ago Closed 20 years ago

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


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

Windows XP
Not set





(Reporter: rhiggins, Unassigned)




(Keywords: regression)


(2 files)

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
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)
didn't we fix this once, jag?
Click the link, try to scroll
Ugh. We fail this testcase with Mozilla 1.7 Alpha and Firefox 0.8
Assignee: general → events
Component: Browser-General → Event Handling
Ever confirmed: true
Flags: blocking1.7b+
Keywords: regression
QA Contact: general → ian
Looks like this is seamonkey only now, I can't reproduce in FireFox, but I can
in SeaMonkey.
This fixes the problem in this case, but it might break other cases. Still
investigating if more is needed...
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"");
Attachment #143524 - Flags: superreview+
Attachment #143524 - Flags: review?(bryner)
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=).
Attachment #143524 - Flags: approval1.7b?
Attachment #143524 - Flags: review?(bryner) → review+
Comment on attachment 143524 [details] [diff] [review]
This fixes the focus problem in this case. for 1.7b.

Attachment #143524 - Flags: approval1.7b? → approval1.7b+
Fix checked in.
Closed: 20 years ago
Resolution: --- → FIXED
*** Bug 206960 has been marked as a duplicate of this bug. ***
Component: Event Handling → User events and focus handling
You need to log in before you can comment on or make changes to this bug.