From 9bb5600e69433b8c3bfeaeb0e5a1fa223602b8ae Mon Sep 17 00:00:00 2001
From: Matthew Bargar <mbargar@gmail.com>
Date: Tue, 8 Aug 2017 12:31:04 -0400
Subject: [PATCH 1/2] Make 'all filters' actions keyboard accessible

---
 src/ui/public/filter_bar/filter_bar.html | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/src/ui/public/filter_bar/filter_bar.html b/src/ui/public/filter_bar/filter_bar.html
index 29a60c192a049..c376c921d4883 100644
--- a/src/ui/public/filter_bar/filter_bar.html
+++ b/src/ui/public/filter_bar/filter_bar.html
@@ -80,7 +80,7 @@
     ng-show="filters.length"
   >
     <div class="filter-description small">
-      <a ng-click="showFilterActions = !showFilterActions">
+      <a ng-click="showFilterActions = !showFilterActions" kbn-accessible-click>
         Actions
         <span
           class="fa"
@@ -117,25 +117,25 @@
       <div class="filter-description"><strong>All filters:</strong></div>
     </div>
       <div class="filter-link">
-        <div class="filter-description"><a ng-click="toggleAll(false)">Enable</a></div>
+        <div class="filter-description"><a ng-click="toggleAll(false)" kbn-accessible-click>Enable</a></div>
       </div>
     <div class="filter-link">
-      <div class="filter-description"><a ng-click="toggleAll(true)">Disable</a></div>
+      <div class="filter-description"><a ng-click="toggleAll(true)" kbn-accessible-click>Disable</a></div>
     </div>
     <div class="filter-link">
-      <div class="filter-description"><a ng-click="pinAll(true)">Pin</a></div>
+      <div class="filter-description"><a ng-click="pinAll(true)" kbn-accessible-click>Pin</a></div>
     </div>
     <div class="filter-link">
-      <div class="filter-description"><a ng-click="pinAll(false)">Unpin</a></div>
+      <div class="filter-description"><a ng-click="pinAll(false)" kbn-accessible-click>Unpin</a></div>
     </div>
     <div class="filter-link">
-      <div class="filter-description"><a ng-click="invertAll()">Invert</a></div>
+      <div class="filter-description"><a ng-click="invertAll()" kbn-accessible-click>Invert</a></div>
     </div>
     <div class="filter-link">
-      <div class="filter-description"><a ng-click="toggleAll()">Toggle</a></div>
+      <div class="filter-description"><a ng-click="toggleAll()" kbn-accessible-click>Toggle</a></div>
     </div>
     <div class="filter-link">
-      <div class="filter-description"><a ng-click="removeAll()" data-test-subj="removeAllFilters">Remove</a></div>
+      <div class="filter-description"><a ng-click="removeAll()" data-test-subj="removeAllFilters" kbn-accessible-click>Remove</a></div>
     </div>
   </div>
 </div>

From 8a3343f854dfe63c334abd28e78e56c909872982 Mon Sep 17 00:00:00 2001
From: Matthew Bargar <mbargar@gmail.com>
Date: Tue, 8 Aug 2017 12:39:54 -0400
Subject: [PATCH 2/2] Add extra aria props

---
 src/ui/public/filter_bar/filter_bar.html | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/src/ui/public/filter_bar/filter_bar.html b/src/ui/public/filter_bar/filter_bar.html
index c376c921d4883..ce8d7bbe8c4b8 100644
--- a/src/ui/public/filter_bar/filter_bar.html
+++ b/src/ui/public/filter_bar/filter_bar.html
@@ -80,7 +80,12 @@
     ng-show="filters.length"
   >
     <div class="filter-description small">
-      <a ng-click="showFilterActions = !showFilterActions" kbn-accessible-click>
+      <a
+        ng-click="showFilterActions = !showFilterActions"
+        kbn-accessible-click
+        aria-expanded="{{!!showFilterActions}}"
+        aria-controls="filterActionsAllContainer"
+      >
         Actions
         <span
           class="fa"
@@ -111,6 +116,7 @@
 <div
   class="filter-bar filter-bar-condensed"
   ng-show="filters.length && showFilterActions"
+  id="filterActionsAllContainer"
 >
   <div class="filter-actions-all">
     <div class="filter-link">