See Changelog

Monotone-Parent: 19bd38381f65942b0903a2e2aaa8abbc9c9862ab
Monotone-Revision: 6c9151f261ad9f73f6614808f73d4ca224c2cb8e

Monotone-Author: flachapelle@inverse.ca
Monotone-Date: 2010-11-08T18:47:11
Monotone-Branch: ca.inverse.sogo

ChangeLog

@@ -12,6 +12,13 @@
 	* UI/WebServerResources/SOGoDragHandles.js (adjust): for vertical
 	drag handles, prioritize the top limit rather than the bottom one.
 
+	* UI/WebServerResources/UIxContactEditor.js (onComboButtonClick):
+	return false so the form is not submitted when the button is clicked.
+
+	* UI/WebServerResources/ContactsUI.js
+	(onCategoriesMenuPrepareVisibility): don't verify the selection if
+	the list doesn't exist (happens in the contact editor).
+
 2010-11-05  Francis Lachapelle  <flachapelle@inverse.ca>
 
 	* UI/MailerUI/UIxMailListActions.m (-getSortedUIDsAction): now

UI/Templates/ContactsUI/UIxContactEditor.wox

@@ -8,6 +8,7 @@
   xmlns:label="OGo:label"
   className="UIxPageFrame"
   title="name"
+  const:userDefaultsKeys="SOGoContactsCategories"
   const:popup="YES"
   >
   <div class="menu" id="categoriesMenu">

UI/WebServerResources/ContactsUI.js

@@ -1302,30 +1302,34 @@ function resetCategoriesMenu() {
 }
 
 function onCategoriesMenuPrepareVisibility() {
-    var rows = contactsList.getSelectedRows();
-    if (rows.length > 0) {
-        var catList = rows[0].getAttribute("categories");
-        var catsArray;
-        if (catList && catList.length > 0) {
-            catsArray = catList.split(",");
-        }
-        else {
-            catsArray = [];
-        }
-
-        var menu = $("categoriesMenu");
-        var ul = menu.down("ul");
-        var listElements = ul.select("li");
-        for (var i = 0; i < listElements.length; i++) {
-            var li = listElements[i];
-            if (catsArray.indexOf(li.category) > -1) {
-                li.addClassName("_chosen");
+    var contactsList = $("contactsList");
+    if (contactsList) {
+        var rows = contactsList.getSelectedRows();
+        if (rows.length > 0) {
+            var catList = rows[0].getAttribute("categories"); log ("cats = " + catList);
+            var catsArray;
+            if (catList && catList.length > 0) {
+                catsArray = catList.split(",");
             }
             else {
-                li.removeClassName("_chosen");
+                catsArray = [];
+            }
+
+            var menu = $("categoriesMenu");
+            var ul = menu.down("ul");
+            var listElements = ul.select("li");
+            for (var i = 0; i < listElements.length; i++) {
+                var li = listElements[i];
+                if (catsArray.indexOf(li.category) > -1) {
+                    li.addClassName("_chosen");
+                }
+                else {
+                    li.removeClassName("_chosen");
+                }
             }
         }
     }
+    return true;
 }
 
 function onCategoriesMenuItemClick() {

UI/WebServerResources/UIxContactEditor.js

@@ -213,7 +213,7 @@ function onComboButtonClick(event) {
                     "left": menuLeft + "px",
                     "width": width + "px" });
 
-    event.preventDefault();
+    return false;
 }
 
 function onCategoryInputChange(event) {