Bug 16640 - Move actions on acquisitions receipt summary page into menu
Summary: Move actions on acquisitions receipt summary page into menu
Status: CLOSED WONTFIX
Alias: None
Product: Koha
Classification: Unclassified
Component: Templates (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Owen Leonard
QA Contact: Testopia
URL: /cgi-bin/koha/acqui/parcel.pl?invoice...
Keywords:
Depends on: 16592 30333
Blocks:
  Show dependency treegraph
 
Reported: 2016-06-02 17:51 UTC by Owen Leonard
Modified: 2023-06-08 22:26 UTC (History)
5 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Small patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Screenshot showing one part of the proposed change (29.76 KB, image/png)
2016-06-02 17:51 UTC, Owen Leonard
Details
Bug 16640 - Move actions on acquisitions receipt summary page into menu (19.19 KB, patch)
2016-06-14 13:58 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 16640 - Move actions on acquisitions receipt summary page into menu (19.32 KB, patch)
2016-06-14 16:28 UTC, Marc Véron
Details | Diff | Splinter Review
Bug 16640 - Move actions on acquisitions receipt summary page into menu (19.38 KB, patch)
2016-06-24 18:03 UTC, Kyle M Hall
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Owen Leonard 2016-06-02 17:51:20 UTC
Created attachment 51965 [details]
Screenshot showing one part of the proposed change

I propose to consolidate some links on the acquisitions receipt summary page into menus:

- MARC and Card view
- Receive, transfer, cancel etc.
Comment 1 Owen Leonard 2016-06-14 13:58:35 UTC Comment hidden (obsolete)
Comment 2 Marc Véron 2016-06-14 16:28:48 UTC Comment hidden (obsolete)
Comment 3 Jonathan Druart 2016-06-19 13:56:56 UTC
I'd say the number of items and orders left should still be displayed directly, without having to click on the actions button.
Comment 4 Kyle M Hall 2016-06-24 18:03:07 UTC
Created attachment 52859 [details] [review]
Bug 16640 - Move actions on acquisitions receipt summary page into menu

This patch moves two groups of links into two menus: MARC and Card
preview into one, Receive, transfer, cancel, etc. into another.

Also changed: The transfer order pop-up window has been sized down to be
more consistent with other similar pop-ups. Baskets and basket
group links now include the whole name rather than just the number to
make it easier to click.

To test you must have a basket or baskets with multiple titles on order,
both received and pending.

- Apply the patch and go to Acquisitions -> Vendor -> Receive shipment
  -> Receive a new shipment (or open an existing invoice).
- In the list of pending orders:
  - Test the 'View' menu for multiple titles. The 'MARC' and 'Card' menu
    items should work correctly.
  - Test the 'Actions' menu for the following actions:
    - Receive
    - Transfer
    - Cancel order
    - Cancel order and delete catalog record
  - Confirm that under some circumstances the 'Cancel order' and 'Cancel
    order and delete catalog record' items are disabled. In these cases
    hovering over the menu items should trigger a tooltip. An
    explanation should be displayed in red at the bottom of the menu.
    Clicking the menu item should trigger an alert with the explanation.

Followed test plan, works as expected.
Signed-off-by: Marc Véron <veron@veron.ch>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 5 Owen Leonard 2016-06-24 18:05:37 UTC
I think based on Jonathan's comment 3 that this needs some more opinions.
Comment 6 Katrin Fischer 2019-02-04 23:13:06 UTC
I like what I see on the screenshot, the only change I'd suggest is making "Receive" the default action (split button). 

I think the remaining orders, items, subscriptions currently takes up a lot of space while cancelling an order is not the default action. It be totally ok with it only showing on mouseover/click.

Idea: Relabel it to receive order (to make it easier to distinguish from receive for serials in translatons.
Comment 7 Nick Clemens 2019-06-10 19:10:20 UTC
Agreeing with both Katrin and Jonathan here - moving to Assigned for future work
Comment 8 Owen Leonard 2022-08-19 14:11:42 UTC
I'm closing this, since any work I may do in the future will take a different approach.