Fix SliverMainAxisGroup scrollOffsetCorrection #174369

manu-sncf · 2025-08-25T13:37:34Z

Pre-launch Checklist

I read the Contributor Guide and followed the process outlined there for submitting PRs.
I read the Tree Hygiene wiki page, which explains my responsibilities.
I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
I signed the CLA.
I listed at least one issue that this PR fixes in the description above.
I updated/added relevant documentation (doc comments with ///).
I added new tests to check the change I am making, or this PR is test-exempt.
I followed the breaking change policy and added Data Driven Fixes where supported.
All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Note: The Flutter team is currently trialing the use of Gemini Code Assist for GitHub. Comments from the gemini-code-assist bot should not be taken as authoritative feedback from the Flutter team. If you find its comments useful you can update your code accordingly, but if you are unsure or disagree with the feedback, please feel free to wait for a Flutter team member's review for guidance on which automated comments should be addressed.

gemini-code-assist

Code Review

This pull request fixes an issue where SliverMainAxisGroup did not correctly handle scrollOffsetCorrection from its children. The change adds logic to RenderSliverMainAxisGroup to check for and propagate this correction, which is the correct approach. A comprehensive regression test has been added to verify the fix in a scenario with a dynamically changing list. My review focuses on improving the new test's maintainability by addressing code duplication and inconsistency.

gemini-code-assist · 2025-08-25T13:38:55Z

packages/flutter/test/widgets/sliver_main_axis_group_test.dart

+  testWidgets(
+    'With SliverList can handle inaccurate scroll offset due to changes in children list',
+    (WidgetTester tester) async {
+      bool skip = true;
+      Widget buildItem(BuildContext context, int index) {
+        return !skip || index.isEven
+            ? Card(
+                child: ListTile(title: Text('item$index', style: const TextStyle(fontSize: 80))),
+              )
+            : Container();
+      }
+
+      await tester.pumpWidget(
+        MaterialApp(
+          theme: ThemeData(useMaterial3: false),
+          home: Scaffold(
+            body: CustomScrollView(
+              slivers: <Widget>[
+                SliverMainAxisGroup(
+                  slivers: <Widget>[
+                    SliverList(delegate: SliverChildBuilderDelegate(buildItem, childCount: 30)),
+                  ],
+                ),
+              ],
+            ),
+          ),
+        ),
+      );
+      // Only even items 0~12 are on the screen.
+      expect(find.text('item0'), findsOneWidget);
+      expect(find.text('item12'), findsOneWidget);
+      expect(find.text('item14'), findsNothing);
+
+      await tester.drag(find.byType(CustomScrollView), const Offset(0.0, -750.0));
+      await tester.pump();
+      // Only even items 16~28 are on the screen.
+      expect(find.text('item15'), findsNothing);
+      expect(find.text('item16'), findsOneWidget);
+      expect(find.text('item28'), findsOneWidget);
+
+      skip = false;
+      await tester.pumpWidget(
+        MaterialApp(
+          home: Scaffold(
+            body: CustomScrollView(
+              slivers: <Widget>[
+                SliverMainAxisGroup(
+                  slivers: <Widget>[
+                    SliverList(delegate: SliverChildBuilderDelegate(buildItem, childCount: 30)),
+                  ],
+                ),
+              ],
+            ),
+          ),
+        ),
+      );
+
+      // Only items 12~19 are on the screen.
+      expect(find.text('item11'), findsNothing);
+      expect(find.text('item12'), findsOneWidget);
+      expect(find.text('item19'), findsOneWidget);
+      expect(find.text('item20'), findsNothing);
+
+      await tester.drag(find.byType(CustomScrollView), const Offset(0.0, 250.0));
+      await tester.pump();
+
+      // Only items 10~16 are on the screen.
+      expect(find.text('item9'), findsNothing);
+      expect(find.text('item10'), findsOneWidget);
+      expect(find.text('item16'), findsOneWidget);
+      expect(find.text('item17'), findsNothing);
+
+      // The inaccurate scroll offset should reach zero at this point
+      await tester.drag(find.byType(CustomScrollView), const Offset(0.0, 250.0));
+      await tester.pump();
+
+      // Only items 7~13 are on the screen.
+      expect(find.text('item6'), findsNothing);
+      expect(find.text('item7'), findsOneWidget);
+      expect(find.text('item13'), findsOneWidget);
+      expect(find.text('item14'), findsNothing);
+
+      // It will be corrected as we scroll, so we have to drag multiple times.
+      await tester.drag(find.byType(CustomScrollView), const Offset(0.0, 250.0));
+      await tester.pump();
+      await tester.drag(find.byType(CustomScrollView), const Offset(0.0, 250.0));
+      await tester.pump();
+      await tester.drag(find.byType(CustomScrollView), const Offset(0.0, 250.0));
+      await tester.pump();
+
+      // Only items 0~6 are on the screen.
+      expect(find.text('item0'), findsOneWidget);
+      expect(find.text('item6'), findsOneWidget);
+      expect(find.text('item7'), findsNothing);
+    },
+  );


The MaterialApp widget is constructed twice in this test, and the two instances are inconsistent as the second one (line 1243) is missing the theme property. This could lead to subtle differences in widget sizes, making the test less reliable.¹

To improve readability, reduce duplication, and ensure consistency, consider extracting the widget creation into a local helper function.

For example:

Widget buildFrame() { return MaterialApp( theme: ThemeData(useMaterial3: false), home: Scaffold( body: CustomScrollView( slivers: <Widget>[ SliverMainAxisGroup( slivers: <Widget>[ SliverList(delegate: SliverChildBuilderDelegate(buildItem, childCount: 30)), ], ), ], ), ), ); } await tester.pumpWidget(buildFrame()); // ... skip = false; await tester.pumpWidget(buildFrame());

Style Guide References

Footnotes

Code should be readable and written correctly. Duplicated code with inconsistencies makes it harder to read, maintain, and can introduce bugs. (link) ↩

That's not a bad point. Are the two pumpWidgets sections of the test self contained too? Maybe you could split it into two separate tests.

Both calls of tester.pumpWidget in same test are necessary to update the widgets tree with skip=true/false that trigger a SliverGeometry.scrollOffsetCorrection and thus reproduce the issue.

@justinmc, , can I resolve this ? As mentioned before, both calls to tester.pumpWidget are necessary in this test. The second call uses the skip variable to modify the SliverList state, which is precisely what triggers the SliverGeometry.scrollOffsetCorrection we need to verify.

justinmc · 2025-08-26T22:09:20Z

@Piinks Tagged you for review since this is about Slivers. I also plan to take a look.

justinmc · 2025-08-26T23:05:47Z

packages/flutter/test/widgets/sliver_main_axis_group_test.dart

+      // Only even items 0~12 are on the screen.
+      expect(find.text('item0'), findsOneWidget);
+      expect(find.text('item12'), findsOneWidget);
+      expect(find.text('item14'), findsNothing);


Nit: You could loop through 0-12 if you think it would be valuable to check everything.

Hi @justinmc, thanks for your review. FYI, this test is a duplicate of "slivers_test.dart" -> "SliverList can handle inaccurate scroll offset due to changes in children list".

I modified the test with a loop, as you suggested.

Is it ok for you @justinmc ?

justinmc · 2025-08-26T23:09:03Z

packages/flutter/test/widgets/sliver_main_axis_group_test.dart

+  testWidgets(
+    'With SliverList can handle inaccurate scroll offset due to changes in children list',
+    (WidgetTester tester) async {
+      bool skip = true;
+      Widget buildItem(BuildContext context, int index) {
+        return !skip || index.isEven
+            ? Card(
+                child: ListTile(title: Text('item$index', style: const TextStyle(fontSize: 80))),
+              )
+            : Container();
+      }
+
+      await tester.pumpWidget(
+        MaterialApp(
+          theme: ThemeData(useMaterial3: false),
+          home: Scaffold(
+            body: CustomScrollView(
+              slivers: <Widget>[
+                SliverMainAxisGroup(
+                  slivers: <Widget>[
+                    SliverList(delegate: SliverChildBuilderDelegate(buildItem, childCount: 30)),
+                  ],
+                ),
+              ],
+            ),
+          ),
+        ),
+      );
+      // Only even items 0~12 are on the screen.
+      expect(find.text('item0'), findsOneWidget);
+      expect(find.text('item12'), findsOneWidget);
+      expect(find.text('item14'), findsNothing);
+
+      await tester.drag(find.byType(CustomScrollView), const Offset(0.0, -750.0));
+      await tester.pump();
+      // Only even items 16~28 are on the screen.
+      expect(find.text('item15'), findsNothing);
+      expect(find.text('item16'), findsOneWidget);
+      expect(find.text('item28'), findsOneWidget);
+
+      skip = false;
+      await tester.pumpWidget(
+        MaterialApp(
+          home: Scaffold(
+            body: CustomScrollView(
+              slivers: <Widget>[
+                SliverMainAxisGroup(
+                  slivers: <Widget>[
+                    SliverList(delegate: SliverChildBuilderDelegate(buildItem, childCount: 30)),
+                  ],
+                ),
+              ],
+            ),
+          ),
+        ),
+      );
+
+      // Only items 12~19 are on the screen.
+      expect(find.text('item11'), findsNothing);
+      expect(find.text('item12'), findsOneWidget);
+      expect(find.text('item19'), findsOneWidget);
+      expect(find.text('item20'), findsNothing);
+
+      await tester.drag(find.byType(CustomScrollView), const Offset(0.0, 250.0));
+      await tester.pump();
+
+      // Only items 10~16 are on the screen.
+      expect(find.text('item9'), findsNothing);
+      expect(find.text('item10'), findsOneWidget);
+      expect(find.text('item16'), findsOneWidget);
+      expect(find.text('item17'), findsNothing);
+
+      // The inaccurate scroll offset should reach zero at this point
+      await tester.drag(find.byType(CustomScrollView), const Offset(0.0, 250.0));
+      await tester.pump();
+
+      // Only items 7~13 are on the screen.
+      expect(find.text('item6'), findsNothing);
+      expect(find.text('item7'), findsOneWidget);
+      expect(find.text('item13'), findsOneWidget);
+      expect(find.text('item14'), findsNothing);
+
+      // It will be corrected as we scroll, so we have to drag multiple times.
+      await tester.drag(find.byType(CustomScrollView), const Offset(0.0, 250.0));
+      await tester.pump();
+      await tester.drag(find.byType(CustomScrollView), const Offset(0.0, 250.0));
+      await tester.pump();
+      await tester.drag(find.byType(CustomScrollView), const Offset(0.0, 250.0));
+      await tester.pump();
+
+      // Only items 0~6 are on the screen.
+      expect(find.text('item0'), findsOneWidget);
+      expect(find.text('item6'), findsOneWidget);
+      expect(find.text('item7'), findsNothing);
+    },
+  );


That's not a bad point. Are the two pumpWidgets sections of the test self contained too? Maybe you could split it into two separate tests.

justinmc · 2025-08-26T23:09:12Z

packages/flutter/test/widgets/sliver_main_axis_group_test.dart

  );
