Created attachment 20497 [details] Screenshot from the documentation Bug 7494 introduced a really nice feature, and the itemBarcodeFallbackSearch syspref. Here are a couple of suggestions for improving the interface related to this syspref: - When the syspref is on, change the text above the barcode field from "Enter item barcode" to something like "Enter item barcode or keyword" - In the yellow warning box, make the phrase that was searched for stand out more. Now it is just run into one string like in the attached screenshot from the manual: "The barcode was not found outside the book Fast cataloging". "outside the book" in this example could be e.g. in bold or italics.
(In reply to Magnus Enger from comment #0) > - When the syspref is on, change the text above the barcode field from > "Enter item barcode" to something like "Enter item barcode or keyword" This is still valid in the current master (3ee3c31). I'll provide a patch for the template file soon. > - In the yellow warning box, make the phrase that was searched for stand out > more. Now it is just run into one string like in the attached screenshot > from the manual: "The barcode was not found outside the book Fast > cataloging". "outside the book" in this example could be e.g. in bold or > italics. This was fixed with commit ae40e5e4c68f50ef1c68e8901e46c73e50c31a0a (Bug 9387), and the search term is now showing in bold. I'm also changing the "Fast cataloging" link to "Add record using fast cataloging" as per the following comment: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7494#c9
Created attachment 55581 [details] [review] Bug 10768 - Improve the interface related to itemBarcodeFallbackSearch When the itemBarcodeFallbackSearch syspref is on, the wording in the interface should reflect that you can enter a barcode OR a keyword. Additionally, in the results of a keyword search the "Fast cataloging" link should be more descriptive. This patch fixes these issues. Test plan: 0) [PREREQUISITES] Ensure you have a Fast Add ('FA') framework defined, and that your itemBarcodeFallbackSearch syspref is set to 'Enable'. 1) Go to Circulation -> Check out, search for a patron, then select a patron to Check out. Notice how the text above the textbox reads "Enter item barcode:". 2) Type something generic (not a barcode) in the textbox so that you'll get at least one item as a result. Notice how the text in the yellow warning box reads "The barcode was not found: <terms> Fast cataloging". 3) Apply the patch. 4) Repeat step 1), now the text above the textbox should read "Enter item barcode or keyword:". 5) Repeat step 2), now the text in the yellow warning box should read "The barcode was not found: <terms> Add record using fast cataloging".
This looks fine except for one small thing: The changed lines include tab characters. Whenever we modify a template which contains tab characters we should replace each tab with four spaces.
Created attachment 55585 [details] [review] Bug 10768 - Improve the interface related to itemBarcodeFallbackSearch When the itemBarcodeFallbackSearch syspref is on, the wording in the interface should reflect that you can enter a barcode OR a keyword. Additionally, in the results of a keyword search the "Fast cataloging" link should be more descriptive. This patch fixes these issues. Test plan: 0) [PREREQUISITES] Ensure you have a Fast Add ('FA') framework defined, and that your itemBarcodeFallbackSearch syspref is set to 'Enable'. 1) Go to Circulation -> Check out, search for a patron, then select a patron to Check out. Notice how the text above the textbox reads "Enter item barcode:". 2) Type something generic (not a barcode) in the textbox so that you'll get at least one item as a result. Notice how the text in the yellow warning box reads "The barcode was not found: <terms> Fast cataloging". 3) Apply the patch. 4) Repeat step 1), now the text above the textbox should read "Enter item barcode or keyword:". 5) Repeat step 2), now the text in the yellow warning box should read "The barcode was not found: <terms> Add record using fast cataloging".
(In reply to Owen Leonard from comment #3) > This looks fine except for one small thing: The changed lines include tab > characters. Whenever we modify a template which contains tab characters we > should replace each tab with four spaces. Well spotted, thanks! I've updated the patch accordingly.
Created attachment 55590 [details] [review] [SIGNED-OFF] Bug 10768 - Improve the interface related to itemBarcodeFallbackSearch When the itemBarcodeFallbackSearch syspref is on, the wording in the interface should reflect that you can enter a barcode OR a keyword. Additionally, in the results of a keyword search the "Fast cataloging" link should be more descriptive. This patch fixes these issues. Test plan: 0) [PREREQUISITES] Ensure you have a Fast Add ('FA') framework defined, and that your itemBarcodeFallbackSearch syspref is set to 'Enable'. 1) Go to Circulation -> Check out, search for a patron, then select a patron to Check out. Notice how the text above the textbox reads "Enter item barcode:". 2) Type something generic (not a barcode) in the textbox so that you'll get at least one item as a result. Notice how the text in the yellow warning box reads "The barcode was not found: <terms> Fast cataloging". 3) Apply the patch. 4) Repeat step 1), now the text above the textbox should read "Enter item barcode or keyword:". 5) Repeat step 2), now the text in the yellow warning box should read "The barcode was not found: <terms> Add record using fast cataloging". Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Created attachment 55639 [details] [review] [PASSED QA] Bug 10768 - Improve the interface related to itemBarcodeFallbackSearch When the itemBarcodeFallbackSearch syspref is on, the wording in the interface should reflect that you can enter a barcode OR a keyword. Additionally, in the results of a keyword search the "Fast cataloging" link should be more descriptive. This patch fixes these issues. Test plan: 0) [PREREQUISITES] Ensure you have a Fast Add ('FA') framework defined, and that your itemBarcodeFallbackSearch syspref is set to 'Enable'. 1) Go to Circulation -> Check out, search for a patron, then select a patron to Check out. Notice how the text above the textbox reads "Enter item barcode:". 2) Type something generic (not a barcode) in the textbox so that you'll get at least one item as a result. Notice how the text in the yellow warning box reads "The barcode was not found: <terms> Fast cataloging". 3) Apply the patch. 4) Repeat step 1), now the text above the textbox should read "Enter item barcode or keyword:". 5) Repeat step 2), now the text in the yellow warning box should read "The barcode was not found: <terms> Add record using fast cataloging". Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
Pushed to master for 16.11, thanks Andreas!
Pushed in 16.05. Will be in 16.05.05.
Pushed to 3.22.x, will be in 3.22.12