229513 - Enable the user to reveal bookmarks found in inline search

Status: RESOLVED FIXED

(Camino Graveyard :: Bookmarks, enhancement)

Description

[Jasper]

User-Agent:       Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.7a) Gecko/20031224 Camino/0.7+
Build Identifier: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.7a) Gecko/20031224 Camino/0.7+

Currently users are not able to determin where bookmarsk that are shown to be
broken are located within their bookmarks collection.

Reproducible: Always

Steps to Reproduce:
1. Go to the broken bookmarks folder, try to use the reveal button.


Actual Results:  
The user isn'ta able to do so.


Perhaps it would be nice to add a "Reveal" contectual menu item for the search
results the top 10 folder anmd the broken bookmarks folder.

Comment 1

[Mike Pinkerton (not reading bugmail)]

seems reasonable

Comment 2

[Josh Aas]

Doesn't need to block 0.9. This isn't very easy to fix since the target
selection could be in either the search table or the bm item outline view (both
can have a selection at the same time).

Comment 3

[Simon Fraser [no longer active]]

Should be a bit easier to do now, but the reveal code needs work, and should
select the item.

Comment 4

[Josh Aas]

updated summary since we don't have any reveal functionality any more

Comment 5

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

See also bug 258977 for revealing bookmarks found in BM search.

Comment 6

[Stuart Morgan]

*** Bug 328545 has been marked as a duplicate of this bug. ***

Comment 7

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

This bug is now (and has been for a while) about revealing the original location (in the hierarchy) of a bookmark/history item found in the inline search.

Comment 8

[Chris Lawson (gone)]

I'll look into this. Marking dependent on the code from bug 288980 for now.

Comment 9

[Chris Lawson (gone)]

Patch for this will be in bug 288980, as the two are very closely related.

Comment 10

[froodian (Ian Leue)]

Fixed by bug 288980