Skip to content

Commit c46b381

Browse files
committed
GitHub: Remove row and cell roles from release asset items.
It would be ideal if this were a table, but we're not currently matching the table correctly and we're breaking the links by giving them a role of cell. For now, just don't mess with it until we have a better solution. Re jcsteh#28.
1 parent 640f437 commit c46b381

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

GitHubA11yFixes.user.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
// @author James Teh <jteh@mozilla.com>
66
// @copyright 2019-2024 Mozilla Corporation, Derek Riemer
77
// @license Mozilla Public License version 2.0
8-
// @version 2024.1
8+
// @version 2024.2
99
// @include https://github.com/*
1010
// ==/UserScript==
1111

@@ -146,9 +146,9 @@ const DYNAMIC_TWEAKS = [
146146
// Issue listing tables.
147147
{selector: '.js-navigation-container:not(.commits-listing)',
148148
tweak: el => el.setAttribute("role", "table")},
149-
{selector: '.Box-row:not(.js-commits-list-item)',
149+
{selector: '.js-navigation-container:not(.commits-listing) .Box-row',
150150
tweak: el => el.setAttribute("role", "row")},
151-
{selector: '.Box-row .d-flex',
151+
{selector: '.js-navigation-container:not(.commits-listing) .Box-row .d-flex',
152152
tweak: el => {
153153
// There's one of these inside every row. It's purely presentational.
154154
makePresentational(el);

0 commit comments

Comments
 (0)