In order to conform to best practices for performance staff client template javascript should be in the footer instead of the header. This patch will do so for the staff client cart.
Created attachment 67278 [details] [review] Bug 19356: Move staff client cart JavaScript to the footer This patch modifies the staff client cart template so that JavaScript is included in the footer instead of the header. To test, apply the patch and test the JavaScript-driven features of the cart: All button controls, DataTables functionality.
Created attachment 67386 [details] [review] [SIGNED-OFF] Bug 19356: Move staff client cart JavaScript to the footer This patch modifies the staff client cart template so that JavaScript is included in the footer instead of the header. To test, apply the patch and test the JavaScript-driven features of the cart: All button controls, DataTables functionality. Signed-off-by: Aleisha Amohia <aleishaamohia@hotmail.com>
Why the conditional for footerjs? If we are setting it statically why not just always include the js in footer? If I understand the variable is to prvent doc-head-close / intranet-bottom.inc from adding more js
Created attachment 68080 [details] [review] Bug 19356: Move staff client cart JavaScript to the footer This patch modifies the staff client cart template so that JavaScript is included in the footer instead of the header. To test, apply the patch and test the JavaScript-driven features of the cart: All button controls, DataTables functionality. Signed-off-by: Aleisha Amohia <aleishaamohia@hotmail.com> Correction for QA: Removed contditional around footer JS (from copy-paste) because it will always be true.
Pushed to master for 17.11, thanks to everybody involved!