disable autoscroll

[lukepighetti]

This PR automatically disables autoscrolling if physics are NeverScrollableScrollPhysics. It also allows the developer to override this automatic behavior using the autoScroll argument on ReorderableGrid, SliverReorderableGrid, and ReorderableGridView

Issues

Resolves #10

Code

return ReorderableGrid(
  gridDelegate: SliverGridDelegateWithFixedCrossAxisCount(
    crossAxisCount: widget.layout.crossAxisCount,
    childAspectRatio: widget.layout.childAspect,
    mainAxisSpacing: widget.layout.gap,
    crossAxisSpacing: widget.layout.gap,
  ),
  itemCount: 9,
  physics: const NeverScrollableScrollPhysics(), ///////////////////////
  itemBuilder: (context, i) {
    return ReorderableGridDragStartListener(
      key: ValueKey(i),
      index: i,
      child: Container(
        color: Colors.red,
        child: Text(i.toString()),
      ),
    );
  },
  onReorder: (i, j) {
    print('from $i to $j');
  },
);

Before

Screen.Recording.2022-06-07.at.8.14.31.AM.mov

After

Screen.Recording.2022-06-07.at.8.31.40.AM.mov

[codecov]

Codecov Report

Merging #11 (7e456ae) into master (115465d) will increase coverage by 0.27%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #11      +/-   ##
==========================================
+ Coverage   80.99%   81.26%   +0.27%     
==========================================
  Files           2        2              
  Lines         484      491       +7     
==========================================
+ Hits          392      399       +7     
  Misses         92       92              

Impacted Files	Coverage Δ
lib/src/reorderable_grid.dart	82.63% <100.00%> (+0.23%)	⬆️
lib/src/reorderable_grid_view.dart	76.57% <100.00%> (+0.42%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 115465d...7e456ae. Read the comment docs.

[casvanluijtelaar]

is there a thought process why autoScroll wasn't added to all gridview constructors?

[lukepighetti]

I intended to, which did I miss? (There was a follow up commit to try to capture them all)