Web Staff Client - Accessibility and the Item Edits Page

Bug #1887869 reported by Mike Risher
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Evergreen
Confirmed
Medium
Unassigned
3.10
Confirmed
Medium
Unassigned
3.11
Confirmed
Medium
Unassigned
3.9
Won't Fix
Undecided
Unassigned

Bug Description

The page that lets you edit items and edit item templates needs to have labels added so that they're easier to use with screen readers. Check all the various input fields on these forms.

Revision history for this message
Mike Risher (mrisher) wrote :

I pushed a branch with these changes:

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/mrisher/lp1887869-item-edits-accessibility

Please note that a few of the changes rely on edits to shared directives. These edits are already done and can be found on bug 1887866.

tags: added: pullrequest
Changed in evergreen:
status: New → Confirmed
Revision history for this message
Terran McCanna (tmccanna) wrote :

Mike, I got a merge conflict when I tried to cherry-pick this to current master. Any chance you could rebase?

Revision history for this message
Mike Risher (mrisher) wrote :
Revision history for this message
Mike Risher (mrisher) wrote :

Testing instructions:

1. You'll need a screen reader for testing. JAWS and NVDA are popular options. Instructions here: https://developer.paciellogroup.com/blog/2015/01/basic-screen-reader-commands-for-accessibility-testing/

If installing a screen reader isn't practical, Firefox can be used to test accessibility also. https://developer.mozilla.org/en-US/docs/Tools/Accessibility_inspector

2. Load changes to both the shared directives (bug #1887866) and these changes to the item edit pages (branch is in comment #3 above)

3. Test to see if the issues in the bug description have been addressed

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

I loaded this in conjunction with the patch for lp1887866 for testing and went through the fields with the Firefox accessibility inspector and all the label names came through correctly. Thanks! My signoff is at:

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/mccanna/lp1887869-item-edits-accessibility-cp-signoff

tags: added: signedoff
Changed in evergreen:
milestone: none → 3.6.1
importance: Undecided → Medium
Changed in evergreen:
milestone: 3.6.1 → 3.6.2
Changed in evergreen:
milestone: 3.6.2 → 3.6.3
tags: added: itemtemplates
removed: webstaffclient
Changed in evergreen:
milestone: 3.6.3 → 3.6.4
Changed in evergreen:
milestone: 3.6.4 → 3.7.2
Beth Willis (willis-a)
tags: added: cat-templates
removed: itemtemplates
Changed in evergreen:
assignee: nobody → MaryAnn Alexander (maryann-alexander)
Changed in evergreen:
assignee: MaryAnn Alexander (maryann-alexander) → nobody
Changed in evergreen:
milestone: 3.7.2 → 3.7.3
no longer affects: evergreen/3.6
Changed in evergreen:
milestone: 3.7.3 → none
Changed in evergreen:
milestone: none → 3.9.1
Changed in evergreen:
milestone: 3.9.1 → 3.9.2
Michele Morgan (mmorgan)
Changed in evergreen:
milestone: 3.9.2 → 3.10.1
Changed in evergreen:
milestone: 3.10.1 → 3.10.2
Revision history for this message
Galen Charlton (gmc) wrote :

Will commit when the merge window reopens after the May releases.

Changed in evergreen:
assignee: nobody → Galen Charlton (gmc)
Revision history for this message
Stephanie Leary (stephanieleary) wrote :

Looking more closely, the ARIA labeling strategy needed significant revision. I have a WIP branch at https://git.evergreen-ils.org/?p=working/Evergreen.git;a=commitdiff;h=b96e0d127919a673829d11ae990177c7a510bd80 but this needs additional review and testing.

tags: added: needswork
removed: signedoff
Galen Charlton (gmc)
Changed in evergreen:
assignee: Galen Charlton (gmc) → nobody
Andrea Neiman (aneiman)
tags: removed: pullrequest
Changed in evergreen:
milestone: 3.10.2 → 3.10.3
Changed in evergreen:
milestone: 3.10.3 → 3.12-beta
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.