Diff options
authorBryce Kwon <bryce@brycekwon.com>
committerBryce Kwon <bryce@brycekwon.com>
commit
parent
tree
download
Name the graph and links header cells
Both were empty, so a screen reader moving through the table announced those columns with no name. An aria-label names them without adding text to the page.
Diffstat (limited to '')
-rw-r--r--source/ui-log.c2
-rw-r--r--source/ui-tree.c2
-rwxr-xr-xtests/t0103-log.sh3
-rwxr-xr-xtests/t0104-tree.sh3
4 files changed, 7 insertions, 3 deletions
diff --git a/source/ui-log.c b/source/ui-log.c
index a01ef4e..870ff06 100644
--- a/source/ui-log.c
+++ b/source/ui-log.c
@@ -585,7 +585,7 @@ void cgit_print_log(const char *tip, int ofs, int cnt, char *grep, char *pattern
html("<tbody class='log'>\n<tr class='nohover'>");
if (commit_graph)
- html("<th></th>");
+ html("<th scope='col' aria-label='Graph'></th>");
else
html("<th class='left' scope='col'>Age</th>");
html("<th class='left' scope='col'>Commit message");
diff --git a/source/ui-tree.c b/source/ui-tree.c
index 1d8bb3d..69c0763 100644
--- a/source/ui-tree.c
+++ b/source/ui-tree.c
@@ -425,7 +425,7 @@ static void ls_head(void)
html("<th class='left ls-mode' scope='col'>Mode</th>");
html("<th class='left' scope='col'>Name</th>");
html("<th class='right' scope='col'>Size</th>");
- html("<th class='ls-links'></th>");
+ html("<th class='ls-links' scope='col' aria-label='Links'></th>");
html("</tr>\n");
}
diff --git a/tests/t0103-log.sh b/tests/t0103-log.sh
index 573baa8..d7e745d 100755
--- a/tests/t0103-log.sh
+++ b/tests/t0103-log.sh
@@ -107,7 +107,8 @@ test_expect_success 'the log draws the commit graph when enabled' '
sed "s/^cache-size=.*/cache-size=0/" cgitrc
} >graphrc &&
CGIT_CONFIG="$PWD/graphrc" QUERY_STRING="url=foo/log/" cgit >tmp &&
- grep "class=.commitgraph." tmp
+ grep "class=.commitgraph." tmp &&
+ grep "<th scope=.col. aria-label=.Graph.></th>" tmp
'
# The search is literal, so text that is no valid regular expression still
diff --git a/tests/t0104-tree.sh b/tests/t0104-tree.sh
index d464f8c..41134a4 100755
--- a/tests/t0104-tree.sh
+++ b/tests/t0104-tree.sh
@@ -11,6 +11,9 @@ test_description='Check content on tree page'
test_expect_success 'generate bar/tree' 'cgit_url "bar/tree" >tmp'
test_expect_success 'find file-1' 'grep "file-1" tmp'
test_expect_success 'find file-50' 'grep "file-50" tmp'
+test_expect_success 'the links column has a name' '
+ grep "<th class=.ls-links. scope=.col. aria-label=.Links.></th>" tmp
+'
test_expect_success 'generate bar/tree/file-50' 'cgit_url "bar/tree/file-50" >tmp'