Error with connected sortables and placeholders (angular1.2).

[thgreasi]

Here is a demo pen.
Just move an item back and forth to the other list.

[CyborgMaster]

So... I thought I would just play around with the pen and see if I could get things working, and I didn't realize that modifying it auto saves, so I think I may have broken your pen. Did I really break it for everyone, or is there a way to get back the original?

[CyborgMaster]

So I think I just figured out how to revert things, now I'm going to see if I can fix the problem.

[CyborgMaster]

Interesting bug. The reason it was breaking is the placeholder you had set up in this example had the same class as the elements in the list. Therefore as it was trying to remove the placeholder from the saved nodes list, it was removing the entire list. This wouldn't happen if the placeholder class was different then the class for the elements of the list.

I added a quick filter on the ng-scope class that angular adds to all the elements which fixes it and added a pull request.

[thgreasi]

Sorry didn't saw that. Juster wanted to try the code from issue #47 for regressions.

[thgreasi]

Can you also review PR #60?

[CyborgMaster]

@thgreasi I'm reviewing #60 and #65 now.

[thgreasi]

Fixed by #65.