--- a/ginebra2/chrome/bedrockchrome/bookmarkview.superpage/bookmarkview.js Fri Sep 17 12:11:40 2010 -0400
+++ b/ginebra2/chrome/bedrockchrome/bookmarkview.superpage/bookmarkview.js Thu Sep 23 15:32:11 2010 -0400
@@ -214,7 +214,7 @@
dbgTitle = dbgTitle.replace(/"/g, """);
var li = _createBookmarkElement(dbgTitle,bmurl,bmid);
$(li).addClass('no-sort');
- ul.appendChild(li);
+ ul.insertBefore(li, ul.firstChild);
ul.childNodes[ul.childNodes.length-1].focus();
if (!window.views.WebView.bedrockTiledBackingStoreEnabled())
@@ -240,7 +240,9 @@
function _launchEditBookmark(r,bmtitle,bmurl,id)
{
- window.bookmarksController.showBookmarkEditDialog(bmtitle,bmurl,id);
+ // bookmark title/url may have been altered by the controller's edit method, so reload it
+ var bm = window.bookmarksController.findBookmark(id);
+ window.bookmarksController.showBookmarkEditDialog(bm.title,bm.url,id);
}
function _deleteBookmark(r,bmid)
@@ -250,7 +252,7 @@
}
-function _openUrl(ele, newUrl) {
+function _openUrl(ele, bmid) {
if(__timerId != "")
{
window.clearTimeout(__timerId);
@@ -281,7 +283,9 @@
window.ViewStack.switchView( "WebView","BookmarkTreeView");
// Laod a page to chrome view
- window.views.WebView.loadUrlToCurrentPage(newUrl);
+ // bookmark title/url may have been altered by the controller's edit method, so reload it
+ var bm = window.bookmarksController.findBookmark(bmid);
+ window.views.WebView.loadUrlToCurrentPage(bm.url);
views.WebView.gesturesEnabled = true;
}
@@ -324,7 +328,7 @@
.mouseover(_setDogear)
.mouseout(_unsetDogear);
$(li).find(".bookmarkItem").
- click(function (event) {_openUrl(event.target, bmfullurl);});
+ click(function (event) {_openUrl(event.target, idValue);});
$(li).find(".bookmarkEditBtn").
click(function (event) {_launchEditBookmark(event.target, bmtitle, bmfullurl, idValue);});
$(li).find(".bookmarkDeleteBtn").