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>

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 7caceca..b62de1d 100644 (file)
--- 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 c495205..df7ae02 100644 (file)
--- 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>