From f45d1d53fb06468c4eeba2644a7c00eb577b9259 Mon Sep 17 00:00:00 2001
From: Nolan Lawson <nolan@nolanlawson.com>
Date: Thu, 15 Feb 2018 22:58:18 -0800
Subject: [PATCH] fix aria-label of notification count

---
 routes/_components/NavItem.html | 18 ++++++++++++++----
 1 file changed, 14 insertions(+), 4 deletions(-)

diff --git a/routes/_components/NavItem.html b/routes/_components/NavItem.html
index 395bb46c..d5bde53d 100644
--- a/routes/_components/NavItem.html
+++ b/routes/_components/NavItem.html
@@ -1,7 +1,7 @@
-<a class='main-nav-link {{forceCurrent || page === name ? "selected" : ""}}'
-   aria-label='{{forceCurrent || page === name ? `${label} (current page)` : label}}'
-   aria-current="{{forceCurrent || page === name}}"
-   href='{{href}}'>
+<a class='main-nav-link {{page === name ? "selected" : ""}}'
+   aria-label='{{ariaLabel}}'
+   aria-current="{{page === name}}"
+   :href >
   {{#if name === 'notifications'}}
     <div class="nav-link-svg-wrapper">
       <svg>
@@ -115,6 +115,16 @@
   export default {
     store: () => store,
     computed: {
+      ariaLabel: (page, name, label, numberOfNotifications) => {
+        let res = label
+        if (page === name) {
+          res += ' (current page)'
+        }
+        if (name === 'notifications') {
+          res += ` (${numberOfNotifications})`
+        }
+        return res
+      },
       numberOfNotifications: ($timelines, $currentInstance, $currentTimeline) => {
         return $currentTimeline !== 'notifications' &&
           $timelines &&