+
+  testWidgets(
+    'With SliverList can handle inaccurate scroll offset due to changes in children list',


I confirmed that this test fails on master.

justinmc · 2025-08-26T23:09:42Z

packages/flutter/lib/src/rendering/sliver_group.dart

+      if (childLayoutGeometry case SliverGeometry(
+        :final double scrollOffsetCorrection,
+      ) when scrollOffsetCorrection != 0) {
+        geometry = SliverGeometry(scrollOffsetCorrection: scrollOffsetCorrection);


I'll defer to @Piinks on whether this is the right way to solve this.

Piinks · 2025-08-28T22:57:29Z

packages/flutter/lib/src/rendering/sliver_group.dart

+      if (childLayoutGeometry case SliverGeometry(
+        :final double scrollOffsetCorrection,
+      ) when scrollOffsetCorrection != 0) {


Can you write this differently? We have not adopted this pattern in the framework because it can be hard for readability in some cases.

Piinks

@yiiim may also be interested in this, they have made a lot of changes here recently.

yiiim

LGTM for the Sliver changes. For tests and formatting issues, I will follow the guidance of @justinmc and @Piinks

Piinks

This LGTM, thank you for the contribution!

Piinks · 2025-09-05T20:33:11Z

There are some merge conflicts that need to be resolved here, and then this should be good to go!

…tter#174369)

flutter/flutter@87d5b75...973320c 2025-09-08 [email protected] Roll Fuchsia Linux SDK from V_KooYFGRmGm6JrqM... to m7Qmvj5wtfPlMA8i8... (flutter/flutter#175067) 2025-09-08 [email protected] Roll Packages from cab2ac2 to 24588c6 (1 revision) (flutter/flutter#175070) 2025-09-08 [email protected] Roll Skia from e1a200e25998 to 0c2b0a00b7b5 (1 revision) (flutter/flutter#175065) 2025-09-08 [email protected] Roll Dart SDK from c7518d480c73 to 83c6b6124380 (1 revision) (flutter/flutter#175064) 2025-09-08 [email protected] Roll Skia from 87ee27220133 to e1a200e25998 (4 revisions) (flutter/flutter#175061) 2025-09-08 [email protected] [ Widget Preview ] Improve `--machine` output (flutter/flutter#175003) 2025-09-08 [email protected] Fix DropdownMenuFormField does not clear text field content on reset … (flutter/flutter#174937) 2025-09-08 [email protected] Roll Skia from 310062ae6aee to 87ee27220133 (1 revision) (flutter/flutter#175051) 2025-09-08 [email protected] Roll Dart SDK from 34cd2b4b56dd to c7518d480c73 (1 revision) (flutter/flutter#175049) 2025-09-07 [email protected] Roll Skia from b00ca433657b to 310062ae6aee (1 revision) (flutter/flutter#175046) 2025-09-07 [email protected] Roll Fuchsia Linux SDK from LRAtLJWUiZNmSVShG... to V_KooYFGRmGm6JrqM... (flutter/flutter#175040) 2025-09-07 [email protected] Roll Skia from 064cf12ad0cc to b00ca433657b (1 revision) (flutter/flutter#175028) 2025-09-06 [email protected] Roll Fuchsia Linux SDK from xG_uERsxHvUwFHpF2... to LRAtLJWUiZNmSVShG... (flutter/flutter#175018) 2025-09-06 [email protected] Roll Dart SDK from f138e58bcf6d to 34cd2b4b56dd (2 revisions) (flutter/flutter#175017) 2025-09-06 [email protected] Roll Skia from 5c60f8a66270 to 064cf12ad0cc (2 revisions) (flutter/flutter#175015) 2025-09-05 [email protected] Roll Skia from f29ea2795934 to 5c60f8a66270 (2 revisions) (flutter/flutter#175010) 2025-09-05 [email protected] Fix SliverMainAxisGroup scrollOffsetCorrection (flutter/flutter#174369) 2025-09-05 [email protected] Roll Skia from 759f406584da to f29ea2795934 (3 revisions) (flutter/flutter#175000) 2025-09-05 [email protected] Roll Dart SDK from 5e3afb74ffdf to f138e58bcf6d (15 revisions) (flutter/flutter#174994) 2025-09-05 [email protected] Roll Packages from 98580c6 to cab2ac2 (2 revisions) (flutter/flutter#174998) 2025-09-05 [email protected] [ios26]fix host engine compile error (flutter/flutter#174723) 2025-09-05 [email protected] Added note about how to compile licenses_cpp (flutter/flutter#174947) 2025-09-05 [email protected] Roll Skia from 845ec125e94c to 759f406584da (3 revisions) (flutter/flutter#174992) 2025-09-05 [email protected] [web] Minor simplification in flutter.js loader (flutter/flutter#174963) 2025-09-05 [email protected] deletes the old license checker. (flutter/flutter#174719) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/flutter-packages Please CC [email protected],[email protected] on the revert to ensure that a human is aware of the problem. To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

…r#9977) flutter/flutter@87d5b75...973320c 2025-09-08 [email protected] Roll Fuchsia Linux SDK from V_KooYFGRmGm6JrqM... to m7Qmvj5wtfPlMA8i8... (flutter/flutter#175067) 2025-09-08 [email protected] Roll Packages from cab2ac2 to 24588c6 (1 revision) (flutter/flutter#175070) 2025-09-08 [email protected] Roll Skia from e1a200e25998 to 0c2b0a00b7b5 (1 revision) (flutter/flutter#175065) 2025-09-08 [email protected] Roll Dart SDK from c7518d480c73 to 83c6b6124380 (1 revision) (flutter/flutter#175064) 2025-09-08 [email protected] Roll Skia from 87ee27220133 to e1a200e25998 (4 revisions) (flutter/flutter#175061) 2025-09-08 [email protected] [ Widget Preview ] Improve `--machine` output (flutter/flutter#175003) 2025-09-08 [email protected] Fix DropdownMenuFormField does not clear text field content on reset … (flutter/flutter#174937) 2025-09-08 [email protected] Roll Skia from 310062ae6aee to 87ee27220133 (1 revision) (flutter/flutter#175051) 2025-09-08 [email protected] Roll Dart SDK from 34cd2b4b56dd to c7518d480c73 (1 revision) (flutter/flutter#175049) 2025-09-07 [email protected] Roll Skia from b00ca433657b to 310062ae6aee (1 revision) (flutter/flutter#175046) 2025-09-07 [email protected] Roll Fuchsia Linux SDK from LRAtLJWUiZNmSVShG... to V_KooYFGRmGm6JrqM... (flutter/flutter#175040) 2025-09-07 [email protected] Roll Skia from 064cf12ad0cc to b00ca433657b (1 revision) (flutter/flutter#175028) 2025-09-06 [email protected] Roll Fuchsia Linux SDK from xG_uERsxHvUwFHpF2... to LRAtLJWUiZNmSVShG... (flutter/flutter#175018) 2025-09-06 [email protected] Roll Dart SDK from f138e58bcf6d to 34cd2b4b56dd (2 revisions) (flutter/flutter#175017) 2025-09-06 [email protected] Roll Skia from 5c60f8a66270 to 064cf12ad0cc (2 revisions) (flutter/flutter#175015) 2025-09-05 [email protected] Roll Skia from f29ea2795934 to 5c60f8a66270 (2 revisions) (flutter/flutter#175010) 2025-09-05 [email protected] Fix SliverMainAxisGroup scrollOffsetCorrection (flutter/flutter#174369) 2025-09-05 [email protected] Roll Skia from 759f406584da to f29ea2795934 (3 revisions) (flutter/flutter#175000) 2025-09-05 [email protected] Roll Dart SDK from 5e3afb74ffdf to f138e58bcf6d (15 revisions) (flutter/flutter#174994) 2025-09-05 [email protected] Roll Packages from 98580c6 to cab2ac2 (2 revisions) (flutter/flutter#174998) 2025-09-05 [email protected] [ios26]fix host engine compile error (flutter/flutter#174723) 2025-09-05 [email protected] Added note about how to compile licenses_cpp (flutter/flutter#174947) 2025-09-05 [email protected] Roll Skia from 845ec125e94c to 759f406584da (3 revisions) (flutter/flutter#174992) 2025-09-05 [email protected] [web] Minor simplification in flutter.js loader (flutter/flutter#174963) 2025-09-05 [email protected] deletes the old license checker. (flutter/flutter#174719) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/flutter-packages Please CC [email protected],[email protected] on the revert to ensure that a human is aware of the problem. To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

…tter#174369)

github-actions bot added framework flutter/packages/flutter repository. See also f: labels. f: scrolling Viewports, list views, slivers, etc. labels Aug 25, 2025

gemini-code-assist bot reviewed Aug 25, 2025

View reviewed changes

justinmc requested a review from Piinks August 26, 2025 22:08

justinmc reviewed Aug 26, 2025

View reviewed changes

Piinks reviewed Aug 28, 2025

View reviewed changes

yiiim approved these changes Aug 29, 2025

View reviewed changes

Piinks approved these changes Sep 5, 2025

View reviewed changes

manu-sncf added 3 commits September 5, 2025 22:37

Fix SliverMainAxisGroup scrollOffsetCorrection

aa9de8a

Review - check all items

b0f6c72

Review - no pattern matching

482c7ea

manu-sncf force-pushed the scroll_offset_correction branch from 0dec903 to 482c7ea Compare September 5, 2025 20:38

Piinks added the autosubmit Merge PR when tree becomes green via auto submit App label Sep 5, 2025

auto-submit bot added this pull request to the merge queue Sep 5, 2025

Merged via the queue into flutter:master with commit 242043f Sep 5, 2025
78 checks passed

flutter-dashboard bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Sep 5, 2025

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 6, 2025

242043fca Fix SliverMainAxisGroup scrollOffsetCorrection (flutter/flu…

6b750db

…tter#174369)

engine-flutter-autoroll mentioned this pull request Sep 6, 2025

Roll Flutter from 87d5b753196c to ad3cd078d192 (13 revisions) flutter/packages#9966

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 6, 2025

242043fca Fix SliverMainAxisGroup scrollOffsetCorrection (flutter/flu…

6834dc5

…tter#174369)

engine-flutter-autoroll mentioned this pull request Sep 6, 2025

Roll Flutter from 87d5b753196c to ad3cd078d192 (13 revisions) flutter/packages#9967

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 7, 2025

242043fca Fix SliverMainAxisGroup scrollOffsetCorrection (flutter/flu…

3d721b0

…tter#174369)

engine-flutter-autoroll mentioned this pull request Sep 7, 2025

Roll Flutter from 87d5b753196c to 06ab279c02bb (15 revisions) flutter/packages#9969

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 7, 2025

242043fca Fix SliverMainAxisGroup scrollOffsetCorrection (flutter/flu…

7e092da

…tter#174369)

engine-flutter-autoroll mentioned this pull request Sep 7, 2025

Roll Flutter from 87d5b753196c to 06ab279c02bb (15 revisions) flutter/packages#9970

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 7, 2025

242043fca Fix SliverMainAxisGroup scrollOffsetCorrection (flutter/flu…

a2ec25f

…tter#174369)

engine-flutter-autoroll mentioned this pull request Sep 7, 2025

Roll Flutter from 87d5b753196c to 0081ee50dc3f (16 revisions) flutter/packages#9971

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 8, 2025

242043fca Fix SliverMainAxisGroup scrollOffsetCorrection (flutter/flu…

c4ef5b7

…tter#174369)

engine-flutter-autoroll mentioned this pull request Sep 8, 2025

Roll Flutter from 87d5b753196c to c14a286a60d9 (21 revisions) flutter/packages#9974

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 8, 2025

242043fca Fix SliverMainAxisGroup scrollOffsetCorrection (flutter/flu…

06675b4

…tter#174369)

engine-flutter-autoroll mentioned this pull request Sep 8, 2025

Roll Flutter from 87d5b753196c to 8c63ca967ec4 (22 revisions) flutter/packages#9975

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 8, 2025

242043fca Fix SliverMainAxisGroup scrollOffsetCorrection (flutter/flu…

9bbfd99

…tter#174369)

engine-flutter-autoroll mentioned this pull request Sep 8, 2025

Roll Flutter from 87d5b753196c to 74459efb413d (23 revisions) flutter/packages#9976

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 8, 2025

242043fca Fix SliverMainAxisGroup scrollOffsetCorrection (flutter/flu…

f3134b5

…tter#174369)

engine-flutter-autoroll mentioned this pull request Sep 8, 2025

Roll Flutter from 87d5b753196c to 973320c0270b (25 revisions) flutter/packages#9977

Merged

manu-sncf deleted the scroll_offset_correction branch September 25, 2025 10:50

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Nov 12, 2025

242043fca Fix SliverMainAxisGroup scrollOffsetCorrection (flutter/flu…

161fe7c

…tter#174369)

Fix SliverMainAxisGroup scrollOffsetCorrection #174369

Fix SliverMainAxisGroup scrollOffsetCorrection #174369

Uh oh!

Conversation

manu-sncf commented Aug 25, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Pre-launch Checklist

Uh oh!

gemini-code-assist bot left a comment

Choose a reason for hiding this comment

Code Review

Uh oh!

gemini-code-assist bot Aug 25, 2025

Choose a reason for hiding this comment

Style Guide References

Footnotes

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

justinmc commented Aug 26, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

manu-sncf Aug 27, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Piinks left a comment

Choose a reason for hiding this comment

Uh oh!

yiiim left a comment

Choose a reason for hiding this comment

Uh oh!

Piinks left a comment

Choose a reason for hiding this comment

Uh oh!

Piinks commented Sep 5, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

manu-sncf commented Aug 25, 2025 •

edited

Loading

manu-sncf Aug 27, 2025 •

edited

Loading

Piinks commented Sep 5, 2025 •

edited

Loading