Closed
Bug 342432
Opened 19 years ago
Closed 18 years ago
After Undo Close Tab, content area of newly-opened page isn't focused
Categories
(Firefox :: Tabbed Browser, defect)
Tracking
()
RESOLVED
FIXED
Firefox 3 alpha1
People
(Reporter: ispiked, Assigned: zeniko)
References
Details
Attachments
(1 file)
(deleted),
patch
|
asaf
:
review+
|
Details | Diff | Splinter Review |
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9a1) Gecko/20060622 Minefield/3.0a1
Steps to reproduce:
1. Open a new tab and close it.
2. Right click on a tab and choose Undo Close Tab.
Results:
Tab clicked on remains focused with dotted outline showing.
Expected results:
Newly-opened tab's content area needs to be focused.
Comment 1•19 years ago
|
||
This is an addition to the main undo-close-tab feature. Shouldn't block bug 342179, which is unrelated AFAICT.
Status: NEW → ASSIGNED
Comment 2•19 years ago
|
||
Re: bug 343803 comment #4, the fix for this bug should also address which tab gets focus when re-opening multiple closed tabs via the history menu.
Comment 3•19 years ago
|
||
I'm guessing that Dietrich meant to take this when he marked it as assigned... :)
Assignee: nobody → dietrich
Status: ASSIGNED → NEW
Reporter | ||
Comment 5•18 years ago
|
||
Yep. I can reproduce on current trunk and branch.
This only happens if you use Undo Close Tab from the tab context menu; the page is focused correctly if you open the tab from the History > Recently Closed Tabs menu.
Keywords: qawanted
Assignee | ||
Comment 6•18 years ago
|
||
Right, this still happens if you right-click on the _current_ tab (which gets the focus with the click and won't let it go).
I'm not sure whether the correct solution should rather be to just move the focus rectangle to the reopened tab though. Or maybe - for accessibility reasons - we're not supposed to move the focus at all.
Aaron: what would the correct behavior be in this case?
Comment 7•18 years ago
|
||
The right thing to do would be to focus the new tab's contents.
Assignee | ||
Comment 8•18 years ago
|
||
Like this?
Attachment #236968 -
Flags: superreview?(aaronleventhal)
Attachment #236968 -
Flags: review?(dietrich)
Comment 9•18 years ago
|
||
Comment on attachment 236968 [details] [diff] [review]
always focus the reopened tab's content area
I can't do superreviews and anyway you don't need them for chrome. You need a toolkit/browser peer to review this (and I'm not one, I don't think dietrich is either). Try bugs.mano or someone listed on mozilla.org.
Attachment #236968 -
Flags: superreview?(aaronleventhal)
Assignee | ||
Comment 10•18 years ago
|
||
Comment on attachment 236968 [details] [diff] [review]
always focus the reopened tab's content area
(In reply to comment #9)
> I can't do superreviews and anyway you don't need them for chrome.
I know, I know. OTOH you're the accessibility guru and I'd like to know whether the solution chosen is acceptable from your point of view (and "superreview" is the closest to an "access-review" flag we have).
Attachment #236968 -
Flags: review?(dietrich) → review?(bugs.mano)
Comment 11•18 years ago
|
||
Comment on attachment 236968 [details] [diff] [review]
always focus the reopened tab's content area
r=mano
Attachment #236968 -
Flags: review?(bugs.mano) → review+
Assignee | ||
Updated•18 years ago
|
Assignee: dietrich → zeniko
Whiteboard: [checkin needed]
Comment 12•18 years ago
|
||
mozilla/browser/components/sessionstore/src/nsSessionStore.js 1.49
Status: NEW → RESOLVED
Closed: 18 years ago
OS: Linux → All
Hardware: PC → All
Resolution: --- → FIXED
Whiteboard: [checkin needed]
Target Milestone: --- → Firefox 3 alpha1
You need to log in
before you can comment on or make changes to this bug.
Description
•