Attachment Details: with cursor in URL box, left/right arrows switch between attachments

[jrivett]

This is related to issue #29725 (https://core.trac.wordpress.org/ticket/29725). The changes made to resolve #29725 work for text boxes that can be edited, but the URL box is read-only, and because of that, the fix has no effect.

On the Attachment Details screen, an attachment's URL is only shown in the URL box. But typically, most of the URL is not shown, since it's longer than the box size. To see the end of the URL, I can use the End key, but that leaves the middle of the URL, which for many URLs I can only see if I can move the cursor with the left/right arrow keys.

You could make the URL box completely inaccessible (disable the cursor completely), but then the box would be essentially useless as a source of information. That seems to leave only one remaining choice, which is to disable attachment navigation using the left/right arrow keys when the cursor is in the URL box.

[joemcgill]

I can confirm that this is still a bug in trunk during the 5.2 beta cycle.

Steps to reproduce:

1. Visit: /wp-admin/upload.php?mode=grid
2. Click on a thumbnail to open the attachment details
3. Place your cursor on the URL field
4. Press the left or right keyboard arrow
5. Observe that the active media item changes

[afercia]

Related: https://core.trac.wordpress.org/ticket/41612#comment:48

[afercia]

43202.diff​ prevents left/right arrow keys navigation from happening when readonly inputs are focused. To test:

• apply the patch and build
• repeat the steps from comment:1
• observe navigation through the attachments doesn't happen

[joedolson]

Tested - patch works as expected. Marking for commit.

Hi, Andrea! Good to see you around.

[joedolson]

In 52792:

Media: Stop arrow keys switching media if URL focused.

Keyboard handlers were prevented from navigating attachments in input fields, but the key handler excluded readonly fields, like the media URL field. This could prevent exploring the URL using the keyboard.

Props jrivett, joemcgill, afercia.
Fixes #43202.