Bug 17972

Summary: Reformat acquisitions sidebar menu with acquisitions and administration sections
Product: Koha Reporter: Owen Leonard <oleonard>
Component: TemplatesAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: katrin.fischer, kyle
Version: master   
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5784
Change sponsored?: --- Patch complexity: Small patch
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 5784    
Bug Blocks:    
Attachments: Screenshot of the proposed change
Bug 17972 - Reformat acquisitions sidebar menu with acquisitions and administration sections
[SIGNED-OFF] Bug 17972 - Reformat acquisitions sidebar menu with acquisitions and administration sections
Bug 17972 - Reformat acquisitions sidebar menu with acquisitions and administration sections
Bug 17972 - Reformat acquisitions sidebar menu with acquisitions and administration sections

Description Owen Leonard 2017-01-23 15:45:51 UTC
Created attachment 59430 [details]
Screenshot of the proposed change

Inspired by the discussion on Bug 5784 I propose reformatting the acquisitions sidebar menu so that /acqui link are in one section and /admin links are in another, with added links to the respective home pages.
Comment 1 Owen Leonard 2017-01-23 16:17:22 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.
Comment 2 Aleisha Amohia 2017-01-23 22:36:24 UTC
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>
Comment 3 Jonathan Druart 2017-01-26 10:22:11 UTC
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>
Comment 4 Jonathan Druart 2017-02-16 10:46:39 UTC
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>
Comment 5 Kyle M Hall 2017-02-17 12:11:09 UTC
Pushed to master for 17.05, thanks Owen!
Comment 6 Katrin Fischer 2017-02-19 20:22:08 UTC
Love it, but...

This won't get ported back to 16.11.x as it is an enhancement.