Bugzilla – Attachment 20526 Details for
Bug 3134
Ability to select multiple reports to delete at once
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 3134: Add the warn class on tr elements
Bug-3134-Add-the-warn-class-on-tr-elements.patch (text/plain), 1.48 KB, created by
Jonathan Druart
on 2013-08-21 08:47:13 UTC
(
hide
)
Description:
Bug 3134: Add the warn class on tr elements
Filename:
MIME Type:
Creator:
Jonathan Druart
Created:
2013-08-21 08:47:13 UTC
Size:
1.48 KB
patch
obsolete
>From 34af5b1cf3916cf7b41f48c3fe2bcfae740f5abc Mon Sep 17 00:00:00 2001 >From: Jonathan Druart <jonathan.druart@biblibre.com> >Date: Wed, 21 Aug 2013 10:23:11 +0200 >Subject: [PATCH] Bug 3134: Add the warn class on tr elements > >This patch adds the same behavior when deleting 1 or more reports. >When checkboxes are checked and the "delete selected" button is pressed, >checked lines are hightlighted in red. > >Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> >--- > .../prog/en/modules/reports/guided_reports_start.tt | 8 +++++++- > 1 file changed, 7 insertions(+), 1 deletion(-) > >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/reports/guided_reports_start.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/reports/guided_reports_start.tt >index 60bbe28..c16013e 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/reports/guided_reports_start.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/reports/guided_reports_start.tt >@@ -129,7 +129,13 @@ $(document).ready(function(){ > alert(_("You must select one or more reports to delete")); > return false; > } >- return confirm(_("Are you sure you want to delete the selected reports?")); >+ $(checkedItems).parents('tr').addClass("warn"); >+ if( confirm(_("Are you sure you want to delete the selected reports?")) ) { >+ return true; >+ } else { >+ $(checkedItems).parents('tr').removeClass("warn"); >+ return false; >+ } > }); > [% END %] > >-- >1.7.10.4
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 3134
:
14604
|
19376
|
19806
|
19843
|
20461
|
20465
|
20516
|
20517
|
20524
|
20525
| 20526 |
20527
|
20528