From df233d56cd2e0259af1b4690429c4737b6d83ee1 Mon Sep 17 00:00:00 2001
From: Ben Klang <ben@alkaloid.net>
Date: Tue, 23 Mar 2010 21:28:32 -0400
Subject: [PATCH] Shout: Revise and simplify destinations UI actions

---
 shout/templates/extensions/list.inc | 143 ++++++++++++++++++++----------------
 shout/themes/screen.css             |   2 +-
 2 files changed, 82 insertions(+), 63 deletions(-)

diff --git a/shout/templates/extensions/list.inc b/shout/templates/extensions/list.inc
index 25f262598..9cd09632b 100644
--- a/shout/templates/extensions/list.inc
+++ b/shout/templates/extensions/list.inc
@@ -36,13 +36,11 @@
             </td>
             <td>
                 <?php
-                $attrs = array('onClick' => 'destinfo("' . $extension . '");',
+                $attrs = array('onClick' => 'showDetail("' . $extension . '");',
                                'id' => 'destX' . $extension . 'toggle');
                 echo Horde::img('tree/plusonly.png', _("Destinations"), $attrs);
                 ?>
-                <span id="destX<?php echo $extension; ?>summary"></span>
-                <div class="extensionDestinations" id="destX<?php echo $extension; ?>info">
-                </div>
+                <span id="destX<?php echo $extension; ?>"></span>
             </td>
         </tr>
         <?php
@@ -56,19 +54,30 @@
 
 var destinations = [];
 var ajax_url = '<?php echo Horde::getServiceLink('ajax', 'shout') ?>';
+var curexten = null;
 
-function resetExtenDest(exten)
+function empty(p)
 {
     var e;
-
-    while ((e = $('destX' + exten + 'info').childNodes[0]) != null) {
-        $('destX' + exten + 'info').removeChild(e);
+    while ((e = $(p).childNodes[0]) != null) {
+        $(p).removeChild(e);
     }
+}
 
-    while ((e = $('destX' + exten + 'summary').childNodes[0]) != null) {
-        $('destX' + exten + 'summary').removeChild(e);
+function showDetail(exten)
+{
+    // Hide any currently expanded extension info
+    if (curexten) {
+        showSummary(curexten);
     }
 
+    $('destX' + exten + 'toggle').src = '<?php echo Horde_Themes::img('tree/minusonly.png') ?>';
+    $('destX' + exten + 'toggle').setAttribute('onclick', 'showSummary('+exten+')');
+    curexten = exten;
+
+    var e = $('destX'+exten);
+    empty(e);
+
     var dest = destinations.get(exten);
 
     if (dest.devices == null) {
@@ -82,11 +91,19 @@ function resetExtenDest(exten)
     if (dest.devices.size() == 0 && dest.numbers.size() == 0) {
         var span = document.createElement('span');
         span.className = 'informational';
-        var text = document.createTextNode("No destinations configured");
+        var text = document.createTextNode('<?php echo _("No destinations configured"); ?>');
+        span.appendChild(text);
+        e.appendChild(span);
+    } else {
+        var span = document.createElement('span');
+        span.className = 'informational';
+        var text = document.createTextNode('<?php echo _("Configured Destinations:"); ?>');
         span.appendChild(text);
-        $('destX' + exten + 'summary').appendChild(span);
+        e.appendChild(span);
     }
 
+    var div = document.createElement('div');
+    div.className = 'extensionDestinations';
     dest.devices.each(function (s) {
         // Fill in detail block
         var img = document.createElement('img');
@@ -106,16 +123,10 @@ function resetExtenDest(exten)
 
         var br = document.createElement('br');
 
-        var info = $('destX' + exten + 'info');
-        info.appendChild(img);
-        info.appendChild(span);
-        info.appendChild(del);
-        info.appendChild(br);
-
-        // Create summary icons
-        img = document.createElement('img');
-        img.src = "<?php echo Horde_Themes::img('shout.png') ?>";
-        $('destX' + exten + 'summary').appendChild(img);
+        div.appendChild(img);
+        div.appendChild(span);
+        div.appendChild(del);
+        div.appendChild(br);
     });
 
 
@@ -138,18 +149,14 @@ function resetExtenDest(exten)
 
         var br = document.createElement('br');
 
-        var info = $('destX' + exten + 'info');
-        info.appendChild(img);
-        info.appendChild(span);
-        info.appendChild(del);
-        info.appendChild(br);
-
-        // Create summary icons
-        img = document.createElement('img');
-        img.src = "<?php echo Horde_Themes::img('telephone-pole.png') ?>";
-        $('destX' + exten + 'summary').appendChild(img);
+        div.appendChild(img);
+        div.appendChild(span);
+        div.appendChild(del);
+        div.appendChild(br);
     });
 
+
+
     var form = document.createElement('form');
     form.method = 'post';
     form.action = ajax_url + 'addDestination';
@@ -159,7 +166,7 @@ function resetExtenDest(exten)
     hidden.name = 'extension';
     hidden.value = exten;
     form.appendChild(hidden);
-    $('destX' + exten + 'info').appendChild(form);
+    div.appendChild(form);
 
     var a = document.createElement('a');
     a.id = 'destX' + exten + 'addDest';
@@ -168,44 +175,56 @@ function resetExtenDest(exten)
     a.setAttribute('onClick', 'addDest(' + exten + ')');
     var t = document.createTextNode('Add destination...');
     a.appendChild(t);
-    $('destX' + exten + 'info').appendChild(a);
-}
+    div.appendChild(a);
 
-function resetDestInfo(collapse)
-{
-    destinations.each(function (item) {
-        var exten;
-        exten = item.key;
-        resetExtenDest(exten);
-        if (collapse) {
-            contract(exten);
-        }
-    });
+    e.appendChild(div);
 }
 
-function destinfo(exten)
+function showSummary(exten)
 {
-    // Use the summary icons span as our state key
-    if ($('destX' + exten + 'summary').style.display == 'none') {
-        // Icons hidden, we are expanded
-        contract(exten);
-    } else {
-        expand(exten);
+    var e = $('destX' + exten);
+    empty(e);
+
+    $('destX' + exten + 'toggle').src = '<?php echo Horde_Themes::img('tree/plusonly.png') ?>';
+    $('destX' + exten + 'toggle').setAttribute('onclick', 'showDetail('+exten+')');
+
+    var dest = destinations.get(exten);
+
+    if (dest.devices == null) {
+        dest.devices = [];
     }
-}
 
-function expand(exten)
-{
-    $('destX' + exten + 'summary').hide();
-    $('destX' + exten + 'info').show();
-    $('destX' + exten + 'toggle').src = '<?php echo Horde_Themes::img('tree/minusonly.png') ?>';
+    if (dest.numbers == null) {
+        dest.numbers = [];
+    }
+
+    if (dest.devices.size() == 0 && dest.numbers.size() == 0) {
+        var span = document.createElement('span');
+        span.className = 'informational';
+        var text = document.createTextNode("No destinations configured");
+        span.appendChild(text);
+        e.appendChild(span);
+    } else {
+        dest.devices.each(function (s) {
+            // Fill in detail block
+            var img = document.createElement('img');
+            img.src = "<?php echo Horde_Themes::img('shout.png') ?>";
+            e.appendChild(img);
+        })
+        dest.numbers.each(function (s) {
+            // Fill in detail block
+            var img = document.createElement('img');
+            img.src = "<?php echo Horde_Themes::img('telephone-pole.png') ?>";
+            e.appendChild(img);
+        })
+    }
 }
 
-function contract(exten)
+function resetDestInfo(collapse)
 {
-    $('destX' + exten + 'summary').show();
-    $('destX' + exten + 'info').hide();
-    $('destX' + exten + 'toggle').src = '<?php echo Horde_Themes::img('tree/plusonly.png') ?>';
+    destinations.each(function (item) {
+        showSummary(item.key)
+    })
 }
 
 function processForm(event)
diff --git a/shout/themes/screen.css b/shout/themes/screen.css
index ee12e3e1b..5812b6299 100644
--- a/shout/themes/screen.css
+++ b/shout/themes/screen.css
@@ -41,7 +41,7 @@ table {
 {
     /*display: none;*/
     position: relative;
-    left: 20px;
+    left: 25px;
 }
 
 a.addDest
-- 
2.11.0