Skip to content

Instantly share code, notes, and snippets.

@jcsteh
Created October 24, 2019 11:08
Show Gist options
  • Save jcsteh/50c5afc4a9884a5c6f58ecd5fbc7e16b to your computer and use it in GitHub Desktop.
Save jcsteh/50c5afc4a9884a5c6f58ecd5fbc7e16b to your computer and use it in GitHub Desktop.
User script to improve the accessibility of GitHub (mark 2)
// ==UserScript==
// @name GitHub Accessibility Fixes
// @namespace http://axSgrease.nvaccess.org/
// @description Improves the accessibility of GitHub.
// @author James Teh <jteh@mozilla.com>
// @copyright 2019 Mozilla Corporation, Derek Riemer
// @license Mozilla Public License version 2.0
// @version 2019.1
// @include https://github.com/*
// ==/UserScript==
/*** Functions for common tweaks. ***/
function makeHeading(el, level) {
el.setAttribute("role", "heading");
el.setAttribute("aria-level", level);
}
function makeRegion(el, label) {
el.setAttribute("role", "region");
el.setAttribute("aria-label", label);
}
function makeButton(el, label) {
el.setAttribute("role", "button");
el.setAttribute("aria-label", label);
}
function makePresentational(el) {
el.setAttribute("role", "presentation");
}
function setLabel(el, label) {
el.setAttribute("aria-label", label);
}
function makeHidden(el) {
el.setAttribute("aria-hidden", "true");
}
function setExpanded(el, expanded) {
el.setAttribute("aria-expanded", expanded ? "true" : "false");
}
var idCounter = 0;
// Get a node's id. If it doesn't have one, make and set one first.
function setAriaIdIfNecessary(elem) {
if (!elem.id) {
elem.setAttribute("id", "axsg-" + idCounter++);
}
return elem.id;
}
function makeElementOwn(parentElement, listOfNodes){
ids = [];
for(let node of listOfNodes){
ids.push(setAriaIdIfNecessary(node));
}
parentElement.setAttribute("aria-owns", ids.join(" "));
}
/*** Code to apply the tweaks when appropriate. ***/
function applyTweak(el, tweak) {
if (Array.isArray(tweak.tweak)) {
let [func, ...args] = tweak.tweak;
func(el, ...args);
} else {
tweak.tweak(el);
}
}
function applyTweaks(root, tweaks, checkRoot) {
for (let tweak of tweaks) {
for (let el of root.querySelectorAll(tweak.selector)) {
applyTweak(el, tweak);
}
if (checkRoot && root.matches(tweak.selector)) {
applyTweak(root, tweak);
}
}
}
let observer = new MutationObserver(function(mutations) {
for (let mutation of mutations) {
try {
if (mutation.type === "childList") {
for (let node of mutation.addedNodes) {
if (node.nodeType != Node.ELEMENT_NODE) {
continue;
}
applyTweaks(node, DYNAMIC_TWEAKS, true);
}
} else if (mutation.type === "attributes") {
applyTweaks(mutation.target, DYNAMIC_TWEAKS, true);
}
} catch (e) {
// Catch exceptions for individual mutations so other mutations are still handled.
console.log("Exception while handling mutation: " + e);
}
}
});
function init() {
applyTweaks(document, LOAD_TWEAKS, false);
applyTweaks(document, DYNAMIC_TWEAKS, false);
options = {childList: true, subtree: true};
if (DYNAMIC_TWEAK_ATTRIBS.length > 0) {
options.attributes = true;
options.attributeFilter = DYNAMIC_TWEAK_ATTRIBS;
}
observer.observe(document, options);
}
/*** Define the actual tweaks. ***/
// Tweaks that only need to be applied on load.
const LOAD_TWEAKS = [
];
// Attributes that should be watched for changes and cause dynamic tweaks to be
// applied. For example, if there is a dynamic tweak which handles the state of
// a check box and that state is determined using an attribute, that attribute
// should be included here.
const DYNAMIC_TWEAK_ATTRIBS = [];
// Tweaks that must be applied whenever a node is added/changed.
const DYNAMIC_TWEAKS = [
// Lines of code which can be commented on.
{selector: '.add-line-comment',
tweak: el => {
// Put the comment button after the code instead of before.
// el is the Add line comment button.
// nextElementSibling is the actual code.
makeElementOwn(el.parentNode, [el.nextElementSibling, el]);
}},
// Make non-comment events into headings; e.g. closing/referencing an issue,
// approving/requesting changes to a PR, merging a PR. Exclude commits and
// commit references because these contain too much detail and there's no
// way to separate the header from the body.
{selector: '.TimelineItem:not(.js-commit) .TimelineItem-body:not(.my-0):not([id^="ref-commit-"])',
tweak: [makeHeading, 3]},
// Table lists; e.g. in issue and commit listings.
{selector: '.js-navigation-container',
tweak: el => el.setAttribute("role", "table")},
{selector: '.Box-row',
tweak: el => el.setAttribute("role", "row")},
{selector: '.Box-row .d-table',
tweak: el => {
// There's one of these inside every row. It's purely presentational.
makePresentational(el);
// Its children are the cells, but they have no common class.
for (let cell of el.children) {
cell.setAttribute("role", "cell");
}
}},
];
/*** Lights, camera, action! ***/
init();
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment