Attachment 35032 Details for Bug 13537 – Bug 13537 - Focus should move to barcode after renewing via checkouts table

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

Bug-13537---Focus-should-move-to-barcode-after-ren.patch (text/plain), 1.57 KB, created by Kyle M Hall (khall) on 2015-01-08 12:02:05 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Kyle M Hall (khall)

Created: 2015-01-08 12:02:05 UTC

Size: 1.57 KB

patch

obsolete

>From 7d07dec9e61b2172d2b03e6f0d5cac6ee9c10733 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] 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>
>---
> koha-tmpl/intranet-tmpl/prog/en/js/checkouts.js |    5 +++++
> 1 files changed, 5 insertions(+), 0 deletions(-)
>
>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.7.2.5

Actions: View | Diff | Splinter Review

Attachments on bug 13537: 35032 | 35127 | 35138