Git Product home page Git Product logo

Comments (9)

dgavey avatar dgavey commented on September 24, 2024

Hey, so glad to hear you got some additions ready to be merged. I appreciate any additions.

You don't need collaborator access to submit a PR. Once you fork the project to your own repo, you can apply the changes you want merged there. Then in github you can submit the PR from there.

I will take a look at them and make sure the tests pass. Speaking of tests, I would love to have some tests made for the additions you're adding. If you find time to do that, that would be appreciated.

Here is an article about how to make a PR if my instructions aren't clear enough.

https://yangsu.github.io/pull-request-tutorial/

from ember-drag-drop.

Bushjumper avatar Bushjumper commented on September 24, 2024

Thanks for the clarification - I've now created the PR on a fork.

#34

I had a failing test locally but I believe that was failing when I originally cloned the repo. I will look into it.

I would like to add new tests - I'll do that next. Would be great to get any feedback, ideas for improvement or ways to simplify if anything jumps out at you. It's a first-pass but should hopefully be useful to others, or at least on its way to that.

from ember-drag-drop.

Bushjumper avatar Bushjumper commented on September 24, 2024

Ah, I had a failing test on master but only when running them in the browser. My PR does have a legit failing test which I will need to fix

from ember-drag-drop.

linearza avatar linearza commented on September 24, 2024

I did some styling work for your addon examples. Also did a bit of a refactor on the code generally. I also fixed the issues that caused the above tests to fail intermittently. Once @Bushjumper has reviewed it and merged it into his branch it will be available on the open pr: #34

I would like to suggest a look at integrating a gesture library like ember-gestures into this addon to handle the events and provide better mobile support. Currently its pretty tightly coupled with events like mousedown which isnt ideal for mobile environments.

from ember-drag-drop.

 avatar commented on September 24, 2024

As I am currently working on a project containing multiple nested sortable objects, I was wondering if there is any chance to include this feature in the near future? All the best and thanks a lot for the great work!

from ember-drag-drop.

dgavey avatar dgavey commented on September 24, 2024

We are on the way to it. Currently on the master branch but not released is
the ability to drag something from one container to another. Nesting will
be the next step.

On Aug 17, 2016 6:18 AM, "OlmoDalco" [email protected] wrote:

As I am currently working on a project containing multiple nested sortable
objects, I was wondering if there is any chance to include this feature in
the near future? All the best and thanks a lot for the great work!


You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#33 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAuHhZ4erjcKL4IDIUAvjwsi943tjq4Kks5qgvwtgaJpZM4H5hyA
.

from ember-drag-drop.

sirvine avatar sirvine commented on September 24, 2024

In my current use case, I've got a parent Ember object that contains N sortable objects, which in turn each contain N draggable objects. When a draggable object is moved, I'm walking the entire parent Ember object and updating the contents and positions of each child Ember object on my backend at the end of each drop. It works for now, but it won't scale well. It would be better if I could isolate the updates on the backend to the specific objects (sortable and draggable) that were affected.

I favor the approach of allowing the user to assign an arbitrary, unique payload (for example, the id of the parent Ember object) to each sortable object, and reporting the fromSortable, toSortable, and draggableDropped to the drop handler. Then the drop event handler can manage the updates by looking up the affected Ember objects.

I began to throw something together along these lines for my own purposes, but I wasn't satisfied with the results. Maybe you already have plans in this direction, or a better API in mind?

from ember-drag-drop.

arinhouck avatar arinhouck commented on September 24, 2024

What is the status on this? Does this include having draggable objects that can be nested? I am currently experiencing issues with having a draggable-object inside another draggable-object. I cannot drag the inner object, it immediately becomes draggable and then stays opacity .5 and loses the context of the mouse dragging the object. @dgavey

from ember-drag-drop.

dgavey avatar dgavey commented on September 24, 2024

We have basic support for multiple sortable containers and the ability to move items between them. We still don't have the ability to support nested folders.

from ember-drag-drop.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.