Closed Bug 224251 Opened 21 years ago Closed 21 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: 21 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: