diff --git a/lib/core/utils/get-scroll.js b/lib/core/utils/get-scroll.js index a0702da148..8782aa9cf3 100644 --- a/lib/core/utils/get-scroll.js +++ b/lib/core/utils/get-scroll.js @@ -19,8 +19,12 @@ axe.utils.getScroll = function getScroll(elm, buffer = 0) { } const style = window.getComputedStyle(elm); - const scrollableX = style.getPropertyValue('overflow-x') !== 'visible'; - const scrollableY = style.getPropertyValue('overflow-y') !== 'visible'; + const overflowXStyle = style.getPropertyValue('overflow-x'); + const overflowYStyle = style.getPropertyValue('overflow-y'); + const scrollableX = + overflowXStyle !== 'visible' && overflowXStyle !== 'hidden'; + const scrollableY = + overflowYStyle !== 'visible' && overflowYStyle !== 'hidden'; /** * check direction of `overflow` and `scrollable` diff --git a/test/core/utils/get-scroll.js b/test/core/utils/get-scroll.js index 64b887bc9b..72065b0434 100644 --- a/test/core/utils/get-scroll.js +++ b/test/core/utils/get-scroll.js @@ -42,10 +42,7 @@ describe('axe.utils.getScroll', function() { '' ); var actual = axe.utils.getScroll(target.actualNode); - assert.isDefined(actual); - assert.hasAllKeys(actual, ['elm', 'top', 'left']); - assert.equal(actual.top, 0); - assert.equal(actual.left, 0); + assert.isUndefined(actual); }); it('returns scroll offset when element overflow is auto', function() { diff --git a/test/core/utils/scroll-state.js b/test/core/utils/scroll-state.js index a8cf6ed1d4..8df4d6700c 100644 --- a/test/core/utils/scroll-state.js +++ b/test/core/utils/scroll-state.js @@ -53,7 +53,7 @@ describe('axe.utils.getScrollState', function() { fixture.innerHTML = '
Content
Content
-Content
Content
+