Remove redundant Pagination header

Created on 12 March 2024, over 1 year ago

Problem/Motivation

The "Pagination" nav elements are labelled both by a hidden header and an aria-label attribute. They should only have one and since it is hidden, it would be best for it to be the attribute. Removing the header also fixes accessibility errors related to skipped heading levels or improper nesting of headers.

Proposed resolution

Remove the headers and the aria-labelledby attributes that refer to them.

Remaining tasks

Implement.

User interface changes

None. (Currently, aria-label is ignored because aria-labelledby is present.)

API changes

None.

Data model changes

None.

🐛 Bug report

Status

Active

Version

4.0

Component

Code

Created by

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

Live updates comments and jobs are added and updated live.

Comments & Activities

Issue created by @Liam Morland
Status changed to Needs review over 1 year ago2:36pm 12 March 2024
Comment over 1 year ago →
🇨🇦Canada Liam Morland Ontario, CA 🇨🇦
Status changed to Closed: won't fix 2 months ago9:09am 25 August 2025
Comment 2 months ago →
🇦🇺Australia jannakha Brisbane!
is not a set value. pagination_heading_level is configured on the view itself by drupal/core.
This change came in with drupal/core (10.2.7 => 10.3.0) config.
If this has to be removed - please submit an issue to the core as it's a core functionality.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024