Shout: Make buttons more visually obvious

diff --git a/shout/lib/Shout.php b/shout/lib/Shout.php
index 176d5bf..fef60c6 100644 (file)
--- a/shout/lib/Shout.php
+++ b/shout/lib/Shout.php
@@ -145,7 +145,7 @@ class Shout
 
         return array(
             'jump' => array(
-                'description' => _("Jump to menu."),
+                'description' => _("Jump to menu"),
                 'args' => array (
                     'menuName' => array(
                         'name' => _("Menu"),
@@ -169,7 +169,7 @@ class Shout
             'leave_message' => array(
                 'description' => _("Go to voicemail"),
                 'args' => array(
-                    'mailbox' => array(
+                    'exten' => array(
                         'name' => _("Mailbox"),
                         'type' => 'enum',
                         'required' => true,

diff --git a/shout/templates/accounts/list.inc b/shout/templates/accounts/list.inc
index 2278c70..f768bab 100644 (file)
--- a/shout/templates/accounts/list.inc
+++ b/shout/templates/accounts/list.inc
@@ -1,14 +1,4 @@
 <div class="header">
-    <ul id="controls">
-        <?php
-        $addurl = Horde::applicationUrl('admin.php');
-        $addurl = Horde_Util::addParameter($addurl, 'action', 'add');
-        ?>
-        <li><a href="<?php echo $addurl; ?>">
-            <?php echo Horde::img('add-extension.png'); ?>&nbsp;New Account
-            </a>
-        </li>
-    </ul>
     Account: <?php echo $_SESSION['shout']['accounts'][$curaccount]; ?>
 </div>
 
@@ -43,3 +33,13 @@
         ?>
     </table>
 </div>
+<ul id="controls">
+    <?php
+    $addurl = Horde::applicationUrl('admin.php');
+    $addurl = Horde_Util::addParameter($addurl, 'action', 'add');
+    ?>
+    <li class="button"><a href="<?php echo $addurl; ?>">
+        <?php echo Horde::img('add-extension.png'); ?>&nbsp;New Account
+        </a>
+    </li>
+</ul>
\ No newline at end of file

diff --git a/shout/templates/devices/list.inc b/shout/templates/devices/list.inc
index 3a8cf84..d500f13 100644 (file)
--- a/shout/templates/devices/list.inc
+++ b/shout/templates/devices/list.inc
@@ -1,14 +1,4 @@
 <div class="header">
-    <ul id="controls">
-        <?php
-        $addurl = Horde::applicationUrl('devices.php');
-        $addurl = Horde_Util::addParameter($addurl, 'action', 'add');
-        ?>
-        <li><a href="<?php echo $addurl; ?>">
-            <?php echo Horde::img('add-device.png'); ?>&nbsp;New Device
-            </a>
-        </li>
-    </ul>
     Account: <?php echo $_SESSION['shout']['accounts'][$curaccount]; ?>
 </div>
 
@@ -48,3 +38,13 @@
         ?>
     </table>
 </div>
+<ul id="controls">
+    <?php
+    $addurl = Horde::applicationUrl('devices.php');
+    $addurl = Horde_Util::addParameter($addurl, 'action', 'add');
+    ?>
+    <li class="button"><a href="<?php echo $addurl; ?>">
+        <?php echo Horde::img('add-device.png'); ?>&nbsp;New Device
+        </a>
+    </li>
+</ul>

diff --git a/shout/templates/dialplan/edit.inc b/shout/templates/dialplan/edit.inc
index 54c6fec..fb491fd 100644 (file)
--- a/shout/templates/dialplan/edit.inc
+++ b/shout/templates/dialplan/edit.inc
@@ -275,7 +275,7 @@ function refreshMenu()
         switch(action) {
         case 'jump':
             var menu = pair.value.args['menuName'];
-            text = document.createTextNode(destinations.get(exten).name);
+            text = document.createTextNode(menu);
             break;
         case 'ringexten':
         case 'leave_message':

diff --git a/shout/templates/dialplan/list.inc b/shout/templates/dialplan/list.inc
index 431215a..9ebb3c7 100644 (file)
--- a/shout/templates/dialplan/list.inc
+++ b/shout/templates/dialplan/list.inc
@@ -1,19 +1,10 @@
 <div class="header">
-    <ul id="controls">
-        <?php
-        $addurl = Horde::applicationUrl('dialplan.php');
-        $addurl = Horde_Util::addParameter($addurl, 'action', 'add');
-        $editurl = Horde::applicationUrl('dialplan.php');
-        $editurl = Horde_Util::addParameter($editlink, 'action', 'edit');
-        ?>
-        <li><a href="<?php echo $addurl; ?>">
-            <?php echo Horde::img('add-menu.png'); ?>&nbsp;New Menu
-            </a>
-        </li>
-    </ul>
     Account: <?php echo $_SESSION['shout']['accounts'][$curaccount]; ?>
 </div>
-
+<?php
+$editurl = Horde::applicationUrl('dialplan.php');
+$editurl = Horde_Util::addParameter($editlink, 'action', 'edit');
+?>
 <div id="extensionList">
     <table width="100%" cellspacing="0" class="striped">
         <tr>
@@ -29,4 +20,15 @@
         </tr>
         <?php } ?>
     </table>
-</div>
\ No newline at end of file
+</div>
+<ul id="controls">
+    <?php
+    $addurl = Horde::applicationUrl('dialplan.php');
+    $addurl = Horde_Util::addParameter($addurl, 'action', 'add');
+
+    ?>
+    <li class="button"><a href="<?php echo $addurl; ?>">
+        <?php echo Horde::img('add-menu.png'); ?>&nbsp;New Menu
+        </a>
+    </li>
+</ul>
\ No newline at end of file

diff --git a/shout/templates/extensions/list.inc b/shout/templates/extensions/list.inc
index 7fbcf47..f3d63d0 100644 (file)
--- a/shout/templates/extensions/list.inc
+++ b/shout/templates/extensions/list.inc
@@ -1,14 +1,4 @@
 <div class="header">
-    <ul id="controls">
-        <?php
-        $addurl = Horde::applicationUrl('extensions.php');
-        $addurl = Horde_Util::addParameter($addurl, 'action', 'add');
-        ?>
-        <li><a href="<?php echo $addurl; ?>">
-            <?php echo Horde::img('add-extension.png'); ?>&nbsp;New Extension
-            </a>
-        </li>
-    </ul>
     Account: <?php echo $_SESSION['shout']['accounts'][$curaccount]; ?>
 </div>
 
@@ -48,6 +38,16 @@
         ?>
     </table>
 </div>
+<ul id="controls">
+    <?php
+    $addurl = Horde::applicationUrl('extensions.php');
+    $addurl = Horde_Util::addParameter($addurl, 'action', 'add');
+    ?>
+    <li class="button"><a href="<?php echo $addurl; ?>">
+        <?php echo Horde::img('add-extension.png'); ?>&nbsp;New Extension
+        </a>
+    </li>
+</ul>
 
 <script type="text/javascript">
 <!--

diff --git a/shout/themes/screen.css b/shout/themes/screen.css
index 8f21c4e..e3fb6ff 100644 (file)
--- a/shout/themes/screen.css
+++ b/shout/themes/screen.css
@@ -175,6 +175,10 @@ ul {
     margin: 5px;
 }
 
+.button a {
+    color: #fff;
+}
+
 .buttonActionLabel
 {
     text-align: right;