http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11360 Christopher Brannon <cbrannon@cdalibrary.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |cbrannon@cdalibrary.org --- Comment #4 from Christopher Brannon <cbrannon@cdalibrary.org> --- Looks good. I tested in a sandbox and it did what was expected. Hold trap failed before the patch applied, and then the hold trap worked after. I like that the barcode field is completely deactivated, rather than the hold trap just taking the focus off. This forces you to push a button. Our current jquery that addresses this just takes the focus off. We tossed around the idea of deactivating the barcode field altogether, but didn't have the nerve to do that much. I think the behavior is acceptable and expected. Christopher -- You are receiving this mail because: You are watching all bug changes.