Description
Owen Leonard
2017-01-23 15:45:51 UTC
Created attachment 59435 [details] [review] Bug 17972 - Reformat acquisitions sidebar menu with acquisitions and administration sections This patch makes changes to acquisitions-menu.inc so that the style matches that of the Administration and Tools sidebar menus. The links are now divided up between Acquisitions (/acqui/) and Administration (/admin/). The link-highlighting JavaScript used elsewhere has been added. To test, apply the patch and view the Acquisitions home page. - Check that the "Acquisitions home" link is highlighted (bold). - Test the other Acquisitions links to confirm that highlighting works. - Click any link in the Administration section of the sidebar menu. - Confirm that everything looks correct and works correctly. Created attachment 59471 [details] [review] [SIGNED-OFF] Bug 17972 - Reformat acquisitions sidebar menu with acquisitions and administration sections This patch makes changes to acquisitions-menu.inc so that the style matches that of the Administration and Tools sidebar menus. The links are now divided up between Acquisitions (/acqui/) and Administration (/admin/). The link-highlighting JavaScript used elsewhere has been added. To test, apply the patch and view the Acquisitions home page. - Check that the "Acquisitions home" link is highlighted (bold). - Test the other Acquisitions links to confirm that highlighting works. - Click any link in the Administration section of the sidebar menu. - Confirm that everything looks correct and works correctly. Signed-off-by: Aleisha Amohia <aleishaamohia@hotmail.com> Created attachment 59569 [details] [review] Bug 17972 - Reformat acquisitions sidebar menu with acquisitions and administration sections This patch makes changes to acquisitions-menu.inc so that the style matches that of the Administration and Tools sidebar menus. The links are now divided up between Acquisitions (/acqui/) and Administration (/admin/). The link-highlighting JavaScript used elsewhere has been added. To test, apply the patch and view the Acquisitions home page. - Check that the "Acquisitions home" link is highlighted (bold). - Test the other Acquisitions links to confirm that highlighting works. - Click any link in the Administration section of the sidebar menu. - Confirm that everything looks correct and works correctly. Signed-off-by: Aleisha Amohia <aleishaamohia@hotmail.com> Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> Created attachment 60318 [details] [review] Bug 17972 - Reformat acquisitions sidebar menu with acquisitions and administration sections This patch makes changes to acquisitions-menu.inc so that the style matches that of the Administration and Tools sidebar menus. The links are now divided up between Acquisitions (/acqui/) and Administration (/admin/). The link-highlighting JavaScript used elsewhere has been added. To test, apply the patch and view the Acquisitions home page. - Check that the "Acquisitions home" link is highlighted (bold). - Test the other Acquisitions links to confirm that highlighting works. - Click any link in the Administration section of the sidebar menu. - Confirm that everything looks correct and works correctly. Signed-off-by: Aleisha Amohia <aleishaamohia@hotmail.com> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Pushed to master for 17.05, thanks Owen! Love it, but... This won't get ported back to 16.11.x as it is an enhancement. |