Use arrow functions to preserve this in timers

alphagov · Jul 20, 2023 · 268d1da · 268d1da
1 parent c2d402c
commit 268d1da
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/src/javascripts/components/copy.mjs b/src/javascripts/components/copy.mjs
@@ -27,7 +27,7 @@ Copy.prototype.copyAction = function () {
     }).on('success', function (e) {
       e.trigger.textContent = 'Code copied'
       e.clearSelection()
-      setTimeout(function () {
+      setTimeout(() => {
         e.trigger.textContent = 'Copy code'
       }, 5000)
     })

diff --git a/src/javascripts/components/options-table.mjs b/src/javascripts/components/options-table.mjs
@@ -44,7 +44,7 @@ const OptionsTable = {
             }
             detailsText.style.display = ''
 
-            window.setTimeout(function () {
+            window.setTimeout(() => {
               tabLink.focus()
               if (isLinkToTable) document.querySelector(hash).scrollIntoView()
             }, 0)

diff --git a/src/javascripts/components/search.mjs b/src/javascripts/components/search.mjs
@@ -76,7 +76,7 @@ Search.prototype.handleSearchQuery = function (query, callback) {
   searchCallback = callback
 
   clearTimeout(inputDebounceTimer)
-  inputDebounceTimer = setTimeout(function () {
+  inputDebounceTimer = setTimeout(() => {
     trackSearchResults(searchQuery, searchResults)
   }, DEBOUNCE_TIME_TO_WAIT())