Eric Montellese comments

Results 7 comments of


                                            Eric Montellese

Logic error in `below()` etc. after updating `order_with_respect_to` fields

I actually also just ran into this - I think that you are right.

Logic error in `below()` etc. after updating `order_with_respect_to` fields

Actually, i think that 'return' may not be right regardless. If '>' work needs to be done If '

Logic error in `below()` etc. after updating `order_with_respect_to` fields

Also it looks like the 'to()' function has some issues in some cases. I'll put together a pull request.

Logic error in `below()` etc. after updating `order_with_respect_to` fields

Please see #93 for a proposed solution to this and a related issue that I was having in the case where an ordered item is dragged from one list to...

Fix for above(), below() and to() to handle certain edge cases.

Looks like the tests fail for this so there's more work to do -- however, I invite your comments before I dive in too deeply.

Fix for above(), below() and to() to handle certain edge cases.

In my opinion, I think this new model is fine -- the order is preserved. (in other words, I think the test is too picky). The actual order values are...

Fix for above(), below() and to() to handle certain edge cases.

I think the test failures are only showing that there are gaps -- but is it required for there not to be in order for the ordered-model to be working?...