From af27c8b26bf9c9971776cc5fca265178c56a7f30 Mon Sep 17 00:00:00 2001
From: Nolan Lawson <nolan@nolanlawson.com>
Date: Tue, 24 Sep 2019 22:30:26 -0700
Subject: [PATCH] fix: remove relative timestamp from tab focus order (#1534)

fixes #1533
---
 src/routes/_components/status/StatusRelativeDate.html | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/src/routes/_components/status/StatusRelativeDate.html b/src/routes/_components/status/StatusRelativeDate.html
index 93932c64..26ca6971 100644
--- a/src/routes/_components/status/StatusRelativeDate.html
+++ b/src/routes/_components/status/StatusRelativeDate.html
@@ -2,6 +2,7 @@
    class="status-relative-date {isStatusInNotification ? 'status-in-notification' : '' }"
    href="/statuses/{originalStatusId}"
    rel="prefetch"
+   {tabindex}
 >
   <time datetime={createdAtDate} title={absoluteFormattedDate}
         aria-label="{timeagoFormattedDate} – click to show thread">
@@ -32,7 +33,12 @@
 <script>
   export default {
     computed: {
-      elementId: ({ uuid }) => `status-relative-date-${uuid}`
+      elementId: ({ uuid }) => `status-relative-date-${uuid}`,
+      tabindex: ({ $disableTapOnStatus }) => (
+        // If you can't tap on the entire status, then you need some way to click on it. Otherwise it's
+        // just a duplicate link in the focus order.
+        $disableTapOnStatus ? '0' : '-1'
+      )
     }
   }
 </script>