[various] Migrate example Radio groups to new RadioGroup API #10155

alexskobozev · 2025-10-01T10:04:53Z

This PR updates multiple package example apps to stop using deprecated groupValue and onChanged on individual Radio widgets and instead use the new RadioGroup<T> ancestor to manage the selected value and change handling.

Why

Flutter deprecated Radio.groupValue and Radio.onChanged in favor of RadioGroup, which centralizes selection state for related radios. The deprecation is currently reported across several examples in this repo (see linked issue for examples and affected packages). This PR resolves those warnings.

What changed (pattern applied across examples)

Wrap related Radio<T> widgets in a RadioGroup<T>(groupValue: ..., onChanged: ...).
Remove groupValue/onChanged from each Radio, leaving only value (+ its label UI).
- Example (from animations demo): old three Radio widgets each had groupValue/onChanged; now they are enclosed in a single RadioGroup<SharedAxisTransitionType> and the Radio children keep just their value.

Packages (examples) updated

animations
camera
camera_android
camera_android_camerax
camera_avfoundation
google_sign_in_web
two_dimensional_scrollables (both table_view and tree_view)

Screenshots

N/A (UI unchanged; only wiring of radios changed).

Links

Fixes: flutter/flutter#170915

Pre-Review Checklist

google-cla · 2025-10-01T10:04:59Z

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

Refactors the `shared_axis_transition.dart` example to use a new `RadioGroup` widget. This removes the deprecated `groupValue` and `onChanged` properties from each individual `Radio` button

Refactors the camera example to use a new `RadioGroup` widget. This removes deprecated `groupValue` and `onChanged` properties from each individual `RadioListTile` and moves them to the parent `RadioGroup`.

Refactors the `camera_android_camerax` example to use a new `RadioGroup` widget. This removes the deprecated `groupValue` and `onChanged` properties from each individual `RadioListTile` and moves them to the parent `RadioGroup`.

Refactors the `camera_avfoundation` example to use a new `RadioGroup` widget. This removes deprecated `onChanged` logic from each `RadioListTile` and moves it to the parent `RadioGroup`.

Refactors the `google_sign_in_web` example to use a new `RadioGroup` widget. This removes the deprecated `groupValue` and `onChanged` properties from each individual `RadioListTile` and moves them to the parent `RadioGroup`.

Refactors the `two_dimensional_scrollables` example to use a new `RadioGroup` widget. This removes the deprecated `groupValue` and `onChanged` properties from each individual `Radio` button and moves them to the parent `RadioGroup`.

Removes `const` from `Text` widgets inside a `Row` that was not itself `const`, and adds `const` to the `Row` itself.

Adds `const` to constructors and curly braces to control flow statements to fix new lint warnings in the `table_explorer.dart` and `tree_explorer.dart` examples.

Formats the `main.dart` file in the example application to align with current project standards. This change primarily involves code formatting and does not alter any functionality.

flutter-dashboard · 2025-10-01T12:02:01Z

It looks like this pull request may not have tests. Please make sure to add tests or get an explicit test exemption before merging.

If you are not sure if you need tests, consider this rule of thumb: the purpose of a test is to make sure someone doesn't accidentally revert the fix. Ask yourself, is there anything in your PR that you feel it is important we not accidentally revert back to how it was before your fix?

Reviewers: Read the Tree Hygiene page and make sure this patch meets those guidelines before LGTMing. If you believe this PR qualifies for a test exemption, contact "@test-exemption-reviewer" in the #hackers channel in Discord (don't just cc them here, they won't see it!). The test exemption team is a small volunteer group, so all reviewers should feel empowered to ask for tests, without delegating that responsibility entirely to the test exemption group.

stuartmorgan-g · 2025-10-01T12:05:29Z

Thanks for the contribution!

I read and followed the [relevant style guides] and ran [the auto-formatter].

You've applied non-standard formatting; please see the CI failures for details. It looks like either you didn't run the current version of the auto-formatter, or you are applying custom line length rules.

Version/CHANGELOG: Exempt. Changes are limited to example apps; no published packages modified (per repo exemption rules).

The repo exemption rule is:

PRs that only affect unpublished parts of example apps.

lib/main.dart is published on pub.dev, and most of the changes in this PR are to lib/main.dart files.

…adio_examples

alexskobozev · 2025-10-02T09:40:29Z

@stuartmorgan-g I've updated the format and changelogs using script

Piinks

Reviewed for animation and two_dimensional_scrollables - I think these would be good to release so that the example reflect the updated APIs from stable.

Piinks · 2025-10-02T15:47:11Z

packages/animations/CHANGELOG.md

@@ -1,5 +1,6 @@
 ## NEXT


Why would we not want to release the updated sample code?

