From: Bill Erickson <berick@esilibrary.com>
Date: Fri, 24 Feb 2012 20:43:47 +0000 (-0500)
Subject: Copy Location Search Groups : admin UI improvements
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=01e0058a678bc8642d12ded314b95dc66d85a3a8;p=evergreen%2Fpines.git

Copy Location Search Groups : admin UI improvements

Two improvements to the location group list.

1. Replace x / checkmark glyphs with clearer Visible / Not Visible
labels.  Thanks for the suggestions, Lebbeous.

2. Vertically align the edit actions block and the drag grips so the
actions are always in the same place.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Dan Scott <dan@coffeecode.net>
---

diff --git a/Open-ILS/src/templates/conify/global/asset/copy_location_group.tt2 b/Open-ILS/src/templates/conify/global/asset/copy_location_group.tt2
index c62ec4ada7..5466e11b4d 100644
--- a/Open-ILS/src/templates/conify/global/asset/copy_location_group.tt2
+++ b/Open-ILS/src/templates/conify/global/asset/copy_location_group.tt2
@@ -112,22 +112,26 @@
 <div class='hidden'>
     <div dojoType='openils.widget.ProgressDialog' jsId='progressDialog'></div>
     <div id='dnd-drag-actions'>
-        <span>
-            <span style='padding:5px;' group="GRPID">
-                <span><a href="javascript:;" onclick="drawGroupEntries(GRPID)">GRPNAME</a></span>
-                [
-                <span>[% l('Visible') %]
-                    <span style='color:green' name='visible'>&#x2713;</span>
-                    <span style='color:red' name='invisible'>&#x2717;</span>
+        <table width='100%'><tr>
+            <td nowrap='nowrap' align='left'>
+                <span style='padding:5px;' group="GRPID">
+                    <span><a href="javascript:;" onclick="drawGroupEntries(GRPID)">GRPNAME</a></span>
                 </span>
+            </td>
+            <td align='right'>
+                [
+                <span style='color:green' name='visible'>[% l('Visible') %]</span>
+                <span style='color:red' name='invisible'>[% l('Not Visible') %]</span>
                 <span><a href="javascript:;" onclick="editGroup(GRPID)">[% l('Edit') %]</a></span>
                 <span><a href="javascript:;" onclick="deleteGroup(GRPID)">[% l('Delete') %]</a></span>
                 ]
-            </span>
-            <span></span>
-            <span class='acplg-drag-handle'></span>
-            <span class='acplg-drag-handle'></span>
-            <span class='acplg-drag-handle'></span>
+            </td>
+            <td align='right' width='32px'>
+                <span class='acplg-drag-handle'></span>
+                <span class='acplg-drag-handle'></span>
+                <span class='acplg-drag-handle'></span>
+            </td>
+        </tr></table>
         </span>
     </div>
     <div id='acplg-edit-dialog'></div>