Merge pull request #4559 from thornbill/fix-getter-returns

Fix getter return value bugs

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;
         }
     });
 

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;
                 }
             });
 

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);