Library Settings Editor - History Link Missing Cursor Change

Bug #2039306 reported by Jennifer Pringle
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

Evergreen 3.11

In the Library Settings Editor when you hover over the History link for a setting the cursor doesn't change into a pointer to indicate that you can click on the link. (Clicking on the History link does open the history, it just doesn't look like it will.)

Changed in evergreen:
assignee: nobody → Stephanie Leary (stephanieleary)
tags: added: accessibility
Revision history for this message
Stephanie Leary (stephanieleary) wrote :

The pointer style indicates a more serious problem: the History link can't be reached using the keyboard. It was written as an <a> tag with a click event rather than a <button>, which would provide both keyboard support and the correct pointer style.

Branch here: https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/sleary/LP2039306-library-settings-history-link

tags: added: pullrequest
Changed in evergreen:
status: New → Confirmed
assignee: Stephanie Leary (stephanieleary) → nobody
Andrea Neiman (aneiman)
Changed in evergreen:
importance: Undecided → Medium
milestone: none → 3.12-beta
Revision history for this message
Jennifer Pringle (jpringle-u) wrote :

Looks good on our test server. The cursor turns into a pointer and I can tab through to the History links on my keyboard.

I have tested this code and consent to signing off on it with my name, Jennifer Pringle and my email address, <email address hidden>.

tags: added: signedoff
Revision history for this message
Jane Sandberg (sandbergja) wrote :

Confirming the same results as Jennifer. Thanks, Stephanie and Jennifer! Pushed to main and rel_3_11. There are merge conflicts with rel_3_10, so I did not attempt the backport.

Changed in evergreen:
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.