From 39529e66c1cf6643df32e6dcbb447db49eb0c514 Mon Sep 17 00:00:00 2001
From: Andrew P Maney <amaney@usc.edu>
Date: Wed, 25 Mar 2020 02:44:24 -0700
Subject: [PATCH] Tasks: add a11y title and aria labels for Toolbar

---
 src/components/Tasks/Toolbar.tsx | 14 +++++++++-----
 1 file changed, 9 insertions(+), 5 deletions(-)

diff --git a/src/components/Tasks/Toolbar.tsx b/src/components/Tasks/Toolbar.tsx
index 115e820..40ca60b 100644
--- a/src/components/Tasks/Toolbar.tsx
+++ b/src/components/Tasks/Toolbar.tsx
@@ -108,7 +108,7 @@ export default function Toolbar(props: PropsType) {
       </Transition>
 
       <div className={classes.button}>
-        <IconButton size="small" onClick={toggleSearchField}>
+        <IconButton size="small" onClick={toggleSearchField} title={showSearchField ? 'Close' : 'Search'}>
           {showSearchField ? <CloseIcon /> : <SearchIcon />}
         </IconButton>
       </div>
@@ -116,14 +116,16 @@ export default function Toolbar(props: PropsType) {
       <div className={classes.button}>
         <IconButton
           size="small"
-          aria-label="more"
-          aria-controls="long-menu"
+          title="Sort"
+          aria-label="Sort"
+          aria-controls="sort-menu"
           aria-haspopup="true"
           onClick={(e) => setSortAnchorEl(e.currentTarget)}
         >
           <SortIcon />
         </IconButton>
         <Menu
+          id="sort-menu"
           anchorEl={sortAnchorEl}
           keepMounted
           open={!!sortAnchorEl}
@@ -140,14 +142,16 @@ export default function Toolbar(props: PropsType) {
       <div className={classes.button}>
         <IconButton
           size="small"
-          aria-label="more"
-          aria-controls="long-menu"
+          title="Options"
+          aria-label="Options"
+          aria-controls="options-menu"
           aria-haspopup="true"
           onClick={(e) => setOptionsAnchorEl(e.currentTarget)}
         >
           <MoreVertIcon />
         </IconButton>
         <Menu
+          id="options-menu"
           anchorEl={optionsAnchorEl}
           keepMounted
           open={!!optionsAnchorEl}