OPAC - My Lists - list action button order could be improved

Bug #2047592 reported by Terran McCanna
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Low
Unassigned
3.11
Fix Released
Low
Unassigned

Bug Description

Bootstrap OPAC 3.10+

The order of the list buttons is:
Share - Download CSV - Make Default List - Edit List

When you click the Share button, it adds a button for "View in Catalog" but puts it between other buttons so it looks like this (with Edit List being pushed to second line):

Hide - Download CSV - make Default List - View in Catalog
Edit List

I think it makes more sense to either a) put the View in Catalog button directly after the Hide button or b) put it at the end on the second line.

Revision history for this message
Terran McCanna (tmccanna) wrote :

Also - I'm not sure why View in Catalog is a different color, but that is probably a different discussion.

Revision history for this message
Terran McCanna (tmccanna) wrote :
tags: added: pullrequest
Revision history for this message
Jason Boyer (jboyer) wrote :

Hi, this could use a rebase against current main due to some changes in lists.tt2.

tags: added: needsrebase
removed: pullrequest
Changed in evergreen:
assignee: nobody → Terran McCanna (tmccanna)
Revision history for this message
Terran McCanna (tmccanna) wrote :

Never try to rebase at the end of a long week. I ended up just creating a new branch against main with the change:

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/mccanna/lp2047592_opac_lists_button_order_rev

tags: added: pullrequest
removed: needsrebase
Changed in evergreen:
assignee: Terran McCanna (tmccanna) → nobody
Revision history for this message
Stephanie Leary (stephanieleary) wrote :

I agree that we should revisit these buttons at some point, but for now this solves the issue! Signoff branch at https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/sleary/lp2047592_opac_lists_button_order_rev-signoff

tags: added: signedoff
Revision history for this message
Galen Charlton (gmc) wrote :

Pushed down to rel_3_11. Thanks, Terran and Stephanie!

Changed in evergreen:
status: New → Confirmed
importance: Undecided → Low
milestone: none → 3.12.3
status: Confirmed → Fix Committed
Changed in evergreen:
status: Fix Committed → Fix Released
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.