The header search forms use "onclick" to trigger the "keep_text()" function. This functionality can be moved to the globally-included js file.
Created attachment 51609 [details] [review] Bug 16549 - Remove the use of "onclick" from header search forms This patch removes the use of "onclick" from all header search forms for the purpose of triggering the "keep_text" function. This behavior is now handled in the globally-included JS file. To test, apply the patch and clear your cache if necessary. - Enter text in any header search form field. Click to each other tab in the header and confirm that your text is copied to each. - Test the behavior of the header search form on at least one page where each is included: - The staff client home page - The advanced search page - The authorities home page - The administration home page - The cataloging home page - The checkin page - The circulation home page - The patrons home page - Acquisitions -> Vendor -> Contracts - Administration -> Cities - Administration -> Currencies and exchange rates - Administration -> Patron categories - Administration -> Printers (why is this page still around?) - Administration -> System preferences - Administration -> Z39.50/SRU servers - Tools -> Notices & slips This patch modifies does not fix the existing (unreported) bug which prevents the keep text function from working in the include file used on these pages: - Acquisitions -> Vendor -> Basket -> New order from suggestion - Administration -> Budgets - The serials home page
Created attachment 51981 [details] [review] Bug 16549 - Remove the use of "onclick" from header search forms This patch removes the use of "onclick" from all header search forms for the purpose of triggering the "keep_text" function. This behavior is now handled in the globally-included JS file. To test, apply the patch and clear your cache if necessary. - Enter text in any header search form field. Click to each other tab in the header and confirm that your text is copied to each. - Test the behavior of the header search form on at least one page where each is included: - The staff client home page - The advanced search page - The authorities home page - The administration home page - The cataloging home page - The checkin page - The circulation home page - The patrons home page - Acquisitions -> Vendor -> Contracts - Administration -> Cities - Administration -> Currencies and exchange rates - Administration -> Patron categories - Administration -> Printers (why is this page still around?) - Administration -> System preferences - Administration -> Z39.50/SRU servers - Tools -> Notices & slips This patch modifies does not fix the existing (unreported) bug which prevents the keep text function from working in the include file used on these pages: - Acquisitions -> Vendor -> Basket -> New order from suggestion - Administration -> Budgets - The serials home page Signed-off-by: Claire Gravely <c.gravely@arts.ac.uk>
Created attachment 52646 [details] [review] Bug 16549 - Remove the use of "onclick" from header search forms This patch removes the use of "onclick" from all header search forms for the purpose of triggering the "keep_text" function. This behavior is now handled in the globally-included JS file. To test, apply the patch and clear your cache if necessary. - Enter text in any header search form field. Click to each other tab in the header and confirm that your text is copied to each. - Test the behavior of the header search form on at least one page where each is included: - The staff client home page - The advanced search page - The authorities home page - The administration home page - The cataloging home page - The checkin page - The circulation home page - The patrons home page - Acquisitions -> Vendor -> Contracts - Administration -> Cities - Administration -> Currencies and exchange rates - Administration -> Patron categories - Administration -> Printers (why is this page still around?) - Administration -> System preferences - Administration -> Z39.50/SRU servers - Tools -> Notices & slips This patch modifies does not fix the existing (unreported) bug which prevents the keep text function from working in the include file used on these pages: - Acquisitions -> Vendor -> Basket -> New order from suggestion - Administration -> Budgets - The serials home page Signed-off-by: Claire Gravely <c.gravely@arts.ac.uk> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Pushed to master for 16.11, thanks Owen!
Pushed in 16.05. Will be in 16.05.02.
This patch appears to introduce a bug reverting previous behaviour. In the staff client search the catalogue when results are displayed instead of the active tab remaining 'Search the catalogue' it is switched to 'Check out'. Users are findling this behaviour irritating
(In reply to Colin Campbell from comment #6) > This patch appears to introduce a bug reverting previous behaviour. This should be fixed by Bug 16903, which says it has been pushed to master but I don't think it really has been.
> This should be fixed by Bug 16903 16903 has now been pushed to master, so please re-test!