Closed Bug 453831 Opened 13 years ago Closed 13 years ago
Focus in limbo after multi-tab launch pref set
1. Under Tools->Options->Main-> Startup group box->When Firefox starts combobox: Change to Show my windows and tabs from last time. 2. Load 2 simple pages (not Google which sets focus onload) 3. Exit and relaunch Accessibility focus is now on the top level window, instead of the document.
Actually, it's not just accessibility focus. Focus in general is broken. I can tell this because when even a single page is loaded with this pref set, arrows won't scroll (obvious using a long enough, scrollable page to test).
Summary: Incorrect focus events for multi-tab launch → Focus in limbo after multi-tab launch pref set
Component: Disability Access APIs → Session Restore
Product: Core → Firefox
QA Contact: accessibility-apis → session.restore
This confuses screen readers as well as messes up regular keyboard users. I suspect it's a simple thing to fix. Something like: instead of focusing windows, focus documents.
It's indeed a one-line fix. Nice catch, Aaron.
Assignee: nobody → zeniko
Status: NEW → ASSIGNED
Attachment #337063 - Flags: review?(dietrich)
Attachment #337063 - Flags: review?(dietrich) → review+
OS: Windows XP → All
Hardware: PC → All
Target Milestone: --- → Firefox 3.1b1
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Comment on attachment 337063 [details] [diff] [review] focus content instead of window This is a simple fix which affects both screen reader access and regular keyboard access after launch.
Attachment #337063 - Flags: approval184.108.40.206?
Attachment #337063 - Flags: approval220.127.116.11? → approval18.104.22.168+
Comment on attachment 337063 [details] [diff] [review] focus content instead of window Approved for 22.214.171.124, a=dveditz for release-drivers
Comment on attachment 337063 [details] [diff] [review] focus content instead of window Didn't make 126.96.36.199. Nominating for 188.8.131.52. We'll revisit this soon.
Attachment #337063 - Flags: approval184.108.40.206? → approval220.127.116.11+
Comment on attachment 337063 [details] [diff] [review] focus content instead of window Approved for 18.104.22.168, a=dveditz for release-drivers approved again, please get this checked in this time. The code freeze is a week away.
Flags: blocking-firefox3.1? → blocking-firefox3.1+
Checking in browser/components/sessionstore/src/nsSessionStore.js; new revision: 1.103; previous revision: 1.102
Verified for 22.214.171.124 with Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:126.96.36.199pre) Gecko/2008120105 GranParadiso/3.0.5pre. Focus is now properly set.
You need to log in before you can comment on or make changes to this bug.