aboutsummaryrefslogtreecommitdiffstats
path: root/ui/widgets
diff options
context:
space:
mode:
authorMichał Gołębiowski-Owczarek <m.goleb@gmail.com>2021-11-08 18:21:03 +0100
committerGitHub <noreply@github.com>2021-11-08 18:21:03 +0100
commitefe3b225179f8f2f3f2c05484f7757f90bc5a27b (patch)
tree95b0fa80529db6830cbf6542d99039189157b0c9 /ui/widgets
parent85fba3f107a4a03bdac43f06c81ab2f2a3c2fea5 (diff)
downloadjquery-ui-efe3b225179f8f2f3f2c05484f7757f90bc5a27b.tar.gz
jquery-ui-efe3b225179f8f2f3f2c05484f7757f90bc5a27b.zip
Sortable: Allow 0-height containers to be sortable as in 1.12.1
Note that container specific events will not fire when the dragged element is interacting with zero height containers. Fixes gh-1998 Closes gh-2008 Co-authored-by: A. Wells <borgboyone@users.noreply.github.com>
Diffstat (limited to 'ui/widgets')
-rw-r--r--ui/widgets/sortable.js121
1 files changed, 57 insertions, 64 deletions
diff --git a/ui/widgets/sortable.js b/ui/widgets/sortable.js
index f44b4832a..37ce4b7bb 100644
--- a/ui/widgets/sortable.js
+++ b/ui/widgets/sortable.js
@@ -421,79 +421,76 @@ return $.widget( "ui.sortable", $.ui.mouse, {
this.helper[ 0 ].style.top = this.position.top + "px";
}
- //Post events to containers
- this._contactContainers( event );
-
- if ( this.innermostContainer !== null ) {
-
- //Do scrolling
- if ( o.scroll ) {
- if ( this._scroll( event ) !== false ) {
+ //Do scrolling
+ if ( o.scroll ) {
+ if ( this._scroll( event ) !== false ) {
- //Update item positions used in position checks
- this._refreshItemPositions( true );
+ //Update item positions used in position checks
+ this._refreshItemPositions( true );
- if ( $.ui.ddmanager && !o.dropBehaviour ) {
- $.ui.ddmanager.prepareOffsets( this, event );
- }
+ if ( $.ui.ddmanager && !o.dropBehaviour ) {
+ $.ui.ddmanager.prepareOffsets( this, event );
}
}
+ }
- this.dragDirection = {
- vertical: this._getDragVerticalDirection(),
- horizontal: this._getDragHorizontalDirection()
- };
-
- //Rearrange
- for ( i = this.items.length - 1; i >= 0; i-- ) {
-
- //Cache variables and intersection, continue if no intersection
- item = this.items[ i ];
- itemElement = item.item[ 0 ];
- intersection = this._intersectsWithPointer( item );
- if ( !intersection ) {
- continue;
- }
-
- // Only put the placeholder inside the current Container, skip all
- // items from other containers. This works because when moving
- // an item from one container to another the
- // currentContainer is switched before the placeholder is moved.
- //
- // Without this, moving items in "sub-sortables" can cause
- // the placeholder to jitter between the outer and inner container.
- if ( item.instance !== this.currentContainer ) {
- continue;
- }
+ this.dragDirection = {
+ vertical: this._getDragVerticalDirection(),
+ horizontal: this._getDragHorizontalDirection()
+ };
- // Cannot intersect with itself
- // no useless actions that have been done before
- // no action if the item moved is the parent of the item checked
- if ( itemElement !== this.currentItem[ 0 ] &&
- this.placeholder[ intersection === 1 ?
- "next" : "prev" ]()[ 0 ] !== itemElement &&
- !$.contains( this.placeholder[ 0 ], itemElement ) &&
- ( this.options.type === "semi-dynamic" ?
- !$.contains( this.element[ 0 ], itemElement ) :
- true
- )
- ) {
+ //Rearrange
+ for ( i = this.items.length - 1; i >= 0; i-- ) {
- this.direction = intersection === 1 ? "down" : "up";
+ //Cache variables and intersection, continue if no intersection
+ item = this.items[ i ];
+ itemElement = item.item[ 0 ];
+ intersection = this._intersectsWithPointer( item );
+ if ( !intersection ) {
+ continue;
+ }
- if ( this.options.tolerance === "pointer" ||
- this._intersectsWithSides( item ) ) {
- this._rearrange( event, item );
- } else {
- break;
- }
+ // Only put the placeholder inside the current Container, skip all
+ // items from other containers. This works because when moving
+ // an item from one container to another the
+ // currentContainer is switched before the placeholder is moved.
+ //
+ // Without this, moving items in "sub-sortables" can cause
+ // the placeholder to jitter between the outer and inner container.
+ if ( item.instance !== this.currentContainer ) {
+ continue;
+ }
- this._trigger( "change", event, this._uiHash() );
+ // Cannot intersect with itself
+ // no useless actions that have been done before
+ // no action if the item moved is the parent of the item checked
+ if ( itemElement !== this.currentItem[ 0 ] &&
+ this.placeholder[ intersection === 1 ?
+ "next" : "prev" ]()[ 0 ] !== itemElement &&
+ !$.contains( this.placeholder[ 0 ], itemElement ) &&
+ ( this.options.type === "semi-dynamic" ?
+ !$.contains( this.element[ 0 ], itemElement ) :
+ true
+ )
+ ) {
+
+ this.direction = intersection === 1 ? "down" : "up";
+
+ if ( this.options.tolerance === "pointer" ||
+ this._intersectsWithSides( item ) ) {
+ this._rearrange( event, item );
+ } else {
break;
}
+
+ this._trigger( "change", event, this._uiHash() );
+ break;
}
}
+ //Post events to containers
+ this._contactContainers( event );
+
//Interconnect with droppables
if ( $.ui.ddmanager ) {
$.ui.ddmanager.drag( this, event );
@@ -888,9 +885,7 @@ return $.widget( "ui.sortable", $.ui.mouse, {
this.options.axis === "x" || this._isFloating( this.items[ 0 ].item ) :
false;
- if ( this.innermostContainer !== null ) {
- this._refreshItemPositions( fast );
- }
+ this._refreshItemPositions( fast );
var i, p;
@@ -1038,8 +1033,6 @@ return $.widget( "ui.sortable", $.ui.mouse, {
}
- this.innermostContainer = innermostContainer;
-
// If no intersecting containers found, return
if ( !innermostContainer ) {
return;