Problem/Motivation

The block used for exposed filters on content overview pages such as /all-topics does not contain a heading element.

Proposed resolution

I think using an h2 element for this would be a good idea since it's usually used on a page whose primary purpose is to filter the content. To make styling easier we can also move the title classes to be on the heading.

Replace <header{{ title_attributes.addClass('complementary-title') }}>{{ label }}</header> with <header><h2 {{ title_attributes.addClass('complementary-title') }}>{{ label }}</h2></header>

User interface changes

The header of a filter block is now properly marked as heading. No visual changes.

API changes

None

Data model changes

None

CommentFileSizeAuthor
#2 3171613-2.patch684 bytesdspachos
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Kingdutch created an issue. See original summary.

dspachos’s picture

FileSize
684 bytes

Here it is

Kingdutch’s picture

Status: Active » Needs review
Issue tags: +Accessibility
Kingdutch’s picture

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.