From 8caaffc6938c9c29a38e7fac394042b0062e3f0b Mon Sep 17 00:00:00 2001
From: Aleisha <aleishaamohia@hotmail.com>
Date: Thu, 14 Apr 2016 22:02:47 +0000
Subject: [PATCH] Bug 16263: making authority tags and subfields actions
 buttons

EDIT: Subfields icon, 'i' to eye

To test:
1) Go to Admin -> Authority types
2) Click the dropdown for one of the frameworks and click MARC structure
3) Confirm the actions subfields, edit and delete all show in the dropdown and work as expected
4) Click subfields
5) Confirm the action delete on this page shows as a font awesome button and works as expected

Sponsored-by: Catalyst IT

Signed-off-by: Arslan Farooq <arslanone@gmail.com>

Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>
Another sign, pretty eye.
No errors.

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
---
 .../modules/admin/auth_subfields_structure.tt |  4 ++--
 .../en/modules/admin/auth_tag_structure.tt    | 22 ++++++++++++-------
 2 files changed, 16 insertions(+), 10 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/auth_subfields_structure.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/auth_subfields_structure.tt
index 7caceca0b6..b62de1ddda 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/auth_subfields_structure.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/auth_subfields_structure.tt
@@ -274,7 +274,7 @@
         <th>Subfield</th>
         <th>Text</th>
         <th>Constraints</th>
-        <th>Delete</th>
+        <th>&nbsp;</th>
     </tr>
     [% FOREACH loo IN loop %]
     <tr>
@@ -307,7 +307,7 @@
                 [% IF ( loo.value_builder ) %] | <strong>Plugin:</strong>[% loo.value_builder %],[% END %]
             [% END %]
         </td>
-        <td><a href="[% loo.delete %]">Delete</a></td>
+        <td><a href="[% loo.delete %]" class="btn btn-mini"><i class="fa fa-trash"></i> Delete</a></td>
     </tr>
     [% END %]
 </table>
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/auth_tag_structure.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/auth_tag_structure.tt
index c495205b6e..df7ae02706 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/auth_tag_structure.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/auth_tag_structure.tt
@@ -12,7 +12,7 @@
  $(document).ready(function() {
     $("#table_authtagstructure").dataTable($.extend(true, {}, dataTablesDefaults, {
         "aoColumnDefs": [
-            { "aTargets": [ -1, -2, -3 ], "bSortable": false, "bSearchable": false },
+            { "aTargets": [ -1 ], "bSortable": false, "bSearchable": false },
         ],
         "sPaginationType": "four_button"
     }));
@@ -211,10 +211,8 @@
             <th>Repeatable</th>
             <th>Mandatory</th>
             <th>Authorized<br />value</th>
-            <th>Subfields</th>
-            <th>Edit</th>
-            <th>Delete</th>
-            </tr>
+            <th>&nbsp;</th>
+        </tr>
     </thead>
     <tbody>
         [% FOREACH loo IN loop %]
@@ -224,9 +222,17 @@
             <td>[% IF ( loo.repeatable ) %]Yes[% ELSE %]No[% END %]</td>
             <td>[% IF ( loo.mandatory ) %]Yes[% ELSE %]No[% END %]</td>
             <td>[% loo.authorised_value %]</td>
-            <td><a href="[% loo.subfield_link %]" class="button">subfields</a></td>
-            <td><a href="[% loo.edit %]">Edit</a></td>
-            <td><a href="[% loo.delete %]">Delete</a></td>
+            <td>
+                <div class="dropdown">
+                    <a class="btn btn-mini dropdown-toggle" id="authtagactions[% loo.tagfield %]" role="button" data-toggle="dropdown" href="#">
+                        Actions <b class="caret"></b></a>
+                    <ul class="dropdown-menu pull-right" role="menu" aria-labelledby="authtagactions[% loo.tagfield %]">
+                        <li><a href="[% loo.subfield_link %]"><i class="fa fa-eye"></i> Subfields</a></li>
+                        <li><a href="[% loo.edit %]"><i class="fa fa-pencil"></i> Edit</a></li>
+                        <li><a href="[% loo.delete %]"><i class="fa fa-trash"></i> Delete</a></li>
+                    </ul>
+                </div>
+            </td>
         </tr>
         [% END %]
     </tbody>
-- 
2.39.5