Bug 14193

Summary: Accessibility: Searching patrons using the alphabetic index doesn't work
Product: Koha Reporter: Katrin Fischer <katrin.fischer>
Component: PatronsAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: brendan, chris, f.demians, gmcharlt, indradg, julian.maurice, kyle.m.hall, nengard
Version: MainKeywords: Academy
Hardware: All   
OS: All   
See Also: http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15602
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15603
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on:    
Bug Blocks: 15603    
Attachments: Bug 14193 - Accessibility: Searching patrons using the alphabetic index doesn't work
Bug 14193 - Accessibility: Searching patrons using the alphabetic index doesn't work
[PASSED QA] Bug 14193 - Accessibility: Searching patrons using the alphabetic index doesn't work

Description Katrin Fischer 2015-05-13 08:46:36 UTC
It's not possible to use the alphabetic index search using only the keyboard.

The librarian reporting this is using braille line, screenreader and keyboard for navigating and using Koha. 

After selecting a character from the alphabetic index list and pressing enter to trigger the feature, the table of patrons is loaded, but it always shows 'no results found', even if there are patrons for that letter in the database.
Comment 1 Katrin Fischer 2016-01-18 00:07:00 UTC
This is a bit tricky, but maybe someone has an idea :)
Comment 2 Owen Leonard 2016-01-19 16:46:47 UTC Comment hidden (obsolete)
Comment 3 Nicole C. Engard 2016-01-19 19:17:40 UTC
Owen,

I applied the patch and cleared my cache, but the tab is not going to the letters.

Did I miss something? I'm in Firefox on Mac

Nicole
Comment 4 Briana Greally 2016-01-19 21:05:34 UTC Comment hidden (obsolete)
Comment 5 Katrin Fischer 2016-01-19 22:11:01 UTC
Created attachment 46948 [details] [review]
[PASSED QA] Bug 14193 - Accessibility: Searching patrons using the alphabetic index doesn't work

Using a mouse to click on the alphabetic index letters works even though
the anchor markup doesn't contain an href attribute. However you
can't tab to them using the keyboard, and I assume the issue with
screen readers is related.

This patch adds a dummy href attribute and a class-based click handler
so that we can get rid of the "onclick" attribute in the markup.

To test, apply the patch and visit the "Patrons" section.

1. Use the tab key to move the focus to one of the alphabet links.
2. Hit "Enter" and confirm that the search is performed correctly.
3. Click any of the alphabet links and confirm that clicking works as
   well.
4. View the details of any patron and click the "Add child" button.
5. In the "Guarantor information" section click the "Change" button.
6. In the search popup, use the tab key to move the focus to one of the
   alphabet links.
2. Hit "Enter" and confirm that the search is performed correctly.
3. Click any of the alphabet links and confirm that clicking works as
   well.

Signed-off-by: Briana <brianagreally@gmail.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 6 Katrin Fischer 2016-01-19 22:11:33 UTC
Thx a lot for this, I had no idea where to start!
Comment 7 Brendan Gallagher 2016-01-23 19:02:41 UTC
Pushed to Master - Should be in the May 2016 release.  - Thanks!
Comment 8 Julian Maurice 2016-01-25 11:51:40 UTC
Patch pushed to 3.22.x, will be in 3.22.2
Comment 9 Frédéric Demians 2016-01-25 16:28:00 UTC
This patch has been pushed to 3.20.x, will be in 3.20.8.