Plugins list table: description header and cells announced as "clickable"

[afercia]

Small thing but it's a bit confusing for screen reader users. Seems it happens just with Firefox + NVDA. When navigating through the Plugins table, the description header and all the description cells get announced as "clickable" elements. See in the screenshot below, where the "NVDA Speech Viewer" reports the text being announced.

As far as I see, there's a delegated click event needed to open the thickbox Plugin details popup that causes this. Simply delegating on the plugin-description div instead of the table cells fixes this, as the div doesn't get announced by screen readers.

[afercia]

In the proposed patch:

• avoid the Plugins table Description cells being announced as "clickable" elements to screen readers

[SergeyBiryukov]

The Details link is not in the .plugin-description div, looks like it should be .plugin-version-author-uri.

[afercia]

Ops, you're right. Also, the plugin-description selector missed the leading dot ".". Looks like it worked just because of the default event that thickbox delegates on the body.

The new patch uses .plugin-version-author-uri btw noticed this way, also in the plugin cards, there are two click events: the one from plugin-install.js and the one from thickbox.js and both will fire when clicking on the details link. Open to suggestions :)

[afercia]

there are two click events

Nevermind, it's using call().

[SergeyBiryukov]

In 33462:

Plugins list table: Make selector for Details links more specific to prevent the Description header from being announced as "clickable".

props afercia.
fixes #32968.