Fixes crossAxisSpacing overflow in RTL (#27945)
diff --git a/packages/flutter/lib/src/rendering/sliver_grid.dart b/packages/flutter/lib/src/rendering/sliver_grid.dart index ce47807..3d23d56 100644 --- a/packages/flutter/lib/src/rendering/sliver_grid.dart +++ b/packages/flutter/lib/src/rendering/sliver_grid.dart
@@ -207,7 +207,7 @@ double _getOffsetFromStartInCrossAxis(double crossAxisStart) { if (reverseCrossAxis) - return crossAxisCount * crossAxisStride - crossAxisStart - childCrossAxisExtent; + return crossAxisCount * crossAxisStride - crossAxisStart - childCrossAxisExtent - (crossAxisStride - childCrossAxisExtent); return crossAxisStart; }
diff --git a/packages/flutter/test/widgets/grid_view_test.dart b/packages/flutter/test/widgets/grid_view_test.dart index 3938ac1..5f4ab09 100644 --- a/packages/flutter/test/widgets/grid_view_test.dart +++ b/packages/flutter/test/widgets/grid_view_test.dart
@@ -536,4 +536,32 @@ expect(tester.getTopLeft(find.byKey(target)), const Offset(600.0, 0.0)); expect(tester.getBottomRight(find.byKey(target)), const Offset(800.0, 200.0)); }); + + testWidgets('GridView crossAxisSpacing', (WidgetTester tester) async { + // Regression test for https://github.com/flutter/flutter/issues/27151. + final Key target = UniqueKey(); + + Widget build(TextDirection textDirection) { + return Directionality( + textDirection: textDirection, + child: GridView.count( + crossAxisCount: 4, + crossAxisSpacing: 8.0, + children: <Widget>[ + Container(key: target), + ], + ), + ); + } + + await tester.pumpWidget(build(TextDirection.ltr)); + + expect(tester.getTopLeft(find.byKey(target)), Offset.zero); + expect(tester.getBottomRight(find.byKey(target)), const Offset(194.0, 194.0)); + + await tester.pumpWidget(build(TextDirection.rtl)); + + expect(tester.getTopLeft(find.byKey(target)), const Offset(606.0, 0.0)); + expect(tester.getBottomRight(find.byKey(target)), const Offset(800.0, 194.0)); + }); }