Attachment 35138 Details for Bug 13537 – [PASSED QA] Bug 13537 - Focus should move to barcode after renewing via checkouts table

[patch] [PASSED QA] Bug 13537 - Focus should move to barcode after renewing via checkouts table

PASSED-QA-Bug-13537---Focus-should-move-to-barcode.patch (text/plain), 1.66 KB, created by Katrin Fischer on 2015-01-13 07:01:52 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Katrin Fischer

Created: 2015-01-13 07:01:52 UTC

Size: 1.66 KB

patch

obsolete

>From 39e28d573eb98164d0b3332b74c250a30ac0077d Mon Sep 17 00:00:00 2001
>From: Kyle M Hall <kyle@bywatersolutions.com>
>Date: Thu, 8 Jan 2015 06:59:26 -0500
>Subject: [PATCH] [PASSED QA] Bug 13537 - Focus should move to barcode after
> renewing via checkouts table
>
>Koha's checkout screen automatically focuses on the barcode field each
>time it is loaded. Since we've moved the checkouts table functions for
>renewing and returning items to ajax methods, this page is no longer
>refreshed, and thus does not refocus to the barcode after performing
>those actions. This should be fixed so that renewing/returning items via
>the checkouts page moves the focus to the barcode field.
>
>Test Plan:
>1) Find a patron with checkouts.
>2) Renew or return a checkout, note the focus does not move to the
>   barcode.
>3) Apply this patch.
>4) Renew or return a checkout, note the focus *does* move to the
>   barcode.
>
>Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
>Signed-off-by: Thomas <tomsStudy@gmail.com>
>
>Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
>---
> koha-tmpl/intranet-tmpl/prog/en/js/checkouts.js | 5 +++++
> 1 file changed, 5 insertions(+)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/js/checkouts.js b/koha-tmpl/intranet-tmpl/prog/en/js/checkouts.js
>index a135496..738dafe 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/js/checkouts.js
>+++ b/koha-tmpl/intranet-tmpl/prog/en/js/checkouts.js
>@@ -114,6 +114,11 @@ $(document).ready(function() {
>             }, "json")
>         });
> 
>+        // Refocus on barcode field if it exists
>+        if ( $("#barcode").length ) {
>+            $("#barcode").focus();
>+        }
>+
>         // Prevent form submit
>         return false;
>     });
>-- 
>1.9.1

Actions: View | Diff | Splinter Review

Attachments on bug 13537: 35032 | 35127 | 35138