More updates to addressbucket tree in selectaddress dialog

mailnews/addrbook/resources/content/addressbucket.xul

@@ -18,7 +18,7 @@
 		<treecol style="width:100%"/>
 
 		<treechildren id="bucketBody">
-			<treeitem id="bucketItem">
+			<treeitem>
 				<treerow>
 					<treecell/>
 				</treerow>

mailnews/addrbook/resources/content/selectaddress.js

@@ -69,38 +69,35 @@ function AddSelectedAddressesIntoBucket(prefix)
 
 function AddAddressIntoBucket(doc, address)
 {
-	var tree = doc.getElementById('addressBucket');
-
-	var item = doc.getElementById("bucketItem");
+	var body = doc.getElementById("bucketBody");
 	
-	//newitem.setAttribute("rowID", num);
-	//newitem.setAttribute("rowName", name);
-
-	var cell = doc.createElement('treecell');
+	var item = doc.createElement('treeitem');
 	var row = doc.createElement('treerow');
+	var cell = doc.createElement('treecell');
 	var text = doc.createTextNode(address);
 	
 	cell.appendChild(text);
 	row.appendChild(cell);
 	item.appendChild(row);
+	body.appendChild(item);
 }
 
 function RemoveSelectedFromBucket()
 {
 	var bucketDoc = frames["browser.addressbucket"].document;
 	dump("bucketDoc = " + bucketDoc + "\n");
-	var item = bucketDoc.getElementById("bucketItem");
-	dump("item = " + item + "\n");
+	var body = bucketDoc.getElementById("bucketBody");
+	dump("body = " + body + "\n");
 	
-	var selArray = item.getElementsByAttribute('selected', 'true');
+	var selArray = body.getElementsByAttribute('selected', 'true');
 	dump("selArray = " + selArray + "\n");
 	if ( selArray && selArray.length )
 	{
-		for ( var row = selArray.length - 1; row >= 0; row-- )
+		for ( var item = selArray.length - 1; item >= 0; item-- )
 		{
-			dump("selArray[row] = " + selArray[row] + "\n");
-			DumpDOM(selArray[row]);
-			item.removeChild(selArray[row]);
+			dump("selArray[item] = " + selArray[item] + "\n");
+			DumpDOM(selArray[item]);
+			body.removeChild(selArray[item]);
 		}
 	}	
 }

mailnews/addrbook/resources/content/selectaddress.xul

@@ -36,7 +36,7 @@ Rights Reserved.
 	<!ENTITY newButton.label	".New..."> 
 	<!ENTITY editButton.label	".Edit..."> 
 	<!ENTITY viewButton.label	".View"> 
-	<!ENTITY removeButton.label	".Remove"> 
+	<!ENTITY removeButton.label	"Remove"> 
 	<!ENTITY okButton.label		".OK"> 
 	<!ENTITY cancelButton.label	".Cancel"> 
 	<!ENTITY helpButton.label	".Help">