Attachment 49748 Details for Bug 16162 – Bug 16162: Making dictionary 'Delete' a button

[patch] Bug 16162: Making dictionary 'Delete' a button

Bug-16162-Making-dictionary-Delete-a-button.patch (text/plain), 2.43 KB, created by Aleisha Amohia on 2016-03-31 21:48:00 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Aleisha Amohia

Created: 2016-03-31 21:48:00 UTC

Size: 2.43 KB

patch

obsolete

>From a625221666c16f2a37f72f2a73f4e74930efa50f Mon Sep 17 00:00:00 2001
>From: Aleisha <aleishaamohia@hotmail.com>
>Date: Tue, 29 Mar 2016 23:44:56 +0000
>Subject: [PATCH] Bug 16162: Making dictionary 'Delete' a button
>MIME-Version: 1.0
>Content-Type: text/plain; charset=UTF-8
>Content-Transfer-Encoding: 8bit
>
>EDIT: This patch should have gotten rid of those tabs
>EDIT2: Will now apply with patch on Bug 16161
>
>To test:
>1) Apply Bug 16161 first
>2) Go to Reports -> View dictionary
>3) Confirm that the 'Delete definition' button now just says 'Delete' and is styled as a font awesome button.
>4) Confirm that the button does not wrap on narrower browsers.
>5) Confirm that column heading is now 'Action'.
>6) Confirm that button deletes as expected. (If you're wondering about the missing confirm message upon delete, see Bug 16161.)
>
>Sponsored-by: Catalyst IT
>
>Signed-off-by: Marc VÃ©ron <veron@veron.ch>
>---
> koha-tmpl/intranet-tmpl/prog/en/modules/reports/dictionary.tt |    6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/reports/dictionary.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/reports/dictionary.tt
>index 1b366e9..fbae1d3 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/reports/dictionary.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/reports/dictionary.tt
>@@ -71,7 +71,7 @@ var MSG_CONFIRM_DELETE = _("Are you sure you want to delete this dictionary defi
> 		<th>Description</th>
> 		<th>Area</th>
> 		<th>Definition</th>
>-		<th>&nbsp;</th>
>+                <th>Action</th>
> 		</tr>
> 		[% FOREACH definition IN definitions %]
>         <tr>
>@@ -79,10 +79,10 @@ var MSG_CONFIRM_DELETE = _("Are you sure you want to delete this dictionary defi
> 			<td>[% definition.description %]</td>
> 			<td>[% definition.areaname %]</td>
> 			<td>[% definition.saved_sql %]</td>
>-			<td><form method="post" action="/cgi-bin/koha/reports/dictionary.pl">
>+                        <td class="actions"><form method="post" action="/cgi-bin/koha/reports/dictionary.pl">
> 			<input type="hidden" name="id" value="[% definition.id %]" />
> 			<input type="hidden" name="phase" value="Delete Definition" />
>-			<input type="submit" name="submit" value="Delete Definition" onclick="return confirm(MSG_CONFIRM_DELETE);" />
>+                        <button type="submit" name="submit" class="btn btn-mini" onclick="return confirm(MSG_CONFIRM_DELETE);"><i class="fa fa-trash"></i> Delete</button>
> 			</form></td>
> 			</tr>
> 		[% END %]
>-- 
>1.7.10.4

Actions: View | Diff | Splinter Review

Attachments on bug 16162: 49661 | 49713 | 49715 | 49748 | 49816 | 49819