Add aria-describedby when multiple links have identical text

This branch adds the barcode (for lack of anything better) to links in the items table. I'm very open to ideas on anything else we could add here to create a better audible user experience.

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/sleary/lp2016343-aria-describedby-item-table

Thanks, Stephanie! I think it's definitely a step in the right direction. Your patch really got my gears turning, so here are a few thoughts:

1. I'm not sure that aria-describedby is the best tool for the job when it comes to lists of links. I tried your patch in NVDA and VoiceOver. NVDA announced the item barcode after the visible link text when focused, but it did not appear in the elements list (which just had "Edit", "View", etc.). On VoiceOver, when I focused one of the links, it gave me the option to listen to the barcode, but as with NVDA, the barcodes did not appear in the link names in the rotor.

2. Both the call number and barcode edit links were announced as "Edit FIC400001583". The call number "edit" links should be announced differently than the item "edit" links -- maybe they could be labeled by the call number label text (e.g. FIC 223)?

3. The "view" link in the barcode column will also need some differentiation.

4. I wonder if labeling the items by their row number, rather than their barcode, might be a better user experience. On the one hand, listening all the way through a barcode number just to see if the very last digit matches what you expect is a little rough; on the other hand, the barcode is more meaningful than just the row number.

Thanks for trying this out, Jane! It's frustrating that the description isn't included in the list. If we add sr-only text to the link itself, it'll foul things up for dictation users; the accessible name has to match what's visible on screen.

I thought about using the row number. Would that be at all helpful when the links are out of context? Virtually anything would be better than the barcode, I would think.

I've updated the branch to distinguish the two Edit links and fix the View link, but that doesn't address the larger issues with the link list. I have not added the row number for now.

Once we reach an acceptable solution here, we will need to make a list of other screens that have repetitive links like this, and apply the same fix throughout.

Some further reading on this:
https://adrianroselli.com/2022/04/accessible-description-exposure.html
https://benmyers.dev/blog/aria-labels-and-descriptions/#aria-describedby

It appears to be the expected behavior that the description is included in some modes, but not others (including the VO rotor). It does get included when the item has focus.

Thanks for the links, Stephanie -- those were very helpful. I really like the idea of creating a good solution here, and then applying that solution to the other screens that need it.

Upon reading the second link, I feel that the differentiator isn't "longer-form, supplemental information" -- it's core to understanding what the links are and where they go.

One thing I found out recently is that aria-labeledby can refer to itself, e.g.:

<span id="barcode-2">My Barcode</span>
<button id="edit-2" aria-labeledby="edit-2 barcode-2">Edit</button>

This announces "Edit My Barcode" in VoiceOver when the button has focus. It announces the same in the rotor. I'm not sure what the implications are for voice control, though.

Noting merge conflicts for both commits on master today

Conflicts taken care of; thanks to Blake and Jason B for troubleshooting!

Works for me! Signed off and fix pushed as far back as 3.11 (did not apply cleanly to 3.10)