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

Conversation

benmeiri
Copy link
Contributor

@benmeiri benmeiri commented Jun 1, 2014

When using sortable between two lists and a helper function (not 'clone') there is a problem if you drop an item at the last spot.
The first time it works, the second time it will add it as the first element of the list.
It continues this way for every even drop as the last item.
You can check it here:
http://codepen.io/anon/pen/FyaJs

Closes #186

When using sortable between two lists and a helper function (not 'clone') there is a problem if you drop an item at the last spot.
The first time it works, the second time it will add it as the first element of the list.
It continues this way for every even drop as the last item.
You can check it here:
http://codepen.io/anon/pen/FyaJs

See issue here:
angular-ui#186
@thgreasi
Copy link
Contributor

thgreasi commented Jun 1, 2014

Thanks for the great job!
I will write a test case and merge it into master tomorrow.

@thgreasi
Copy link
Contributor

thgreasi commented Jun 3, 2014

I currently face problems while trying to create a test case for this to be merged. Simulating a drag to the end of a list seems "hard".

@benmeiri
Copy link
Contributor Author

benmeiri commented Jun 4, 2014

No way that it will be committed without a test?

On Tue, Jun 3, 2014 at 7:16 PM, Thodoris Greasidis <notifications@github.com

wrote:

I currently face problems while trying to create a test case for this to
be merged. Simulating a drag to the end of a list seems "hard".


Reply to this email directly or view it on GitHub
#187 (comment)
.

@thgreasi
Copy link
Contributor

thgreasi commented Jun 4, 2014

Since I have spent over 3 hours without any progress on cross-list last-place sorting, I might merge it during the weekend (that nobody is around).
:-)

@thgreasi thgreasi added this to the v0.12.7 milestone Jun 5, 2014
@thgreasi thgreasi changed the title Fixed bug fix(sortable): fix connected lists last place helper issue Jun 5, 2014
@thgreasi thgreasi changed the title fix(sortable): fix connected lists last place helper issue fix(sortable): fix connected lists with helper for last spot drops Jun 5, 2014
thgreasi added a commit that referenced this pull request Jun 5, 2014
fix(sortable): fix connected lists with helper for last spot drops
@thgreasi thgreasi merged commit 382c88e into angular-ui:master Jun 5, 2014
@thgreasi
Copy link
Contributor

thgreasi commented Jun 5, 2014

I justed merged this PR and opened #197 to track the lack of tests.

@thgreasi thgreasi mentioned this pull request Jun 5, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Helper function and connection lists bug
2 participants