pub.dev only shows lib/main.dart, and the changes in this package aren't to that file, so publishing wouldn't do anything (unless people are running examples out of their pub cache, but that's not something we expect/support in the repo policy).

Piinks · 2025-10-02T15:47:30Z

packages/two_dimensional_scrollables/CHANGELOG.md

@@ -1,5 +1,6 @@
 ## NEXT


Same applies here; the changes aren't to lib/main.dart.

packages/camera/camera_avfoundation/example/lib/main.dart

stuartmorgan-g · 2025-10-02T17:20:48Z

packages/google_sign_in/google_sign_in_web/example/lib/src/button_configuration_column.dart

  required String title,
  required List<T> values,
  T? selected,
  void Function(T?)? onChanged,


Is this ever null? If so, this is also a behavior change for the same reason. (If not, it should be replumbed through the file to be non-nullable.)

It can be null in button tester to support the state when there are no auth credentials.
Added enabled attribute instead of nullable onChanged since it has the same behaviour. Thanks for the catch
Before on/off (nullable onChanged)

After on/off (enabled attribute)

This reverts commit f6b2de3.

…adio_examples

Disables the `RadioListTile` widgets in the example when their `onChanged` callback is null. This provides a visual indication that the options are not selectable. The change also simplifies the `onChanged` handler logic within the `RadioGroup`.

Disables the `RadioListTile` widgets for camera selection while a video is being recorded. This prevents the user from attempting to switch cameras during a recording session, which is not a supported action.

stuartmorgan-g · 2025-10-03T14:38:12Z

packages/google_sign_in/google_sign_in_web/example/lib/src/button_configuration_column.dart

+    children: <Widget>[
+      RadioGroup<T>(
+        groupValue: selected,
+        onChanged: (T? v) => (onChanged ?? (_) {})(v),


What is the purpose of the wrapping function here? This seems much more complicated than just doing onChanged: onChanged ?? (_) {},

Replaced this with onChanged: onChanged ?? (_) {}, but seems if we run flutter run -d chrome --target=lib/button_tester.dart radiogroup is not working. Found that onChanged ?? (_) {} just returns a function, it never calls it. That’s why the callback in RadioGroup is never triggered. The wrapper actually invokes the callback with the selected value. Without it, the radios don’t switch because nothing ever notifies the group.

But you're right, I've simplified it

Found that onChanged ?? (_) {} just returns a function, it never calls it.

I don't follow. That expression is supposed to return a function without calling it, because it's providing a function as a parameter.

In this line for example, onChanged is a function reference, not a function call.

Simplifies the `onChanged` callback in the `RadioGroup` widget by using a null-aware `call` instead of a null-coalescing operator with an empty function.

github-actions bot added p: animations p: camera p: google_sign_in platform-android platform-ios platform-macos platform-web labels Oct 1, 2025

alexskobozev changed the title ~~Handle deprecated radio examples~~ [animations][camera][camera_android][camera_android_camerax][camera_avfoundation][google_sign_in_web][two_dimensional_scrollables] Migrate example Radio groups to new RadioGroup API Oct 1, 2025

alexskobozev marked this pull request as ready for review October 1, 2025 10:08

alexskobozev requested review from hellohuanlin, vashworth, camsim99, mdebbar, Piinks, bparrishMines and hannah-hyj as code owners October 1, 2025 10:08

alexskobozev added 7 commits October 1, 2025 14:20

[animations] Refactor Radio widgets in example

32a6914

Refactors the `shared_axis_transition.dart` example to use a new `RadioGroup` widget. This removes the deprecated `groupValue` and `onChanged` properties from each individual `Radio` button

[camera] Refactor Radio widgets in example

64064df

Refactors the camera example to use a new `RadioGroup` widget. This removes deprecated `groupValue` and `onChanged` properties from each individual `RadioListTile` and moves them to the parent `RadioGroup`.

[camera] Refactor Radio widgets in example

157a932

Refactors the camera example to use a new `RadioGroup` widget. This removes deprecated `groupValue` and `onChanged` properties from each individual `RadioListTile` and moves them to the parent `RadioGroup`.

[camera_avfoundation] Refactor Radio widgets in example

bc09c33

Refactors the `camera_avfoundation` example to use a new `RadioGroup` widget. This removes deprecated `onChanged` logic from each `RadioListTile` and moves it to the parent `RadioGroup`.

[google_sign_in_web] Refactor Radio widgets in example

dfc6124

Refactors the `google_sign_in_web` example to use a new `RadioGroup` widget. This removes the deprecated `groupValue` and `onChanged` properties from each individual `RadioListTile` and moves them to the parent `RadioGroup`.

alexskobozev force-pushed the handle_deprecated_radio_examples branch from 4053511 to 57f7be9 Compare October 1, 2025 10:21

Fix lints in animations example

7b2db36

Removes `const` from `Text` widgets inside a `Row` that was not itself `const`, and adds `const` to the `Row` itself.

alex-kobozev-talabat mentioned this pull request Oct 1, 2025

[packages] groupValue and onChanged are deprecated flutter/flutter#170915

Open

alexskobozev added 2 commits October 1, 2025 14:48

[two_dimensional_scrollables] Fix lints in examples

fe84be7

Adds `const` to constructors and curly braces to control flow statements to fix new lint warnings in the `table_explorer.dart` and `tree_explorer.dart` examples.

[camera_avfoundation] Format example main.dart

b813cf9

Formats the `main.dart` file in the example application to align with current project standards. This change primarily involves code formatting and does not alter any functionality.

stuartmorgan-g changed the title ~~[animations][camera][camera_android][camera_android_camerax][camera_avfoundation][google_sign_in_web][two_dimensional_scrollables] Migrate example Radio groups to new RadioGroup API~~ [various] Migrate example Radio groups to new RadioGroup API Oct 1, 2025

stuartmorgan-g marked this pull request as draft October 1, 2025 12:05

Adds miscellaneous formatting fixes

5bcf490

alexskobozev force-pushed the handle_deprecated_radio_examples branch from f4ca783 to 8f4d89f Compare October 2, 2025 07:21

Merge remote-tracking branch 'upstream/main' into handle_deprecated_r…

c6cdb22

…adio_examples

alexskobozev force-pushed the handle_deprecated_radio_examples branch from 8f4d89f to c6cdb22 Compare October 2, 2025 08:40

Update changelogs

f6b2de3

alexskobozev marked this pull request as ready for review October 2, 2025 09:39

Piinks reviewed Oct 2, 2025

View reviewed changes

stuartmorgan-g requested changes Oct 2, 2025

View reviewed changes

alexskobozev added 5 commits October 3, 2025 11:12

Revert "Update changelogs"

09df509

This reverts commit f6b2de3.

Merge remote-tracking branch 'upstream/main' into handle_deprecated_r…

7b2347e

…adio_examples

[camera_avfoundation] Disable camera selection during recording

3234a2b

Disables the `RadioListTile` widgets for camera selection while a video is being recorded. This prevents the user from attempting to switch cameras during a recording session, which is not a supported action.

Update changelogs

aaba578

alexskobozev force-pushed the handle_deprecated_radio_examples branch from dcb050f to aaba578 Compare October 3, 2025 12:37

stuartmorgan-g reviewed Oct 3, 2025

View reviewed changes

alexskobozev requested a review from stuartmorgan-g October 6, 2025 06:12

[google_sign_in_web] Simplify onChanged callback in example

5e347e4

Simplifies the `onChanged` callback in the `RadioGroup` widget by using a null-aware `call` instead of a null-coalescing operator with an empty function.

alexskobozev force-pushed the handle_deprecated_radio_examples branch from ad8b151 to 5e347e4 Compare October 6, 2025 09:38

Merge branch 'main' into handle_deprecated_radio_examples

3ded134

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

[various] Migrate example Radio groups to new RadioGroup API #10155

[various] Migrate example Radio groups to new RadioGroup API #10155

alexskobozev commented Oct 1, 2025 •

edited

Loading

Uh oh!

google-cla bot commented Oct 1, 2025

Uh oh!

flutter-dashboard bot commented Oct 1, 2025

Uh oh!

stuartmorgan-g commented Oct 1, 2025

Uh oh!

alexskobozev commented Oct 2, 2025 •

edited

Loading

Uh oh!

Piinks left a comment

Uh oh!

Piinks Oct 2, 2025

Uh oh!

stuartmorgan-g Oct 2, 2025

Uh oh!

Piinks Oct 2, 2025

Uh oh!

stuartmorgan-g Oct 2, 2025

Uh oh!

Uh oh!

stuartmorgan-g Oct 2, 2025

Uh oh!

alex-kobozev-talabat Oct 3, 2025

Uh oh!

stuartmorgan-g Oct 3, 2025

Uh oh!

alexskobozev Oct 6, 2025 •

edited

Loading

Uh oh!

alexskobozev Oct 6, 2025

Uh oh!

stuartmorgan-g Oct 6, 2025

Uh oh!

Uh oh!

[various] Migrate example Radio groups to new RadioGroup API #10155

Are you sure you want to change the base?

[various] Migrate example Radio groups to new RadioGroup API #10155

Conversation

alexskobozev commented Oct 1, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Why

What changed (pattern applied across examples)

Packages (examples) updated

Screenshots

Links

Pre-Review Checklist

Uh oh!

google-cla bot commented Oct 1, 2025

Uh oh!

flutter-dashboard bot commented Oct 1, 2025

Uh oh!

stuartmorgan-g commented Oct 1, 2025

Uh oh!

alexskobozev commented Oct 2, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Piinks left a comment

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!

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!

alexskobozev Oct 6, 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!

Uh oh!

alexskobozev commented Oct 1, 2025 •

edited

Loading

alexskobozev commented Oct 2, 2025 •

edited

Loading

alexskobozev Oct 6, 2025 •

edited

Loading