From 5cfdb1647f9f96e92fb7b177d9dc15788dfe8423 Mon Sep 17 00:00:00 2001
From: Bill Thornton <thornbill@users.noreply.github.com>
Date: Tue, 2 May 2023 12:53:20 -0400
Subject: [PATCH] Fix getter return value bugs

---
 src/components/scrollManager.js  | 2 +-
 src/legacy/focusPreventScroll.js | 2 +-
 src/scripts/dom.js               | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/components/scrollManager.js b/src/components/scrollManager.js
index 7417c7643e..52c46bcaf7 100644
--- a/src/components/scrollManager.js
+++ b/src/components/scrollManager.js
@@ -39,9 +39,9 @@ try {
     const elem = document.createElement('div');
 
     const opts = Object.defineProperty({}, 'behavior', {
-        // eslint-disable-next-line getter-return
         get: function () {
             supportsScrollToOptions = true;
+            return null;
         }
     });
 
diff --git a/src/legacy/focusPreventScroll.js b/src/legacy/focusPreventScroll.js
index 8f912207b5..6ed7d6d017 100644
--- a/src/legacy/focusPreventScroll.js
+++ b/src/legacy/focusPreventScroll.js
@@ -12,9 +12,9 @@ if (HTMLElement.prototype.nativeFocus === undefined) {
             }, true);
 
             const opts = Object.defineProperty({}, 'preventScroll', {
-                // eslint-disable-next-line getter-return
                 get: function () {
                     supportsPreventScrollOption = true;
+                    return null;
                 }
             });
 
diff --git a/src/scripts/dom.js b/src/scripts/dom.js
index 75bf68cc1f..35ff2078a6 100644
--- a/src/scripts/dom.js
+++ b/src/scripts/dom.js
@@ -87,9 +87,9 @@ export function parentWithClass(elem, classNames) {
 let supportsCaptureOption = false;
 try {
     const opts = Object.defineProperty({}, 'capture', {
-        // eslint-disable-next-line getter-return
         get: function () {
             supportsCaptureOption = true;
+            return null;
         }
     });
     window.addEventListener('test', null, opts);