Skip to content
This repository was archived by the owner on Sep 8, 2020. It is now read-only.

Commit 4674a53

Browse files
committed
Merge pull request #163 from thgreasi/master
refactor(sortable): change to achieve 100% code coverage on tests
2 parents 6fbde25 + 664e7de commit 4674a53

File tree

3 files changed

+3
-5
lines changed

3 files changed

+3
-5
lines changed

bower.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "angular-ui-sortable",
3-
"version": "0.12.3",
3+
"version": "0.12.4",
44
"description": "This directive allows you to jQueryUI Sortable.",
55
"author": "https://github.com/angular-ui/ui-sortable/graphs/contributors",
66
"license": "MIT",

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "angular-ui-sortable",
3-
"version": "0.12.3",
3+
"version": "0.12.4",
44
"description": "This directive allows you to jQueryUI Sortable.",
55
"author": "https://github.com/angular-ui/ui-sortable/graphs/contributors",
66
"license": "MIT",

src/sortable.js

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -84,9 +84,7 @@ angular.module('ui.sortable', [])
8484
var phElement = placeholder.element();
8585
// workaround for jquery ui 1.9.x,
8686
// not returning jquery collection
87-
if (!phElement.jquery) {
88-
phElement = angular.element(phElement);
89-
}
87+
phElement = angular.element(phElement);
9088

9189
// exact match with the placeholder's class attribute to handle
9290
// the case that multiple connected sortables exist and

0 commit comments

Comments
 (0)