Closed Bug 241625 Opened 21 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: