Magic-Mirror/gecko-dev
1
0
Fork 0
mirror of https://github.com/mozilla/gecko-dev.git synced 2025-02-11 10:08:41 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Bug 432366 - Paste of bookmark on "Bookmarks Menu" in sidebar causes "Recently Bookmarked" folder to expand. r=dietrich.

Browse Source
This commit is contained in:
Asaf Romano 2008-07-08 12:33:17 +03:00
parent 3b13e2d585
commit 191f5d8909
2 changed files with 15 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
browser/components/places/content/controller.js
View File

@ -763,7 +763,7 @@ PlacesController.prototype = {
// select the new item
var insertedNodeId = PlacesUtils.bookmarks
.getIdForItemAt(ip.itemId, ip.index);
this._view.selectItems([insertedNodeId], ip.itemId);
this._view.selectItems([insertedNodeId], false);
}
},
@ -783,7 +783,7 @@ PlacesController.prototype = {
// select the new item
var insertedNodeId = PlacesUtils.bookmarks
.getIdForItemAt(ip.itemId, ip.index);
this._view.selectItems([insertedNodeId]);
this._view.selectItems([insertedNodeId], false);
}
},
@ -799,7 +799,7 @@ PlacesController.prototype = {
// select the new item
var insertedNodeId = PlacesUtils.bookmarks
.getIdForItemAt(ip.itemId, ip.index);
this._view.selectItems([insertedNodeId]);
this._view.selectItems([insertedNodeId], false);
},
/**
@ -1234,7 +1234,7 @@ PlacesController.prototype = {
insertedNodeIds.push(PlacesUtils.bookmarks
.getIdForItemAt(ip.itemId, ip.index + i));
if (insertedNodeIds.length > 0)
this._view.selectItems(insertedNodeIds);
this._view.selectItems(insertedNodeIds, false);
}
};

12
browser/components/places/content/tree.xml
View File

@ -585,7 +585,14 @@
-->
<method name="selectItems">
<parameter name="aIDs"/>
<parameter name="aOpenContainers"/>
<body><![CDATA[
// By default, we do search and select within containers which were
// closed (note that containers in which nodes were not found are
// closed).
if (aOpenContainers === undefined)
aOpenContainers = true;
var ids = aIDs; // don't manipulate the caller's array
// Array of nodes found by findNodes which are to be selected
@ -625,8 +632,11 @@
nodesURIChecked.indexOf(node.uri) != -1)
return foundOne;
nodesURIChecked.push(node.uri);
asContainer(node);
if (!aOpenContainers && !node.containerOpen)
return foundOne;
nodesURIChecked.push(node.uri);
// Remember the beginning state so that we can re-close
// this node if we don't find any additional results here.