Skip to content

Commit 56d6ddc

Browse files
committed
Draggable: Fix options tests
These failed when running through tests/unit/all.html, due to the smaller iframe. Reducing a browser window enough triggered the same issue. Making the top offset much smaller fixes that. The rest is fixing bad indent and adds a missing module call.
1 parent 800d76a commit 56d6ddc

File tree

2 files changed

+14
-12
lines changed

2 files changed

+14
-12
lines changed

tests/unit/draggable/draggable.html

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@
4848
}
4949
.sortable {
5050
position: relative;
51-
top: 8000px;
51+
top: 800px;
5252
left: 10px;
5353
width: 300px;
5454
padding: 0;
@@ -81,17 +81,17 @@
8181
<div style="width: 1px; height: 1000px;"></div>
8282
<div style="position: absolute; width: 1px; height: 2000px;"></div>
8383
<ul id="sortable" class="sortable">
84-
<li id="draggableSortable">Item 0</li>
85-
<li id="draggableSortable2">Item 1</li>
86-
<li>Item 2</li>
87-
<li>Item 3</li>
88-
</ul>
89-
<ul id="sortable2" class="sortable">
90-
<li id="draggableSortableClone" class="sortable2Item">Item 0</li>
91-
<li>Item 1</li>
92-
<li>Item 2</li>
93-
<li>Item 3</li>
94-
</ul>
84+
<li id="draggableSortable">Item 0</li>
85+
<li id="draggableSortable2">Item 1</li>
86+
<li>Item 2</li>
87+
<li>Item 3</li>
88+
</ul>
89+
<ul id="sortable2" class="sortable">
90+
<li id="draggableSortableClone" class="sortable2Item">Item 0</li>
91+
<li>Item 1</li>
92+
<li>Item 2</li>
93+
<li>Item 3</li>
94+
</ul>
9595
</div>
9696

9797
</body>

tests/unit/draggable/options.js

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,8 @@ define( [
66
"ui/widgets/sortable"
77
], function( $, testHelper ) {
88

9+
module( "draggable: options" );
10+
911
// TODO: This doesn't actually test whether append happened, possibly remove
1012
test( "{ appendTo: 'parent' }, default, no clone", function() {
1113
expect( 4 );

0 commit comments

Comments
 (0)