Created attachment 51506 [details] [review] Bug 16524: Use floating toolbar on item search To test: 1) Go to item search 2) Confirm toolbar at top of page, goes down page as you scroll 3) Both output options (screen and CSV) work as expected Sponsored-by: Catalyst IT
I don't think the output options belong in the toolbar since they are not an "action." I think they belong in the main body of the form, probably in their own <fieldset>.
Created attachment 51553 [details] [review] Bug 16524: Use floating toolbar on item search To test: 1) Go to item search 2) Confirm toolbar at top of page with Search button, goes down page as you scroll 3) Confirm search button has been removed from final output fieldset 4) Confirm everything works as expected Sponsored-by: Catalyst IT
Created attachment 51570 [details] [review] [SIGNED-OFF] Bug 16524: Use floating toolbar on item search To test: 1) Go to item search 2) Confirm toolbar at top of page with Search button, goes down page as you scroll 3) Confirm search button has been removed from final output fieldset 4) Confirm everything works as expected Sponsored-by: Catalyst IT Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Created attachment 51594 [details] [review] Bug 16524: Use floating toolbar on item search To test: 1) Go to item search 2) Confirm toolbar at top of page with Search button, goes down page as you scroll 3) Confirm search button has been removed from final output fieldset 4) Confirm everything works as expected Sponsored-by: Catalyst IT Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Pushed to Master - Should be in the November 2016 Release. Thanks!
Pushed in 16.05. Will be in 16.05.01.