Skip to content

Adds support for setting drop targets for draggables #227

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Adds support for setting drop targets for draggables #227

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
88a2146
Adds support for setting drop targets for draggables
Feb 26, 2016
67829e3
Updates README for target usage
Feb 26, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 10 additions & 0 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -45,6 +45,16 @@ Drop area usage:
Drop area
</div>
```
#### Setting Targets
You can set specific drop zone targets for individual draggable items by using
the `ng-drop-id` and `ng-drop-target` attributes:
```html
<div ng-drop="true" ng-drop-id="myDropZone_1"></div>
<div ng-drop="true" ng-drop-id="myDropZone_2"></div>

<span ng-drag="true" ng-drag-data="{obj}" ng-drop-target="myDropZone_1"></span>
<span ng-drag="true" ng-drag-data="{obj2}" ng-drop-target="myDropZone_2"></span>
```

### Angular Controller:

Expand Down
12 changes: 9 additions & 3 deletions ngDraggable.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@ angular.module("ngDraggable", [])
link: function (scope, element, attrs) {
scope.value = attrs.ngDrag;
var offset,_centerAnchor=false,_mx,_my,_tx,_ty,_mrx,_mry;
var _dropTarget = attrs.ngDropTarget;
var _hasTouch = ('ontouchstart' in window) || window.DocumentTouch && document instanceof DocumentTouch;
var _pressEvents = 'touchstart mousedown';
var _moveEvents = 'touchmove mousemove';
Expand Down Expand Up @@ -205,14 +206,14 @@ angular.module("ngDraggable", [])

moveElement(_tx, _ty);

$rootScope.$broadcast('draggable:move', { x: _mx, y: _my, tx: _tx, ty: _ty, event: evt, element: element, data: _data, uid: _myid, dragOffset: _dragOffset });
$rootScope.$broadcast('draggable:move', { x: _mx, y: _my, tx: _tx, ty: _ty, event: evt, element: element, data: _data, uid: _myid, dragOffset: _dragOffset, dropTarget: _dropTarget});
};

var onrelease = function(evt) {
if (!_dragEnabled)
return;
evt.preventDefault();
$rootScope.$broadcast('draggable:end', {x:_mx, y:_my, tx:_tx, ty:_ty, event:evt, element:element, data:_data, callback:onDragComplete, uid: _myid});
$rootScope.$broadcast('draggable:end', {x:_mx, y:_my, tx:_tx, ty:_ty, event:evt, element:element, data:_data, callback:onDragComplete, uid: _myid, dropTarget: _dropTarget});
element.removeClass('dragging');
element.parent().find('.drag-enter').removeClass('drag-enter');
reset();
Expand Down Expand Up @@ -268,6 +269,8 @@ angular.module("ngDraggable", [])
link: function (scope, element, attrs) {
scope.value = attrs.ngDrop;
scope.isTouching = false;

var _dropId = attrs.ngDropId;

var _lastDropTouch=null;

Expand Down Expand Up @@ -328,7 +331,10 @@ angular.module("ngDraggable", [])

// don't listen to drop events if this is the element being dragged
// only update the styles and return
if (!_dropEnabled || _myid === obj.uid) {
if (!_dropEnabled
|| _myid === obj.uid
// If the dropTaret and dropId don't match, don't update
|| obj.dropTarget !== _dropId) {
updateDragStyles(false, obj.element);
return;
}
Expand Down