Closed
Bug 305928
Opened 19 years ago
Closed 19 years ago
add bookmark dialog - expand button does not expand
Categories
(Firefox :: Bookmarks & History, defect)
Tracking
()
VERIFIED
FIXED
Firefox1.5
People
(Reporter: Peter6, Assigned: mconnor)
References
Details
(Keywords: regression, verified1.8)
Attachments
(1 file)
1.42 KB,
patch
|
vlad
:
review+
dbaron
:
approval1.8b4+
|
Details | Diff | Splinter Review |
Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.8b4) Gecko/20050825
Firefox/1.0+ ID:2005082506
1.Open FF and any page
2.Press Bookmark this page
3.Press the "expand" button
4.the folder does not expand but adds the [New Folder] button instead
Comment 1•19 years ago
|
||
only hapeens when "Bookmarks" is selcted.
JS Console says:
Error: uncaught exception: [Exception... "Component returned failure code:
0x80004003 (NS_ERROR_INVALID_POINTER) [nsIBookmarksService.getParentChain]"
nsresult: "0x80004003 (NS_ERROR_INVALID_POINTER)" location: "JS frame ::
chrome://browser/content/bookmarks/bookmarksTree.xml :: selectResource :: line
293" data: no]
Comment 2•19 years ago
|
||
seen here too on branch Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US;
rv:1.8b4) Gecko/20050825 Firefox/1.0+ (OS X 10.4.2)
Reporter | ||
Comment 3•19 years ago
|
||
Same JSC error when a live bookmark is added (via locationbar icon)
Comment 4•19 years ago
|
||
Clicking the "Add" button doesn't work either :( Mozilla/5.0 (X11; U; Linux
i686; en-US; rv:1.8b4) Gecko/20050825 Firefox/1.0+
Updated•19 years ago
|
Flags: blocking1.8b4? → blocking1.8b4+
Assignee | ||
Comment 5•19 years ago
|
||
stupid. This collided with ben's backout of feedview on branch because he
included a fix for 304705 in the patch, which I didn't pick up on in my quick
reading of the patch. Gah. This removes what he added as the simplest means
to resolve the conflict between our patches.
Attachment #193866 -
Flags: review?(vladimir)
Attachment #193866 -
Flags: approval1.8b4?
Attachment #193866 -
Flags: review?(vladimir) → review+
Attachment #193866 -
Flags: approval1.8b4? → approval1.8b4+
Assignee | ||
Updated•19 years ago
|
Comment 6•19 years ago
|
||
v.fixed in both places with today's builds.
Status: RESOLVED → VERIFIED
Keywords: fixed1.8 → verified1.8
You need to log in
before you can comment on or make changes to this bug.
Description
•