From a8b99d39748d1dc9fa1c9ac6d0de88d8ce44b3fc Mon Sep 17 00:00:00 2001
From: CJ Cenizal <cj@cenizal.com>
Date: Thu, 11 May 2017 12:46:04 -0700
Subject: [PATCH] Improve accessibility by adding alternative text for
 screen-readers. (#11548)

* Ensure img elements have alt text where necessary (none do).
- Add aria-hidden to global nav icon.

* Remove unused title attribute from SavedObjectFinder.
---
 .../kibana/public/dashboard/top_nav/add_panel.html         | 7 +++++--
 .../kibana/public/visualize/wizard/step_2.html             | 1 -
 .../global_nav/global_nav_link/global_nav_link.html        | 3 +--
 src/ui/public/directives/saved_object_finder.js            | 1 -
 4 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/core_plugins/kibana/public/dashboard/top_nav/add_panel.html b/src/core_plugins/kibana/public/dashboard/top_nav/add_panel.html
index 43ed95963b899..7888e7cae886b 100644
--- a/src/core_plugins/kibana/public/dashboard/top_nav/add_panel.html
+++ b/src/core_plugins/kibana/public/dashboard/top_nav/add_panel.html
@@ -25,7 +25,6 @@
 
   <div class="list-group-item list-group-item--noBorder" ng-switch-when="visualization">
     <saved-object-finder
-      title="Choose a visualization"
       use-local-management="true"
       type="visualizations"
       on-add-new="opts.addNewVis"
@@ -34,6 +33,10 @@
   </div>
 
   <div class="list-group-item list-group-item--noBorder" ng-switch-when="search">
-    <saved-object-finder title="Choose a search" type="searches" use-local-management="true" on-choose="opts.addSearch"></saved-object-finder>
+    <saved-object-finder
+      type="searches"
+      use-local-management="true"
+      on-choose="opts.addSearch"
+    ></saved-object-finder>
   </div>
 </div>
diff --git a/src/core_plugins/kibana/public/visualize/wizard/step_2.html b/src/core_plugins/kibana/public/visualize/wizard/step_2.html
index 64d06798f8742..0d6b224e440ad 100644
--- a/src/core_plugins/kibana/public/visualize/wizard/step_2.html
+++ b/src/core_plugins/kibana/public/visualize/wizard/step_2.html
@@ -34,7 +34,6 @@ <h2 class="kuiTitle kuiVerticalRhythm">
 
       <!-- Saved searches -->
       <saved-object-finder
-        title="Saved Searches"
         type="searches"
         class="wizard-row visualizeWizardSavedObjectFinder kuiVerticalRhythm"
         make-url="step2WithSearchUrl"
diff --git a/src/ui/public/chrome/directives/global_nav/global_nav_link/global_nav_link.html b/src/ui/public/chrome/directives/global_nav/global_nav_link/global_nav_link.html
index 887ba6adae57e..c7091d3386d5b 100644
--- a/src/ui/public/chrome/directives/global_nav/global_nav_link/global_nav_link.html
+++ b/src/ui/public/chrome/directives/global_nav/global_nav_link/global_nav_link.html
@@ -13,12 +13,11 @@
     data-test-subj="appLink"
   >
     <div class="global-nav-link__icon">
-      <!-- Empty alt attribute is for accessibility -->
       <img
         ng-if="icon"
         class="global-nav-link__icon-image"
         kbn-src="{{ '/' + icon }}"
-        alt=""
+        aria-hidden
       >
 
       <span
diff --git a/src/ui/public/directives/saved_object_finder.js b/src/ui/public/directives/saved_object_finder.js
index ddb50664fd59b..ef1892641a1a8 100644
--- a/src/ui/public/directives/saved_object_finder.js
+++ b/src/ui/public/directives/saved_object_finder.js
@@ -15,7 +15,6 @@ module.directive('savedObjectFinder', function ($location, $injector, kbnUrl, Pr
     restrict: 'E',
     scope: {
       type: '@',
-      title: '@?',
       // optional make-url attr, sets the userMakeUrl in our scope
       userMakeUrl: '=?makeUrl',
       // optional on-choose attr, sets the userOnChoose in our scope