Bugzilla – Attachment 38426 Details for
Bug 11464
Saving preferences - Nothing to Save
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 11464: Prevent wrong "nothing to save" message on saving syspref
Bug-11464-Prevent-wrong-nothing-to-save-message-on.patch (text/plain), 1.88 KB, created by
Jonathan Druart
on 2015-04-23 15:05:45 UTC
(
hide
)
Description:
Bug 11464: Prevent wrong "nothing to save" message on saving syspref
Filename:
MIME Type:
Creator:
Jonathan Druart
Created:
2015-04-23 15:05:45 UTC
Size:
1.88 KB
patch
obsolete
>From 9ddd80f901513a7c72c0bfead72696b525d27b30 Mon Sep 17 00:00:00 2001 >From: Jonathan Druart <jonathan.druart@biblibre.com> >Date: Thu, 23 Apr 2015 16:57:31 +0200 >Subject: [PATCH] Bug 11464: Prevent wrong "nothing to save" message on saving > syspref > >Sometime you get the "Nothing to save" message when clicking on the "save >all preferences" button, even if something has changed. > >On irritating behavior is the middle click: if you use the middle click >to paste some text in the input, the "keyup" event won't be trigger. >This patch replaces the "keyup" event with the "change", which works >as we want here. > >The issues: >1/ Search for "wanted" (admin/preferences.pl?op=search&searchfield=wanted) >Empty the input, click save >Type something and remove it >You are able to save, but nothing has changed >2/ Put something in your selection buffer and use the middle click to >paste it in the input. >No change detected. > >Test plan: >Confirm the issues are fixed and you don't find any regression >Note that now the "(modified)" string is displayed when the input loses >the focus. >--- > 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 9800f47..04135ef 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' ).keyup( function () { >+ .find( 'input.preference, textarea.preference' ).on('change', function () { > if ( this.defaultValue === undefined || this.value != this.defaultValue ) mark_modified.call( this ); > } ).end() > .find( 'select.preference' ).change( mark_modified ); >-- >2.1.0
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 11464
:
38426
|
38462
|
38616