summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPhil Hughes <me@iamphill.com>2016-08-08 08:54:58 +0100
committerPhil Hughes <me@iamphill.com>2016-08-17 17:12:47 +0100
commit6c3caa00f84cf446344d6f896d3283bc2e8e7fb6 (patch)
treef9c649c19dde215d87b8a53c1bf1f8a282cb9b31
parent11497095377a1d97024426ac2de846c6bb995a7b (diff)
downloadgitlab-ce-6c3caa00f84cf446344d6f896d3283bc2e8e7fb6.tar.gz
Fixed error when trying to drag list several times
-rw-r--r--app/assets/javascripts/boards/components/board_list.js.es61
-rw-r--r--app/assets/javascripts/boards/stores/boards_store.js.es68
2 files changed, 5 insertions, 4 deletions
diff --git a/app/assets/javascripts/boards/components/board_list.js.es6 b/app/assets/javascripts/boards/components/board_list.js.es6
index 79504999068..df64d57b5c3 100644
--- a/app/assets/javascripts/boards/components/board_list.js.es6
+++ b/app/assets/javascripts/boards/components/board_list.js.es6
@@ -36,7 +36,6 @@
},
ready: function () {
const options = _.extend({
- // sort: false,
group: 'issues',
disabled: this.disabled,
onAdd: function (e) {
diff --git a/app/assets/javascripts/boards/stores/boards_store.js.es6 b/app/assets/javascripts/boards/stores/boards_store.js.es6
index abc64aef3d5..22c84de66a9 100644
--- a/app/assets/javascripts/boards/stores/boards_store.js.es6
+++ b/app/assets/javascripts/boards/stores/boards_store.js.es6
@@ -79,7 +79,9 @@
listTo = this.findList('position', newIndex);
listFrom.position = newIndex;
- if (newIndex > listTo.position) {
+ if (newIndex === listTo.position) {
+ listTo.position = oldIndex;
+ } else if (newIndex > listTo.position) {
listTo.position--;
} else {
listTo.position++;
@@ -97,8 +99,6 @@
return issue.label;
});
- listFrom.removeIssue(issue);
-
// Add to new lists issues if it doesn't already exist
if (!issueTo) {
listTo.addIssue(issue, listFrom);
@@ -109,6 +109,8 @@
list.removeIssue(issue);
});
issue.removeLabels(listLabels);
+ } else {
+ listFrom.removeIssue(issue);
}
},
findList: function (key, val) {