asc-csa/ckanext-asc-csa

13) accessibility (pagination aria-label)

nfee006 opened this issue · 2 comments

In the pagination, the « aria-label » attribute needs to be in the <a> rather than in the <li>

ex. https://data.asc-csa.gc.ca/dataset
<li aria-label="(current) Go to page 1" class="active"><a href="/dataset?page=1">1</a></li>

N.B. the pagination is set in https://github.com/asc-csa/ckanext-asc-csa/blob/master/ckan/lib/helpers.py, starting at line 1345. This file must overwrite ckan/lib/helpers.py in vanilla CKAN in order to work properly.

should be addressed by #107

Confirmed.