Unverified Commit 01de0a84 authored by Lauri Eskola's avatar Lauri Eskola
Browse files

Issue #3333401 by nicxvan, bnjmnm: Pager h4 causes accessibility flag on many pages

parent 7347556c
......@@ -33,8 +33,7 @@
*/
#}
{% if items %}
<nav class="pager" role="navigation" aria-labelledby="{{ heading_id }}">
<h4 id="{{ heading_id }}" class="visually-hidden">{{ 'Pagination'|t }}</h4>
<nav class="pager" role="navigation" aria-label="{{ 'Pagination'|t }}">
<ul class="pager__items js-pager__items">
{# Print first item if we are not on the first page. #}
{% if items.first %}
......
......@@ -13,8 +13,7 @@
*/
#}
{% if items.previous or items.next %}
<nav role="navigation" aria-labelledby="{{ heading_id }}">
<h4 id="{{ heading_id }}" class="visually-hidden">{{ 'Pagination'|t }}</h4>
<nav class="pager" role="navigation" aria-label="{{ 'Pagination'|t }}">
<ul class="js-pager__items">
{% if items.previous %}
<li>
......
......@@ -11,8 +11,7 @@
*/
#}
{% if items.previous or items.next %}
<nav class="pager" role="navigation" aria-labelledby="{{ heading_id }}">
<h4 id="{{ heading_id }}" class="pager__heading visually-hidden">{{ 'Pagination'|t }}</h4>
<nav class="pager" role="navigation" aria-label="{{ 'Pagination'|t }}">
<ul class="pager__items js-pager__items">
{% if items.previous %}
<li class="pager__item pager__item--previous">
......
......@@ -35,8 +35,7 @@
*/
#}
{% if items %}
<nav class="pager" role="navigation" aria-labelledby="{{ heading_id }}">
<h4 id="{{ heading_id }}" class="visually-hidden">{{ 'Pagination'|t }}</h4>
<nav class="pager" role="navigation" aria-label="{{ 'Pagination'|t }}">
<ul class="pager__items js-pager__items">
{# Print first item if we are not on the first page. #}
{% if items.first %}
......
......@@ -18,8 +18,7 @@
]
%}
{% if items.previous or items.next %}
<nav{{ attributes.addClass('pager').setAttribute('role', 'navigation').setAttribute('aria-labelledby', heading_id) }}>
<h4{{ title_attributes.addClass('visually-hidden').setAttribute('id', heading_id) }}>{{ 'Pagination'|t }}</h4>
<nav class="pager" role="navigation" aria-label="{{ 'Pagination'|t }}">
<ul{{ content_attributes.addClass('pager__items', 'js-pager__items') }}>
{% if items.previous %}
{% apply spaceless %}
......
......@@ -31,8 +31,7 @@
*/
#}
{% if items %}
<nav class="pager layout--content-medium" role="navigation" aria-labelledby="{{ heading_id }}">
<h4 id="{{ heading_id }}" class="visually-hidden">{{ 'Pagination'|t }}</h4>
<nav class="pager" role="navigation" aria-label="{{ 'Pagination'|t }}">
<ul class="pager__items js-pager__items">
{# Print first item if we are not on the first page. #}
{% if items.first %}
......
......@@ -11,8 +11,7 @@
*/
#}
{% if items.previous or items.next %}
<nav class="pager" role="navigation" aria-labelledby="{{ heading_id }}">
<h4 id="{{ heading_id }}" class="visually-hidden">{{ 'Pagination'|t }}</h4>
<nav class="pager" role="navigation" aria-label="{{ 'Pagination'|t }}">
<ul class="pager__items js-pager__items">
{# Print previous item if we are not on the first page. #}
{% if items.previous %}
......
......@@ -31,8 +31,7 @@
*/
#}
{% if items %}
<nav class="pager" role="navigation" aria-labelledby="{{ heading_id }}">
<h4 id="{{ heading_id }}" class="visually-hidden">{{ 'Pagination'|t }}</h4>
<nav class="pager" role="navigation" aria-label="{{ 'Pagination'|t }}">
<ul class="pager__items js-pager__items">
{# Print first item if we are not on the first page. #}
{% if items.first %}
......
......@@ -11,8 +11,7 @@
*/
#}
{% if items.previous or items.next %}
<nav class="pager" role="navigation" aria-labelledby="{{ heading_id }}">
<h4 id="{{ heading_id }}" class="pager__heading visually-hidden">{{ 'Pagination'|t }}</h4>
<nav class="pager" role="navigation" aria-label="{{ 'Pagination'|t }}">
<ul class="pager__items js-pager__items">
{% if items.previous %}
<li class="pager__item pager__item--previous">
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment