Would like to see someone make the Acquisitions toolbar (in basket.pl) float when scrolling down, and remove the Add Order to Basket section at the bottom of the screen. This would eliminate loading redundant code since there is a modal version of this same section, and would load slightly faster since it isn't populating as long of a screen. Christopher
Created attachment 50701 [details] [review] Bug 12333: Acquisitions toolbar and removing Add to basket section at bottom To test: 1) Go to Acqui -> do a vendor search -> view a basket 2) Confirm that the Add Order to basket section at the bottom is gone. Confirm the add to basket button in the toolbar still works as expected 3) Confirm that when you make your browser shorter, the toolbar follows the page down as you scroll Sponsored-by: Catalyst IT
Created attachment 50717 [details] [review] [SIGNED-OFF] Bug 12333: Acquisitions toolbar and removing Add to basket section at bottom To test: 1) Go to Acqui -> do a vendor search -> view a basket 2) Confirm that the Add Order to basket section at the bottom is gone. Confirm the add to basket button in the toolbar still works as expected 3) Confirm that when you make your browser shorter, the toolbar follows the page down as you scroll Sponsored-by: Catalyst IT Signed-off-by: Owen Leonard <oleonard@myacpl.org>
I like this!
Created attachment 50796 [details] [review] [PASSED QA] Bug 12333: Acquisitions toolbar and removing Add to basket section at bottom To test: 1) Go to Acqui -> do a vendor search -> view a basket 2) Confirm that the Add Order to basket section at the bottom is gone. Confirm the add to basket button in the toolbar still works as expected 3) Confirm that when you make your browser shorter, the toolbar follows the page down as you scroll Sponsored-by: Catalyst IT Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Pushed to master! Should be in the May 2016 release! Qatlho' Aleisha!