From bbf9ea0942622a40d1adafeaed7045e0cf6ff8fd Mon Sep 17 00:00:00 2001 From: Mike Sherov Date: Sat, 9 Aug 2014 23:11:16 -0400 Subject: Draggable: ignore overflow:hidden containers with scroll option While it is true that overflow:hidden elements can be scrolled programatically, this breaks user expectation. Therefore, do not scroll inside an overflow:hidden container. --- tests/unit/draggable/draggable_options.js | 14 ++++++++++++++ 1 file changed, 14 insertions(+) (limited to 'tests') diff --git a/tests/unit/draggable/draggable_options.js b/tests/unit/draggable/draggable_options.js index a94452d9a..99fd608bf 100644 --- a/tests/unit/draggable/draggable_options.js +++ b/tests/unit/draggable/draggable_options.js @@ -878,6 +878,20 @@ test( "scroll, scrollSensitivity, and scrollSpeed", function() { TestHelpers.draggable.restoreScroll( document ); }); +test( "scroll ignores containers that are overflow: hidden", function() { + expect( 2 ); + + var element = $( "#draggable1" ).draggable({ scroll: true }).appendTo( "#scrollParent" ); + + element.simulate( "drag", { + dx: 1300, + dy: 1300 + }); + + equal( $( "#scrollParent" ).scrollTop(), 0, "container doesn't scroll vertically" ); + equal( $( "#scrollParent" ).scrollLeft(), 0, "container doesn't scroll horizontally" ); +}); + test( "#6817: auto scroll goes double distance when dragging", function() { expect( 2 ); -- cgit v1.2.3