Prefer using window.setTimeout() and window.setInterval()

Shows the compiler that we’re using DOM timers and intervals, rather than Node.js globals
alphagov · Jun 20, 2023 · 465294b · 465294b
1 parent 28d016a
commit 465294b
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/packages/govuk-frontend/src/govuk/components/button/button.mjs b/packages/govuk-frontend/src/govuk/components/button/button.mjs
@@ -102,7 +102,7 @@ export class Button {
       return false
     }
 
-    this.debounceFormSubmitTimer = setTimeout(() => {
+    this.debounceFormSubmitTimer = window.setTimeout(() => {
       this.debounceFormSubmitTimer = null
     }, DEBOUNCE_TIMEOUT_IN_SECONDS * 1000)
   }

diff --git a/packages/govuk-frontend/src/govuk/components/character-count/character-count.mjs b/packages/govuk-frontend/src/govuk/components/character-count/character-count.mjs
@@ -250,7 +250,7 @@ export class CharacterCount {
    * @private
    */
   handleFocus () {
-    this.valueChecker = setInterval(() => {
+    this.valueChecker = window.setInterval(() => {
       if (!this.lastInputTimestamp || (Date.now() - 500) >= this.lastInputTimestamp) {
         this.updateIfValueChanged()
       }