From 42ac46fa28edb0d91a02b64269440a7e34b59ad6 Mon Sep 17 00:00:00 2001
From: Michael Winkworth <mwinkwo@btinternet.com>
Date: Sat, 9 Sep 2023 00:03:04 +0100
Subject: [PATCH] Add ability to tab to DuplicateLink (#2646)

* Add ability to tab to DuplicateLink

The change that allows the DuplicateLink to be tabbed to is the removal of the `tabindex`. The other changes are to make the link only appear as wide as the text, otherwise the tab outline appears around the whole `<span>`, which makes it look weird.

* Fix formatting after attribute removal
---
 CONTRIBUTORS                   |  1 +
 ts/editor/DuplicateLink.svelte | 20 ++++++++++----------
 2 files changed, 11 insertions(+), 10 deletions(-)

diff --git a/CONTRIBUTORS b/CONTRIBUTORS
index 78573dc7e..c2f66a772 100644
--- a/CONTRIBUTORS
+++ b/CONTRIBUTORS
@@ -136,6 +136,7 @@ Marko Juhanne <github.com/mjuhanne>
 Gabriel Heinatz <anorot@gmail.com>
 Monty Evans <montyevans@gmail.com>
 Nil Admirari <https://github.com/nihil-admirari>
+Michael Winkworth <github.com/SteelColossus>
 
 ********************
 
diff --git a/ts/editor/DuplicateLink.svelte b/ts/editor/DuplicateLink.svelte
index 02185effc..d4bc2151a 100644
--- a/ts/editor/DuplicateLink.svelte
+++ b/ts/editor/DuplicateLink.svelte
@@ -7,19 +7,19 @@ License: GNU AGPL, version 3 or later; http://www.gnu.org/licenses/agpl.html
     import * as tr from "@tslib/ftl";
 </script>
 
-<a
-    class="duplicate-link"
-    href="/#"
-    tabindex="-1"
-    on:click={() => bridgeCommand("dupes")}
->
-    {tr.editingShowDuplicates()}
-</a>
+<span class="duplicate-link-container">
+    <a class="duplicate-link" href="/#" on:click={() => bridgeCommand("dupes")}>
+        {tr.editingShowDuplicates()}
+    </a>
+</span>
 
 <style lang="scss">
-    .duplicate-link {
-        color: var(--highlight-color);
+    .duplicate-link-container {
         text-align: center;
         flex-grow: 1;
     }
+
+    .duplicate-link {
+        color: var(--highlight-color);
+    }
 </style>