modified slot.js to allow for collection drag and drop #956
Conversation
…s user to manifest list view
| //TODO: | ||
| //this works; | ||
| //but you might want to check if some "publish" action would be better | ||
| _this.addItem(); |
There was a problem hiding this comment.
Good point, but I think this will be okay for now. We need to think about how the different top-level screens can be represented in the data structure, and how to update them accordingly without calling the methods of things.
Any update needs to ultimately be done through an "action", so that the state manager can send out notifications to subscribers(UI components) that depend on that state.
|
This seems like a good way forward using existing patterns. Although the coverage has decreased, I'm adding going ahead with it since the slot.js d-n-d code needs some serious refactoring to be testable. The different branches seriously need to be broken out into functions. |
|
Hooray!!! On Wed, Jun 15, 2016 at 3:19 PM, aeschylus notifications@github.com wrote:
Dr. Jeffrey C. Witt |
modified slot.js to allow for collection drag and drop that then moves user to manifest list view.
Should resolve issue: #941