Closed Bug 241625 Opened 20 years ago Closed 20 years ago

"Open in tabs" doesn't focus content area in first tab

Categories

(Firefox :: Bookmarks & History, defect)

x86
Windows XP
defect
Not set
normal

Tracking

()

VERIFIED FIXED

People

(Reporter: jruderman, Assigned: vlad)

Details

(Keywords: fixed-aviary1.0)

Attachments

(1 file)

Steps to reproduce:
1. Middle-click on a bookmark folder on the bookmarks toolbar.
2. Wait for the first page (at least) to load.
3. Press Down.

Expected: scroll the first page down
Result: nothing happens.

If I press Tab after step 2, the content area gains focus, and pressing Down works.
Attached patch patchSplinter Review
This changes openGroupBookmark to use w._content.focus() instead of
browser.focus().  browser.focus() seems to work correctly in the second case in
openOneBookmark but not in the first case in openOneBookmark or in
openGroupBookmark.
Assignee: p_ch → jruderman
Status: NEW → ASSIGNED
Attachment #149954 - Flags: review?(bugs)
Bryner says this should be debugged so we know why the old code didn't work.
Assignee: jruderman → bryner
Status: ASSIGNED → NEW
Flags: blocking-aviary1.0?
Flags: blocking-aviary1.0? → blocking-aviary1.0+
Attachment #149954 - Flags: review?(bugs)
Assignee: bryner → vladimir
bryner, any ideas on what to do about this?  I'm inclined to take this patch
as-is for 1.0..
Comment on attachment 149954 [details] [diff] [review]
patch

bandaid, will do for now.
Attachment #149954 - Flags: review+
Attachment #149954 - Flags: approval-aviary?
in on aviary.
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Keywords: fixed-aviary1.0
may have caused regression bug Bug 263060
(In reply to comment #7)
> may have caused regression bug Bug 263060

nope, just tested both with and without this patch.
verified fixed as seen on Windows 2004-10-07-06-0.9,  Mac 2004-10-07-05-0.9 and
linux 2004-10-07-09-0.9
Status: RESOLVED → VERIFIED
sorry for bugspam, long-overdue mass reassign of ancient QA contact bugs, filter on "beltznerLovesGoats" to get rid of this mass change
QA Contact: mconnor → bookmarks
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: