Webstaff Holdings View action menu can be disorienting

Bug #1717551 reported by Bill Erickson
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Wishlist
Unassigned

Bug Description

Evergreen 3.0.beta

The "Actions" (or right-click) menu for rows in the web staff record detail Holdings View tab can be a little bit disorienting. For example, a menu action labeled "Copies" appears 3 times. Each performs a different action based on which heading (Add, Edit, Delete) the action sits under. If the heading is not view, the user is forced to scroll up to find the heading. What's more, in a rush it would be very easy to select the wrong action.

Perhaps in addition to the grouping, the action could be part of the label as well. For example:

Edit
 -- Edit Volumes
 -- Edit Copies
 ...

Andrea Neiman (aneiman)
tags: added: cataloging
Katlyn Beck (klbeck)
Changed in evergreen:
assignee: nobody → Katlyn Beck (klbeck)
Revision history for this message
Katlyn Beck (klbeck) wrote :

I added clearer language to the options under the add, edit, and delete sections.

The branch for this fix can be found here:
http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/klbeck/lp1717551-holdings-view-action-menu-adjustment

I noticed the same issue in the interfaces for circulation -> item status -> list and detail views. Would it be worth reporting bugs for those as well?

Changed in evergreen:
assignee: Katlyn Beck (klbeck) → nobody
tags: added: pullrequest
Bill Erickson (berick)
Changed in evergreen:
assignee: nobody → Bill Erickson (berick)
status: New → Confirmed
Revision history for this message
Bill Erickson (berick) wrote :

Looks good Katlyn. Thanks!

I have pushed a sign-off branch here:

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/berick/lp1717551-holding-actions-extend-labels

I will hold off on merging until others have had a chance to review.

Katlyn, opening a new bug for circ -> item status would be useful, but you may want to wait and see how this bug plays out.

tags: added: signedoff
Changed in evergreen:
milestone: none → 3.2.3
assignee: Bill Erickson (berick) → nobody
Changed in evergreen:
milestone: 3.2.3 → 3.3-beta1
Revision history for this message
Dan Wells (dbw2) wrote :

I moved this to 'wishlist' given that is not a bug fix, and will have translation implications for a very highly visible menu.

If others strongly feel this should be backported, that's probably fine, I could get on board with that.

Changed in evergreen:
importance: Undecided → Low
importance: Low → Wishlist
no longer affects: evergreen/3.2
no longer affects: evergreen/3.1
Revision history for this message
Bill Erickson (berick) wrote :

Grabbing for 3.3 merging...

Changed in evergreen:
assignee: nobody → Bill Erickson (berick)
Revision history for this message
Bill Erickson (berick) wrote :

Re-tested and merged to master for 3.3. I think this is a nice addition. Thanks, Katlyn.

Dan, keeping this as a feature request is reasonable to me, more so given i18n implications. Marking as fix committed.

Changed in evergreen:
status: Confirmed → Fix Committed
assignee: Bill Erickson (berick) → nobody
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.