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

Comments

Kingdutch created an issue. See original summary.

dspachos’s picture

StatusFileSize
new684 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.