Open chrysn opened 7 years ago
Thanks for your PR!
How about using the handle
option, that does more or less the same?
Also, in case that you work with transcluded directives, take a look at the Integrating with directives doing transclusion section of README.
Any feedback on this?
there will be as soon as i'm back at my workstation at the start of the week.
ad handle: I do use handle already, but that seems unrelated to me. ad example: I've stripped down the example to https://codepen.io/anon/pen/zzEbbe to demonstrate the issue. the ng-repeat is on an outer div (that's not styled by itself), and only an inner div is styled as float; it has a distinct handle.
Can you add the a set of curly in your while
statement to fix the linter errors and see whether this breaks any tests?
Can you share some details about why the sorted items can't be under the same parent?
For now, the official way to achieve such effects would be by using the helper
and placeholder
options.
Updated with braces; commits not smushed for better review. I don't know how to interpret the current test failure.
ad "why not same parent": The items are not under the same parent because the angular directives that provide the (conditionally floating) items leave their own tags inbetween.
I'm afraid that this might break things added on v0.17 that added support for transcluded directives. That's a really strange error! Does it also throw the same error locally?
I get the same error after I pulled in the whole test suite.
Is there any way to run the test suite in a more direct way than encapsulated in karma, to drill down on what makes things go wrong?
The sortable container might not be the item's direct parent; this happens eg. in floating setups when the ng-repeated DOM node is a directive that only includes the floating node as a child, so the resulting tree with directives and CSS resolved looks like
I'm unsure on why the .find(opts['ui-model-items']) -> .find(opts['items']) is necessary exactly (or why it doesn't say the latter originally), but it is required in the same situation; were I to (just on a guess) set {ui-model-items:'.tilefloater', items:'.tilefloater'}, there'd be leftover placeholders, and without the change, .find() doesn't catch the actually moved item.