Merge pull request #11 from lukepighetti/lukepighetti/disable-autoscroll

LGTM, thanks

lib/src/reorderable_grid.dart

@@ -52,6 +52,7 @@ class ReorderableGrid extends StatefulWidget {
     this.keyboardDismissBehavior = ScrollViewKeyboardDismissBehavior.manual,
     this.restorationId,
     this.clipBehavior = Clip.hardEdge,
+    this.autoScroll,
   })  : assert(itemCount >= 0),
         super(key: key);
 
@@ -121,6 +122,10 @@ class ReorderableGrid extends StatefulWidget {
 
   final SliverGridDelegate gridDelegate;
 
+  /// Overrides if autoscrolling is enabled. Defaults to false if `physics` is
+  /// [NeverScrollableScrollPhysics]
+  final bool? autoScroll;
+
   /// The state from the closest instance of this class that encloses the given
   /// context.
   ///
@@ -258,6 +263,8 @@ class ReorderableGridState extends State<ReorderableGrid> {
             itemCount: widget.itemCount,
             onReorder: widget.onReorder,
             proxyDecorator: widget.proxyDecorator,
+            autoScroll: widget.autoScroll ??
+                widget.physics is! NeverScrollableScrollPhysics,
           ),
         ),
       ],
@@ -297,6 +304,7 @@ class SliverReorderableGrid extends StatefulWidget {
     required this.onReorder,
     required this.gridDelegate,
     this.proxyDecorator,
+    this.autoScroll = true,
   })  : assert(itemCount >= 0),
         super(key: key);
 
@@ -314,6 +322,10 @@ class SliverReorderableGrid extends StatefulWidget {
 
   final SliverGridDelegate gridDelegate;
 
+  /// If auto scrolling is enabled. Should be disabled if associated scroll
+  /// physics are [NeverScrollableScrollPhysics]
+  final bool autoScroll;
+
   @override
   SliverReorderableGridState createState() => SliverReorderableGridState();
 
@@ -589,7 +601,10 @@ class SliverReorderableGridState extends State<SliverReorderableGrid>
   }
 
   Future<void> _autoScrollIfNecessary() async {
-    if (_autoScrolling || _dragInfo == null || _dragInfo!.scrollable == null) {
+    if (_autoScrolling ||
+        _dragInfo == null ||
+        _dragInfo!.scrollable == null ||
+        widget.autoScroll == false) {
       return;
     }
 

lib/src/reorderable_grid_view.dart

@@ -255,6 +255,7 @@ class ReorderableGridView extends StatefulWidget {
     this.scrollController,
     this.anchor = 0.0,
     this.proxyDecorator,
+    this.autoScroll,
   })  : assert(
           children.every((Widget w) => w.key != null),
           'All children of this widget must have a key.',
@@ -304,6 +305,7 @@ class ReorderableGridView extends StatefulWidget {
     this.scrollController,
     this.anchor = 0.0,
     this.proxyDecorator,
+    this.autoScroll,
   })  : assert(itemCount >= 0),
         super(key: key);
 
@@ -345,6 +347,7 @@ class ReorderableGridView extends StatefulWidget {
     this.scrollController,
     this.anchor = 0.0,
     this.proxyDecorator,
+    this.autoScroll,
   })  : gridDelegate = SliverGridDelegateWithFixedCrossAxisCount(
           crossAxisCount: crossAxisCount,
           mainAxisSpacing: mainAxisSpacing,
@@ -397,6 +400,7 @@ class ReorderableGridView extends StatefulWidget {
     this.scrollController,
     this.anchor = 0.0,
     this.proxyDecorator,
+    this.autoScroll,
   })  : gridDelegate = SliverGridDelegateWithMaxCrossAxisExtent(
           maxCrossAxisExtent: maxCrossAxisExtent,
           mainAxisSpacing: mainAxisSpacing,
@@ -477,6 +481,10 @@ class ReorderableGridView extends StatefulWidget {
   /// {@macro flutter.widgets.reorderable_list.proxyDecorator}
   final ReorderItemProxyDecorator? proxyDecorator;
 
+  /// Overrides if autoscrolling is enabled. Defaults to false if `physics` is
+  /// [NeverScrollableScrollPhysics]
+  final bool? autoScroll;
+
   @override
   _ReorderableGridViewState createState() => _ReorderableGridViewState();
 }
@@ -625,6 +633,8 @@ class _ReorderableGridViewState extends State<ReorderableGridView> {
             itemCount: widget.itemCount,
             onReorder: widget.onReorder,
             proxyDecorator: widget.proxyDecorator ?? _proxyDecorator,
+            autoScroll: widget.autoScroll ??
+                widget.physics is! NeverScrollableScrollPhysics,
           ),
         ),
       ],