Staff catalog facets "show less" has incorrect count

Bug #2046974 reported by Andrea Neiman
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Medium
Unassigned
3.11
Fix Released
Medium
Unassigned

Bug Description

3.12.0 ; 3.11.2

Follow up from bug 1934018.

The new staff catalog facets have a "Less" link to show fewer facets; however, when hovering over the "Less" link, the description is always "Show 5" even if there are fewer than 5 results

Similarly, the "More" link always displays even if there are not more options to be shown in the facets.

These links should be context aware, with counts, if possible.

Changed in evergreen:
status: New → Confirmed
Revision history for this message
Stephanie Leary (stephanieleary) wrote (last edit ):

Here's a branch that hides the "more"/"less" button if the facet has 5 or fewer links: https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/sleary/LP2046974-facet-more-link-count

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

The test should be "> 5", not ">= 5", no? Otherwise the toggle still gets displayed if the facet happens to have exactly five entries.

Galen Charlton (gmc)
Changed in evergreen:
importance: Undecided → Medium
Revision history for this message
Stephanie Leary (stephanieleary) wrote :

You are right; I second-guessed myself on that at the last minute for some reason. I've fixed that and squashed the fix into the same branch.

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

Thanks! Tested and signed off: working/user/gmcharlt/lp2046974_signoff

tags: added: signedoff
Bill Erickson (berick)
Changed in evergreen:
assignee: nobody → Bill Erickson (berick)
milestone: none → 3.12.3
Revision history for this message
Bill Erickson (berick) wrote :

Merged to main and 3.12.

I added 3.11 as a target to this bug but did not merge since there are merge conflicts with 3.11.

Changed in evergreen:
status: Confirmed → Fix Committed
assignee: Bill Erickson (berick) → nobody
Revision history for this message
Galen Charlton (gmc) wrote :

I was comfortable with the merge conflict resolution, so I've gone ahead and pushed it to rel_3_11 as well.

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.