Skip to content

Commit

Permalink
fix(@dpc-sdp/ripple-tide-search): added missing aria-expanded to sear…
Browse files Browse the repository at this point in the history
…ch filter toggle
  • Loading branch information
jeffdowdle committed Sep 19, 2023
1 parent 7c2dc41 commit d6601e8
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -330,11 +330,13 @@ watch(
v-if="userFilters && userFilters.length > 0"
class="tide-search-refine-btn"
:expanded="filtersExpanded"
aria-controls="tide-search-listing-filters"
@click="handleToggleFilters"
>{{ toggleFiltersLabel }}</RplSearchBarRefine
>
<RplExpandable
v-if="userFilters && userFilters.length > 0"
id="tide-search-listing-filters"
:expanded="filtersExpanded"
class="rpl-u-margin-t-4"
>
Expand Down
6 changes: 5 additions & 1 deletion packages/ripple-tide-search/components/TideSearchPage.vue
Original file line number Diff line number Diff line change
Expand Up @@ -223,10 +223,14 @@ watch(
<RplSearchBarRefine
class="tide-search-refine-btn"
:expanded="filtersExpanded"
aria-controls="tide-search-page-filters"
@click="toggleFilters"
>{{ toggleFiltersLabel }}</RplSearchBarRefine
>
<RplExpandable :expanded="filtersExpanded">
<RplExpandable
id="tide-search-page-filters"
:expanded="filtersExpanded"
>
<RplForm
v-if="staticFacetOptions !== null"
id="tide-search-filter-form"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@ withDefaults(defineProps<Props>(), {

<template>
<button
:aria-expanded="expanded"
:class="{
'rpl-search-bar-refine': true,
'rpl-search-bar-refine--expanded': expanded,
Expand Down

0 comments on commit d6601e8

Please sign in to comment.