Acq Wishlist --> Additional action menu added to bottom of page

Bug #1486201 reported by Christine Burns
20
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Evergreen
Won't Fix
Wishlist
Unassigned

Bug Description

Acquisitions

Wishlist

2.8

We would to add an additional "Action menu" to the bottom of the page in a few acquisitions interfaces.

Currently there is only one actions button at the top of the page on Purchase Orders & Selection lists. For large orders / lists this means a lot of scrolling back to the top of the page.

It would be very helpful if the actions menu was also added to the bottom of the page or if a simple "back to top" button was added.

Tags: acq acq-po
tags: added: acq
Changed in evergreen:
importance: Undecided → Wishlist
Revision history for this message
Tiffany Little (tslittle) wrote :

Here's a patch:

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=commit;h=04a977e3729e20ce9d1d378efab27e7b520f3d83

This adds a "Back to Top" link at the end of the line items that jumps you back up to the top of the PO.

This was created after my fix for this bug: https://bugs.launchpad.net/evergreen/+bug/1418694 so there are other changes to li_table.tt2 that are reflected there. If there's a more Git-appropriate way to handle that, please let me know and I'll fix.

tags: added: pullrequest
Changed in evergreen:
milestone: none → 3.next
Revision history for this message
Mike Risher (mrisher) wrote :

I tried this out. I wasn't 100% sure what page this was supposed to be on, but I found it on line item search results. This path: /eg/staff/acq/legacy/search/unified

My test was unsuccesful. When I clicked the new link I wasn't taken to the top. It's trying to link to id #oils-acq-picklist-header but that id isn't on the page.

Also, we should aboid inline styling and try to stick to bootstrap classes. So instead of
style="font-weight:bold;margin-top:20px;" We can use bootstrap's class="font-weight-bold" and for the top margin class="mt-3" (3 might be the wrong number. mt-4 or mt-5 might be closer to margin-top: 20px)

tags: added: needsrepatch
removed: pullrequest
Changed in evergreen:
status: New → Confirmed
tags: added: acq-po
removed: needsrepatch
Revision history for this message
Tiffany Little (tslittle) wrote :

I'm going to mark this as Won't Fix because I think it's--as Jennifer Pringle says on the other bug--the same issue as bug 2006767 with a different solution. Given the age of this bug and that we have the Angular PO/LI interface now, and thus more options, I think we should consider this a bug of its time.

If anyone disagrees with me, please comment and I can always reopen it.

Changed in evergreen:
status: Confirmed → Won't Fix
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.