Skip to content

Commit e040a53

Browse files
committed
Remove role=treeitem
This was causing an accessability violation, because treeitems must be a child of a tree. https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Roles/tree_role
1 parent d27a1d1 commit e040a53

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

app/assets/stylesheets/blacklight/_facets.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -144,7 +144,7 @@
144144
align-items: flex-start;
145145
padding: 3px 0;
146146

147-
&[role="treeitem"] {
147+
&.treeitem {
148148
display: block;
149149
}
150150

app/components/blacklight/facet_item_pivot_component.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ def call
3232

3333
id = "h-#{self.class.mint_id}" if @collapsing && has_items?
3434

35-
content_tag @wrapping_element, role: 'treeitem', class: 'treeitem' do
35+
content_tag @wrapping_element, class: 'treeitem' do
3636
concat(content_tag('span', class: "d-flex flex-row align-items-center") do
3737
concat facet_toggle_button(id) if has_items? && @collapsing
3838
concat content_tag('span', render(facet), class: "facet-values d-flex flex-row flex-grow-1 #{'facet-leaf-node' if has_items? && @collapsing}", id: id && "#{id}_label")

0 commit comments

Comments
 (0)