Closed Bug 224251 Opened 22 years ago Closed 22 years ago

Bookmarks panel doesn't focus bookmarks tree when selected

Categories

(SeaMonkey :: Bookmarks & History, defect)

Product:
SeaMonkey
Component:
Bookmarks & History
Type:
defect
Priority:
Not set
Severity:
minor

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: neil, Assigned: p_ch)

Details

(Keywords: regression)

Attachments

(1 file)

Proposed patch
757 bytes, patch
p_ch
: review+
alecf
: superreview+
Details | Diff | Splinter Review
Steps to reproduce problem: 1. If the sidebar is closed, open it 2. Select the history panel 3. Select the bookmarks panel Expected result: Both panels focus their respective tree Actual result: Bookmarks panel does not focus its tree Additional information: Caused by the BookmarksTree.xml landing
Attached patch Proposed patch — — Splinter Review
The sidebar loader doesn't know how to focus anonymous nodes, so get the bookmarks-tree to move the focus to the anonymous tree.
Comment on attachment 134537 [details] [diff] [review] Proposed patch This was the one which I meant to ask your review, perhaps the other request might have been by mistake.
Attachment #134537 - Flags: review?(p_ch)
Comment on attachment 134537 [details] [diff] [review] Proposed patch np, but this patch depends anyways on the one in bug 50223
Attachment #134537 - Flags: review?(p_ch) → review+
Comment on attachment 134537 [details] [diff] [review] Proposed patch pch, I don't think that this is dependent, because bug 50223 already works for the history sidebar.
Attachment #134537 - Flags: superreview?(alecf)
Comment on attachment 134537 [details] [diff] [review] Proposed patch sr=alecf
Attachment #134537 - Flags: superreview?(alecf) → superreview+
Fix checked in.
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: