Comment 6 for bug 1929741

Notes from testing for a bit this morning:

- I was able to activate a completely empty PO. Neither "Allow activation with zero copy line items" or "Activate without loading items" were checked. This was just the regular "Activate" button.

- There are options in the Actions menu that shouldn't be accessible depending on the state of the PO. "Add Brief Record" should be greyed out when the PO is in any state other than 'pending.' Mark Received, Mark Unreceived, and Cancel Lineitems should be greyed out when the PO is 'pending'.
-- Because they're not, I was able to add a brief record to an on-order PO, and trying to receive something on a pending PO gives you the ugly message "Batch operation failed: ACQ_LI_RECEIVE_FAILED".

- Clicking the link for the provider in the PO should open the provider profile in another tab, not the same window/tab.

- The link for Cancel Order is always visible, even when the PO is pending. It shouldn't appear until the PO is on-order.

- When a line item is cancelled with a keep_debits=true reason (like Delayed:Backorder), the line item changes to white (it should be blue) and the line item state bubble just says 'Canceled'. It should display the whole cancel reason. If that's too long, it should at least say 'Delayed' instead of 'Canceled'.

- When I'm in the Items area, I added 2 items and hit Save Changes. If I then remove those two items but *don't* hit Save Changes again, the line item still shows a count of 2 copies no matter how many times you flip back and forth between the main PO and the Items, even though the copies are deleted. I ran into this because...

- I did not notice the Save Changes button at all for a bit because it's so far over to the right. My preference would be similar to the layout we have now, with Item Count | Field for Count | Apply and then the green Save Changes box directly to the right of it, with the Distribution Formula box below it. If others like them all on the same line, I'd at least like the green Save Changes to be aligned left so it's more obvious.

- Line Item-View History -- Audit time, expected receive time, edit time, and create time should all be timestamps.

- Acquisitions (Experimental)>Create Purchase Order -- The "Import Bibs and Create Copies" ticky should be removed, since it's not present in the current Create Purchase Order dialog and I'm not sure what it would do there anyway.

- Fields that are currently typeahead that should be dropdowns:
* Direct charges - Fund
* Items - Copy Location, Fund, Circ Mod
* Cancel Reason dialog box - Cancel Reason

UX preferences, subject to others' preferences and feasibility:

- I'd prefer that the fields in the Items batch updater and copies were not filled with the helper text. See bug 1906862.

- It would be nice if line items had a border around them, even if it was a light gray one(?) to separate them from the white of the back of the page. All of the line item colors are generally pretty pastel/light, and defining them with a border is easier on the eyes. This is especially useful when you do have canceled line items, since they're white. Without a border they just kind of look like they're floating on the page, for lack of a better description.

- I prefer having buttons for Activate/Activate without Loading Items/Cancel Order, personally. It makes them stand out, especially because you can get the cue that there's something off if they're greyed out.