Tab order is not intuitive on admin splash pages

Bug #2015137 reported by Jane Sandberg
14
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Medium
Unassigned
3.11
Fix Released
Medium
Unassigned
3.9
Fix Released
Medium
Unassigned

Bug Description

Expected behavior:

When I go to Admin > Local Administration, I can use the Tab key to navigate the links in alphabetical order.

Actual behavior:
The tab order is:
* Address Alerts
* Field Documentation
* Search Filter Groups
* Age Overdue Circulations to Lost, etc.

This means that you cannot quickly tab between, say, all the settings that begin with "MARC", "Circulation", or "Shelving Location".

This also affects booking, acquisitions, and server admin screens.

Beth Willis (willis-a)
Changed in evergreen:
status: New → Confirmed
Revision history for this message
Stephanie Leary (stephanieleary) wrote :

Since CSS columns are widely supported now, we could do away with the row/column logic in staff/share/link-table/link-table.component.ts and simply make this an unordered list with style="columns: {{columnCount}} auto"

That would remove the bottom border on the rows, but that's so faint that I can barely tell it's there.

Revision history for this message
Stephanie Leary (stephanieleary) wrote :

Branch: https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/sleary/lp-2015137-link-table-taborder

Note that the rows no longer have borders.

To test, make sure keyboard navigation is enabled in your browser and visit any of the Angular settings screens: Administration > Server Administration (or Local, Acquisitions, or Booking).

Focus somewhere near the page title and press Tab repeatedly. The links should be highlighted in alphabetical order, down the columns.

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

Works for me. Signoff branch is user/gmcharlt/lp2015137_signoff

Changed in evergreen:
importance: Undecided → Medium
tags: added: signedoff
Revision history for this message
Jane Sandberg (sandbergja) wrote :

Great patch! Thanks, Stephanie and Galen. Pushed to rel_3_9 and above.

Changed in evergreen:
status: Confirmed → Fix Committed
milestone: none → 3.10.2
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.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.