From e95b5a7904ba7493d4d7683a7edce9a9d6af5e0c Mon Sep 17 00:00:00 2001
From: delucac <delucac@mskcc.org>
Date: Tue, 2 Aug 2016 14:50:53 -0400
Subject: [PATCH] Add icon titles for accessibility.

Make clear the icons purpose with a `title` tooltip.
---
 assets/embed/templates/actions.tmpl     |  8 ++++----
 assets/embed/templates/base.tmpl        | 16 ++++++++--------
 assets/embed/templates/editor.tmpl      |  2 +-
 assets/embed/templates/frontmatter.tmpl |  8 ++++----
 assets/embed/templates/listing.tmpl     |  2 +-
 5 files changed, 18 insertions(+), 18 deletions(-)

diff --git a/assets/embed/templates/actions.tmpl b/assets/embed/templates/actions.tmpl
index 635da57f..b5ed7358 100644
--- a/assets/embed/templates/actions.tmpl
+++ b/assets/embed/templates/actions.tmpl
@@ -1,16 +1,16 @@
 {{ define "actions" }}
  <div class="action" id="open">
-  <i class="material-icons">open_in_new</i> <span>See raw</span>
+  <i class="material-icons" title="See raw">open_in_new</i> <span>See raw</span>
  </div>
  {{ if .IsDir }}
   <div class="action" id="rename">
-   <i class="material-icons">mode_edit</i>
+   <i class="material-icons" title="Edit">mode_edit</i>
   </div>
  {{ end }}
  <div class="action" id="download">
-  <i class="material-icons">file_download</i> <span>Download</span>
+  <i class="material-icons" title="Download">file_download</i> <span>Download</span>
  </div>
  <div class="action" id="delete">
-  <i class="material-icons">delete</i> <span>Delete</span>
+  <i class="material-icons" title="Delete">delete</i> <span>Delete</span>
  </div>
 {{ end }}
diff --git a/assets/embed/templates/base.tmpl b/assets/embed/templates/base.tmpl
index e8f3b59d..6d24209f 100644
--- a/assets/embed/templates/base.tmpl
+++ b/assets/embed/templates/base.tmpl
@@ -21,14 +21,14 @@
      <a href="{{ $lnk }}">
      {{ end }}
      <div class="action{{ if eq $lnk ""}} disabled{{ end }}" id="prev">
-      <i class="material-icons">subdirectory_arrow_left</i>
+      <i class="material-icons" title="Previous">subdirectory_arrow_left</i>
      </div>
      {{ if ne $lnk ""}}
      </a>
     {{ end }}
 
     <div class="action" id="open-nav">
-        <i class="material-icons">menu</i>
+        <i class="material-icons" title="Menu">menu</i>
     </div>
 
     <p>
@@ -48,7 +48,7 @@
             <a href="{{ $lnk }}">
             {{ end }}
             <div class="action{{ if eq $lnk ""}} disabled{{ end }}" id="prev">
-             <i class="material-icons">subdirectory_arrow_left</i>
+             <i class="material-icons" title="Previous">subdirectory_arrow_left</i>
             </div>
             {{ if ne $lnk ""}}
             </a>
@@ -64,23 +64,23 @@
        {{ if .IsDir}}
 
        <div id="search">
-           <i class="material-icons">storage</i>
+           <i class="material-icons" title="Storage">storage</i>
            <input type="text" placeholder="Execute a command...">
            <div>Write your git, mercurial or svn command and press enter.</div>
        </div>
 
         <div class="action" id="view">
-         <i class="material-icons">view_headline</i> <span>Switch view</span>
+         <i class="material-icons" title="Switch view">view_headline</i> <span>Switch view</span>
         </div>
         <div class="action" id="upload">
-         <i class="material-icons">file_upload</i> <span>Upload</span>
+         <i class="material-icons" title="Upload">file_upload</i> <span>Upload</span>
         </div>
         {{ else }}
         {{ template "actions" . }}
        {{ end }}
 
        <div class="action" id="logout">
-        <i class="material-icons">exit_to_app</i> <span>Logout</span>
+        <i class="material-icons" title="Logout">exit_to_app</i> <span>Logout</span>
        </div>
    </div>
    <div id="overlay"></div>
@@ -90,7 +90,7 @@
    <div id="toolbar">
     <div>
      <div class="action" id="back">
-      <i class="material-icons">arrow_back</i>
+      <i class="material-icons" title="Back">arrow_back</i>
      </div>
      <p>
       <span id="selected-number">0</span>
diff --git a/assets/embed/templates/editor.tmpl b/assets/embed/templates/editor.tmpl
index ecf8774f..121fa369 100644
--- a/assets/embed/templates/editor.tmpl
+++ b/assets/embed/templates/editor.tmpl
@@ -22,7 +22,7 @@
    <div>
     <button id="submit" type="submit" data-type="{{ .Class }}">
      <span>
-      <i class="material-icons">save</i>
+      <i class="material-icons" title="Save">save</i>
      </span>
      <span>save</span>
     </button>
diff --git a/assets/embed/templates/frontmatter.tmpl b/assets/embed/templates/frontmatter.tmpl
index 59c8d0ab..2eba94a1 100644
--- a/assets/embed/templates/frontmatter.tmpl
+++ b/assets/embed/templates/frontmatter.tmpl
@@ -7,7 +7,7 @@
     <div id="{{ $value.Name }}-{{ $key }}" data-type="array-item">
      {{ template "value" $value }}
      <div class="action delete" data-delete="{{ $value.Name }}-{{ $key }}">
-      <i class="material-icons">close</i>
+      <i class="material-icons" title="Close">close</i>
      </div>
     </div>
     {{ else }}
@@ -15,7 +15,7 @@
      <label for="{{ $value.Name }}">{{ $value.Title }}</label>
      {{ template "value" $value }}
      <div class="action delete" data-delete="block-{{ $value.Name }}">
-      <i class="material-icons">close</i>
+      <i class="material-icons" title="Close">close</i>
      </div>
     </div>
    {{ end }}
@@ -50,10 +50,10 @@
    <h3>{{ .Title }}</h3>
   {{ end }}
   <div class="action add">
-   <i class="material-icons">add</i>
+   <i class="material-icons" title="Add">add</i>
   </div>
   <div class="action delete" data-delete="{{ .Name }}">
-   <i class="material-icons">close</i>
+   <i class="material-icons" title="Close">close</i>
   </div>
   {{ template "blocks" .Content }}
  </fieldset>
diff --git a/assets/embed/templates/listing.tmpl b/assets/embed/templates/listing.tmpl
index 6f19f6c6..b7dc7aa0 100644
--- a/assets/embed/templates/listing.tmpl
+++ b/assets/embed/templates/listing.tmpl
@@ -36,7 +36,7 @@
 
   <div class="floating">
    <div class="action" id="new">
-    <i class="material-icons">add</i>
+    <i class="material-icons" title="New">add</i>
    </div>
   </div>
  {{ end }}