Attachment 40344 Details for Bug 14389 – Bug 14389 - Editing a syspref in a textarea does not enable the Save button

[patch] Bug 14389 - Editing a syspref in a textarea does not enable the Save button

Bug-14389---Editing-a-syspref-in-a-textarea-does-n.patch (text/plain), 1.49 KB, created by Liz Rea on 2015-06-18 23:54:44 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Liz Rea

Created: 2015-06-18 23:54:44 UTC

Size: 1.49 KB

patch

obsolete

>From f3af0cb3ad1465ab664c263d941d7500e65f0242 Mon Sep 17 00:00:00 2001
>From: Liz Rea <liz@catalyst.net.nz>
>Date: Tue, 16 Jun 2015 16:12:57 +1200
>Subject: [PATCH] Bug 14389 - Editing a syspref in a textarea does not enable
> the Save button
>
>Test plan:
>  1. Navigate to the "opaccredits" syspref (or any other textarea, i.e.,
>     "Click to Edit", syspref) in the system preferences editor.
>  2. Change its contents, by either pasting or typing. The field may not
>     be marked as modified, even after you click outside the box.
>  3. Apply the patch.
>  4. Reload the page and try again; either pasting or typing should mark
>     the field as changed and allow you to save.
>---
> koha-tmpl/intranet-tmpl/prog/en/js/pages/preferences.js | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/js/pages/preferences.js b/koha-tmpl/intranet-tmpl/prog/en/js/pages/preferences.js
>index 04135ef..b6537d9 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/js/pages/preferences.js
>+++ b/koha-tmpl/intranet-tmpl/prog/en/js/pages/preferences.js
>@@ -63,7 +63,7 @@ $( document ).ready( function () {
>     }
> 
>     $( '.prefs-tab' )
>-        .find( 'input.preference, textarea.preference' ).on('change', function () {
>+        .find( 'input.preference, textarea.preference' ).on('input', function () {
>             if ( this.defaultValue === undefined || this.value != this.defaultValue ) mark_modified.call( this );
>         } ).end()
>         .find( 'select.preference' ).change( mark_modified );
>-- 
>1.9.1

Actions: View | Diff | Splinter Review

Attachments on bug 14389: 40156 | 40182 | 40344 | 40345 | 40361 | 40772