Attachment 51659 Details for Bug 16437 – Bug 16437 - (follow-up) Automatic item modifications by age needs prettying

[patch] Bug 16437 - (follow-up) Automatic item modifications by age needs prettying

Bug-16437---follow-up-Automatic-item-modifications.patch (text/plain), 1.62 KB, created by Jonathan Druart on 2016-05-20 14:41:38 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2016-05-20 14:41:38 UTC

Size: 1.62 KB

patch

obsolete

>From 65308c83dfe6808e76014e8b626cf108ff16abac Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>Date: Fri, 20 May 2016 15:40:24 +0100
>Subject: [PATCH] Bug 16437 - (follow-up) Automatic item modifications by age
> needs prettying
>
>Do not move the scrollbar to the top when removing a 'block' to a new
>rule.
>
>Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>---
> .../intranet-tmpl/prog/js/automatic_item_modification_by_age.js     | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/js/automatic_item_modification_by_age.js b/koha-tmpl/intranet-tmpl/prog/js/automatic_item_modification_by_age.js
>index 1ebce82..d14b55a 100644
>--- a/koha-tmpl/intranet-tmpl/prog/js/automatic_item_modification_by_age.js
>+++ b/koha-tmpl/intranet-tmpl/prog/js/automatic_item_modification_by_age.js
>@@ -36,7 +36,8 @@ function clone_block(block) {
>     $(new_block).find('a.remove_block').show();
>     var blocks = $(block).parent();
>     $(blocks).append(new_block);
>-    $(blocks).find('a.remove_block').click(function(){
>+    $(blocks).find('a.remove_block').click(function(e){
>+        e.preventDefault();
>         remove_block_action($(this));
>     }).show();
> }
>@@ -91,7 +92,8 @@ $(document).ready(function() {
>         if( $("#rules").find(".conditions > .condition").length > 1 ) {
> 
>         }
>-        new_rule.find('.remove_rule').click(function(){
>+        new_rule.find('.remove_rule').click(function(e){
>+            e.preventDefault();
>             remove_rule_action( $(this) );
>         }).show();
>         new_rule.find('.add_rule').remove();
>-- 
>2.7.0

Actions: View | Diff | Splinter Review

Attachments on bug 16437: 51232 | 51233 | 51243 | 51658 | 51659