From 6c0878b985f9cac68e0a3eb16bca29ff1c6be0c2 Mon Sep 17 00:00:00 2001
From: "mvl%exedo.nl" <mvl%exedo.nl@18797224-902f-48f8-a5cc-f745e15eee43>
Date: Sat, 10 Jul 2004 12:17:06 +0000
Subject: [PATCH] Improve UI for managing categories in prefs bug 249294,
 r=mostafah

git-svn-id: svn://10.0.0.236/trunk@158991 18797224-902f-48f8-a5cc-f745e15eee43
---
 .../resources/content/pref/calendarPref.xul   | 94 +++++++++++++++++--
 .../calendar/resources/locale/en-US/prefs.dtd |  5 +
 2 files changed, 91 insertions(+), 8 deletions(-)

diff --git a/mozilla/calendar/resources/content/pref/calendarPref.xul b/mozilla/calendar/resources/content/pref/calendarPref.xul
index 1e482eda466..5720e131270 100644
--- a/mozilla/calendar/resources/content/pref/calendarPref.xul
+++ b/mozilla/calendar/resources/content/pref/calendarPref.xul
@@ -55,7 +55,13 @@
    <script type="application/x-javascript">
       var labelLong = "&pref.dateformat.long;";      
       var labelShort = "&pref.dateformat.short;";
+      var addTitle = "&pref.categories.add.title;";
+      var editTitle = "&pref.categories.edit.title;";
+      var addPrompt = "&pref.categories.add.prompt;";
+   </script>
 
+   <script type="application/x-javascript">
+    <![CDATA[
       function calendarPrefOnLoad()
       {
          var now = new Date();
@@ -74,17 +80,83 @@
          var selectedIndex = dateFormatMenuList.selectedIndex;
          dateFormatMenuList.selectedIndex = -1;
          dateFormatMenuList.selectedIndex = selectedIndex;
-       }
-      
+      }
+
+      // An array to store the categoris in, because taking them from
+      // the listbox is unreliable. It might update too late.
+      var gCategoryList;
+
+      // not in onload, because that will be called too early
+      // if this panel is the default to open. and it is.
+      function Startup()
+      {
+        var categories = document.getElementById("categories").value;
+        gCategoryList = categories.split(",");
+        updateCategoryList();
+      }
+
+      function updateCategoryList()
+      {
+        document.getElementById("categories").value = gCategoryList.join(",");
+        var listbox = document.getElementById("catergorieslist")
+
+        listbox.clearSelection();
+        while (listbox.firstChild)
+            listbox.removeChild(listbox.firstChild);
+
+        for (var i = 0; i < gCategoryList.length; i++) {
+          listbox.appendItem(gCategoryList[i], gCategoryList[i]);
+        }
+      }
+
+      function addCategory()
+      {
+        var promptService = Components.classes["@mozilla.org/embedcomp/prompt-service;1"]
+                                      .getService(Components.interfaces.nsIPromptService);
+        var result = {value:null};
+        var dummy = { value: 0 };
+        if (promptService.prompt(null, addTitle, addPrompt,
+                                 result, null, dummy) &&
+            result.value.length) {
+          gCategoryList.push(result.value);
+          updateCategoryList();
+        }
+      }
+
+      function editCategory()
+      {
+        var list = document.getElementById("catergorieslist");
+        if (list.selectedIndex) {
+          var promptService = Components.classes["@mozilla.org/embedcomp/prompt-service;1"]
+                                        .getService(Components.interfaces.nsIPromptService);
+          var result = {value: gCategoryList[list.selectedIndex]};
+          var dummy = { value: 0 };
+
+          if (promptService.prompt(null, editTitle, addPrompt,
+                                   result, null, dummy) &&
+              result.value.length) {
+            gCategoryList.splice(list.selectedIndex, 1, result.value);
+          }
+        }
+        updateCategoryList();
+      }
+
+      function deleteCategory()
+      {
+        var list = document.getElementById("catergorieslist");
+        gCategoryList.splice(list.selectedIndex, 1);
+        updateCategoryList();
+      }
+
       /* 
       ** NOTE: 
       ** When you add a pref to this page, you have to add its ID to the following list.
       */
-       
       var _elementIDs = [
       "dateformat", "defaultlength", "defaultsnoozelength",
       "reloadonlaunch", "categories", "storeingmt"
        ];
+     ]]>
    </script>
 
    <groupbox align="start" orient="vertical">
@@ -114,10 +186,16 @@
       <checkbox id="reloadonlaunch" preftype="bool" prefstring="calendar.servers.reloadonlaunch" label="&pref.reloadonlaunch;" />
    </groupbox>
    <groupbox>
-      <caption label="&pref.categories.label;"/>
-         <vbox>
-		 <textbox id="categories" prefstring="calendar.categories.names"/>
-		 <description>&pref.categories.help.description;</description>
-      </vbox>
+     <caption label="&pref.categories.label;"/>
+     <hbox>
+       <listbox flex="1" id="catergorieslist"/>
+       <vbox>
+         <spacer flex="1"/>
+         <button label="Add" oncommand="addCategory()"/>
+         <button label="Edit" oncommand="editCategory()"/>
+         <button label="Remove" oncommand="deleteCategory()"/>
+       </vbox>
+     </hbox>
+     <textbox id="categories" prefstring="calendar.categories.names" hidden="true"/>
    </groupbox>
 </page>
diff --git a/mozilla/calendar/resources/locale/en-US/prefs.dtd b/mozilla/calendar/resources/locale/en-US/prefs.dtd
index c602edcfbb1..11b21c5dbdc 100644
--- a/mozilla/calendar/resources/locale/en-US/prefs.dtd
+++ b/mozilla/calendar/resources/locale/en-US/prefs.dtd
@@ -126,6 +126,11 @@
 
 <!ENTITY pref.categories.help.description   "Use a comma between each category you want to add.">
 
+<!ENTITY pref.categories.add.title    "Add Category">
+<!ENTITY pref.categories.add.prompt   "Enter the name of the category">
+<!ENTITY pref.categories.edit.title   "Edit Category">
+
+
 <!ENTITY pref.reloadonlaunch       "Reload Remote Calendars On Startup">
 
 <!ENTITY pref.publish.caption        "Publishing Settings">