Dominic Elm
Dominic Elm
@timdeschryver Yes, that's exactly why we didn't introduce that. I like the idea of a directive, because you could potentially register multiple scroll containers. That sounds reasonable.
@timdeschryver I think a directive is not really sufficient enough because there might be scroll containers outside of the `select-container` and I wouldn't be able to query those using the...
> [...] but won't they all be children of `dts-select-container`? Not necessarily, because you could also have scroll containers outside of the select-container no? For instance in many apps the...
Hey @bvkimball, thanks for your PR. This is actually something I was fiddling with as well at some point but it never made it into master. I like this refactoring...
Love it. Thanks a bunch!
Yep that demo is looking slick! Thanks for this. Will be a great addition. Make sure to not break existing functionality. As to the "click to select", I am not...
Let me know when this PR is ready for review and I ll look at it. Maybe you can also take the list of action items I posted earlier and...
Yep, agreed that the QueryList shouldn't have been public in the first place. If it causes a breaking change we can revert this and publish this as a new major...
Yea ok, that makes sense. Then I guess we can ignore this for now. It's fine because it's mostly related to the CDK then. Tests are failing now. I think...
So when I try the drag and drop I have this weird behavior when you move an item in the same list, then something's off with the placeholder. Would you...