refactor: clean up unused imports, remove TODO comments, and format code across samples

ericwindmill · ericwindmill · commit 6a3ffef57b26 · 2026-04-15T11:55:49.000-07:00
diff --git a/add_to_app/books/flutter_module_books/lib/main.dart b/add_to_app/books/flutter_module_books/lib/main.dart
@@ -66,8 +66,6 @@ class _BookDetailState extends State<BookDetail> {
 
     // Registering this `FlutterBookApiHandler` class lets us receive incoming
     // calls from the platform.
-    // TODO(gaaclarke): make the setup method an instance method so it's
-    // injectable https://github.com/flutter/flutter/issues/59119.
     FlutterBookApi.setup(
       FlutterBookApiHandler(
         // The `FlutterBookApi` just has one method. Just give a closure for that
@@ -82,7 +80,6 @@ class _BookDetailState extends State<BookDetail> {
               this.book!.title = titleTextController.text;
             });
             // Subtitle could be null.
-            // TODO(gaaclarke): https://github.com/flutter/flutter/issues/59118.
             subtitleTextController.text = book.subtitle ?? '';
             subtitleTextController.addListener(() {
               this.book!.subtitle = subtitleTextController.text;
diff --git a/cupertino_gallery/lib/gallery_home.dart b/cupertino_gallery/lib/gallery_home.dart
@@ -37,11 +37,11 @@ class GalleryHome extends StatelessWidget {
             return switch (index) {
               0 => const WidgetsPage(),
               1 => SettingsPage(
-                    onThemeChange: onThemeChange,
-                    isDarkMode: isDarkMode,
-                    onTextSizeChange: onTextSizeChange,
-                    textSize: textSize,
-                  ),
+                onThemeChange: onThemeChange,
+                isDarkMode: isDarkMode,
+                onTextSizeChange: onTextSizeChange,
+                textSize: textSize,
+              ),
               _ => const Center(child: Text('Widgets')),
             };
           },
diff --git a/cupertino_gallery/lib/main.dart b/cupertino_gallery/lib/main.dart
@@ -37,24 +37,31 @@ class _CupertinoGalleryAppState extends State<CupertinoGalleryApp> {
   @override
   Widget build(BuildContext context) {
     final baseTheme = CupertinoThemeData(
-      brightness:
-          _themeMode == ThemeMode.dark ? Brightness.dark : Brightness.light,
+      brightness: _themeMode == ThemeMode.dark
+          ? Brightness.dark
+          : Brightness.light,
     );
     final textTheme = baseTheme.textTheme.copyWith(
-      textStyle:
-          baseTheme.textTheme.textStyle.copyWith(fontSize: 14 * _textSize),
-      actionTextStyle: baseTheme.textTheme.actionTextStyle
-          .copyWith(fontSize: 14 * _textSize),
-      tabLabelTextStyle: baseTheme.textTheme.tabLabelTextStyle
-          .copyWith(fontSize: 10 * _textSize),
-      navTitleTextStyle: baseTheme.textTheme.navTitleTextStyle
-          .copyWith(fontSize: 17 * _textSize),
+      textStyle: baseTheme.textTheme.textStyle.copyWith(
+        fontSize: 14 * _textSize,
+      ),
+      actionTextStyle: baseTheme.textTheme.actionTextStyle.copyWith(
+        fontSize: 14 * _textSize,
+      ),
+      tabLabelTextStyle: baseTheme.textTheme.tabLabelTextStyle.copyWith(
+        fontSize: 10 * _textSize,
+      ),
+      navTitleTextStyle: baseTheme.textTheme.navTitleTextStyle.copyWith(
+        fontSize: 17 * _textSize,
+      ),
       navLargeTitleTextStyle: baseTheme.textTheme.navLargeTitleTextStyle
           .copyWith(fontSize: 34 * _textSize),
-      navActionTextStyle: baseTheme.textTheme.navActionTextStyle
-          .copyWith(fontSize: 17 * _textSize),
-      pickerTextStyle: baseTheme.textTheme.pickerTextStyle
-          .copyWith(fontSize: 21 * _textSize),
+      navActionTextStyle: baseTheme.textTheme.navActionTextStyle.copyWith(
+        fontSize: 17 * _textSize,
+      ),
+      pickerTextStyle: baseTheme.textTheme.pickerTextStyle.copyWith(
+        fontSize: 21 * _textSize,
+      ),
       dateTimePickerTextStyle: baseTheme.textTheme.dateTimePickerTextStyle
           .copyWith(fontSize: 21 * _textSize),
     );
diff --git a/cupertino_gallery/lib/settings_page.dart b/cupertino_gallery/lib/settings_page.dart
@@ -140,4 +140,3 @@ class _SettingsPageState extends State<SettingsPage> {
     );
   }
 }
-
diff --git a/cupertino_gallery/lib/widgets/action_sheet_page.dart b/cupertino_gallery/lib/widgets/action_sheet_page.dart
@@ -6,9 +6,7 @@ class ActionSheetPage extends StatelessWidget {
   @override
   Widget build(BuildContext context) {
     return CupertinoPageScaffold(
-      navigationBar: const CupertinoNavigationBar(
-        middle: Text('Action Sheet'),
-      ),
+      navigationBar: const CupertinoNavigationBar(middle: Text('Action Sheet')),
       child: Center(
         child: CupertinoButton(
           child: const Text('Show Action Sheet'),
@@ -30,7 +28,7 @@ class ActionSheetPage extends StatelessWidget {
                     onPressed: () {
                       Navigator.pop(context);
                     },
-                  )
+                  ),
                 ],
                 cancelButton: CupertinoActionSheetAction(
                   isDefaultAction: true,
diff --git a/cupertino_gallery/lib/widgets/activity_indicator_page.dart b/cupertino_gallery/lib/widgets/activity_indicator_page.dart
@@ -6,12 +6,8 @@ class ActivityIndicatorPage extends StatelessWidget {
   @override
   Widget build(BuildContext context) {
     return const CupertinoPageScaffold(
-      navigationBar: CupertinoNavigationBar(
-        middle: Text('Activity Indicator'),
-      ),
-      child: Center(
-        child: CupertinoActivityIndicator(),
-      ),
+      navigationBar: CupertinoNavigationBar(middle: Text('Activity Indicator')),
+      child: Center(child: CupertinoActivityIndicator()),
     );
   }
 }
diff --git a/cupertino_gallery/lib/widgets/checkbox_page.dart b/cupertino_gallery/lib/widgets/checkbox_page.dart
@@ -13,9 +13,7 @@ class _CheckboxPageState extends State<CheckboxPage> {
   @override
   Widget build(BuildContext context) {
     return CupertinoPageScaffold(
-      navigationBar: const CupertinoNavigationBar(
-        middle: Text('Checkbox'),
-      ),
+      navigationBar: const CupertinoNavigationBar(middle: Text('Checkbox')),
       child: Center(
         child: CupertinoCheckbox(
           value: _value,
diff --git a/cupertino_gallery/lib/widgets/date_picker_page.dart b/cupertino_gallery/lib/widgets/date_picker_page.dart
@@ -6,15 +6,11 @@ class DatePickerPage extends StatelessWidget {
   @override
   Widget build(BuildContext context) {
     return CupertinoPageScaffold(
-      navigationBar: const CupertinoNavigationBar(
-        middle: Text('Date Picker'),
-      ),
+      navigationBar: const CupertinoNavigationBar(middle: Text('Date Picker')),
       child: Center(
         child: SizedBox(
           height: 200,
-          child: CupertinoDatePicker(
-            onDateTimeChanged: (DateTime newDate) {},
-          ),
+          child: CupertinoDatePicker(onDateTimeChanged: (DateTime newDate) {}),
         ),
       ),
     );
diff --git a/cupertino_gallery/lib/widgets/picker_page.dart b/cupertino_gallery/lib/widgets/picker_page.dart
@@ -6,20 +6,14 @@ class PickerPage extends StatelessWidget {
   @override
   Widget build(BuildContext context) {
     return CupertinoPageScaffold(
-      navigationBar: const CupertinoNavigationBar(
-        middle: Text('Picker'),
-      ),
+      navigationBar: const CupertinoNavigationBar(middle: Text('Picker')),
       child: Center(
         child: SizedBox(
           height: 200,
           child: CupertinoPicker(
             itemExtent: 32,
             onSelectedItemChanged: (int index) {},
-            children: const <Widget>[
-              Text('One'),
-              Text('Two'),
-              Text('Three'),
-            ],
+            children: const <Widget>[Text('One'), Text('Two'), Text('Three')],
           ),
         ),
       ),
diff --git a/cupertino_gallery/lib/widgets/search_text_field_page.dart b/cupertino_gallery/lib/widgets/search_text_field_page.dart
@@ -6,9 +6,7 @@ class SearchTextFieldPage extends StatelessWidget {
   @override
   Widget build(BuildContext context) {
     return const CupertinoPageScaffold(
-      navigationBar: CupertinoNavigationBar(
-        middle: Text('Search Text Field'),
-      ),
+      navigationBar: CupertinoNavigationBar(middle: Text('Search Text Field')),
       child: Center(
         child: Padding(
           padding: EdgeInsets.all(16.0),
diff --git a/cupertino_gallery/lib/widgets/slider_page.dart b/cupertino_gallery/lib/widgets/slider_page.dart
@@ -13,9 +13,7 @@ class _SliderPageState extends State<SliderPage> {
   @override
   Widget build(BuildContext context) {
     return CupertinoPageScaffold(
-      navigationBar: const CupertinoNavigationBar(
-        middle: Text('Slider'),
-      ),
+      navigationBar: const CupertinoNavigationBar(middle: Text('Slider')),
       child: Center(
         child: CupertinoSlider(
           value: _value,
diff --git a/cupertino_gallery/lib/widgets/switch_page.dart b/cupertino_gallery/lib/widgets/switch_page.dart
@@ -13,9 +13,7 @@ class _SwitchPageState extends State<SwitchPage> {
   @override
   Widget build(BuildContext context) {
     return CupertinoPageScaffold(
-      navigationBar: const CupertinoNavigationBar(
-        middle: Text('Switch'),
-      ),
+      navigationBar: const CupertinoNavigationBar(middle: Text('Switch')),
       child: Center(
         child: CupertinoSwitch(
           value: _value,
diff --git a/cupertino_gallery/lib/widgets/text_field_page.dart b/cupertino_gallery/lib/widgets/text_field_page.dart
@@ -6,15 +6,11 @@ class TextFieldPage extends StatelessWidget {
   @override
   Widget build(BuildContext context) {
     return const CupertinoPageScaffold(
-      navigationBar: CupertinoNavigationBar(
-        middle: Text('Text Field'),
-      ),
+      navigationBar: CupertinoNavigationBar(middle: Text('Text Field')),
       child: Center(
         child: Padding(
           padding: EdgeInsets.all(16.0),
-          child: CupertinoTextField(
-            placeholder: 'Enter text',
-          ),
+          child: CupertinoTextField(placeholder: 'Enter text'),
         ),
       ),
     );
diff --git a/cupertino_gallery/lib/widgets/text_theme_page.dart b/cupertino_gallery/lib/widgets/text_theme_page.dart
@@ -6,9 +6,7 @@ class TextThemePage extends StatelessWidget {
   @override
   Widget build(BuildContext context) {
     return CupertinoPageScaffold(
-      navigationBar: const CupertinoNavigationBar(
-        middle: Text('Text Theme'),
-      ),
+      navigationBar: const CupertinoNavigationBar(middle: Text('Text Theme')),
       child: Center(
         child: Column(
           mainAxisAlignment: MainAxisAlignment.center,
@@ -35,9 +33,9 @@ class TextThemePage extends StatelessWidget {
             const SizedBox(height: 16),
             Text(
               'This is the nav large title text style',
-              style: CupertinoTheme.of(context)
-                  .textTheme
-                  .navLargeTitleTextStyle,
+              style: CupertinoTheme.of(
+                context,
+              ).textTheme.navLargeTitleTextStyle,
             ),
             const SizedBox(height: 16),
             Text(
@@ -47,8 +45,9 @@ class TextThemePage extends StatelessWidget {
             const SizedBox(height: 16),
             Text(
               'This is the date time picker text style',
-              style:
-                  CupertinoTheme.of(context).textTheme.dateTimePickerTextStyle,
+              style: CupertinoTheme.of(
+                context,
+              ).textTheme.dateTimePickerTextStyle,
             ),
           ],
         ),
diff --git a/cupertino_gallery/lib/widgets/time_picker_page.dart b/cupertino_gallery/lib/widgets/time_picker_page.dart
@@ -6,9 +6,7 @@ class TimePickerPage extends StatelessWidget {
   @override
   Widget build(BuildContext context) {
     return CupertinoPageScaffold(
-      navigationBar: const CupertinoNavigationBar(
-        middle: Text('Time Picker'),
-      ),
+      navigationBar: const CupertinoNavigationBar(middle: Text('Time Picker')),
       child: Center(
         child: SizedBox(
           height: 200,
diff --git a/date_planner/lib/event_list.dart b/date_planner/lib/event_list.dart
@@ -62,9 +62,6 @@ class EventList extends StatelessWidget {
                       ),
                       children: [
                         for (Event e in events.sorted(p))
-                          // TODO: Support swipe action for deleting.
-                          // Should probably use Dismissable?
-                          // https://api.flutter.dev/flutter/widgets/Dismissible-class.html
                           EventRow(
                             event: e,
                             onTap: () async {
diff --git a/desktop_photo_search/fluent_ui/lib/main.dart b/desktop_photo_search/fluent_ui/lib/main.dart
@@ -5,7 +5,6 @@
 import 'dart:io';
 
 import 'package:fluent_ui/fluent_ui.dart';
-import 'package:flutter/foundation.dart';
 import 'package:flutter/services.dart';
 import 'package:logging/logging.dart';
 import 'package:menubar/menubar.dart' as menubar;
@@ -47,7 +46,6 @@ void main() {
   );
 }
 
-
 class UnsplashSearchApp extends StatelessWidget {
   const UnsplashSearchApp({super.key});
 
diff --git a/desktop_photo_search/fluent_ui/lib/src/widgets/split.dart b/desktop_photo_search/fluent_ui/lib/src/widgets/split.dart
@@ -19,8 +19,6 @@ import 'package:flutter/material.dart';
 ///
 /// The user can drag the widget with key [dividerKey] to change
 /// the space allocated between [firstChild] and [secondChild].
-// TODO(djshuckerow): introduce support for a minimum fraction a child
-//                    is allowed.
 class Split extends StatefulWidget {
   /// Builds a split oriented along [axis].
   const Split({
@@ -125,12 +123,6 @@ class _SplitState extends State<Split> {
       });
     }
 
-    // TODO(https://github.com/flutter/flutter/issues/43747): use an icon.
-    // The material icon for a drag handle is not currently available.
-    // For now, draw an indicator that is 3 lines running in the direction
-    // of the main axis, like a hamburger menu.
-    // TODO(https://github.com/flutter/devtools/issues/1265): update mouse
-    // to indicate that this is resizable.
     final dragIndicator = Flex(
       direction: isHorizontal ? Axis.vertical : Axis.horizontal,
       mainAxisSize: MainAxisSize.min,
diff --git a/desktop_photo_search/fluent_ui/lib/unsplash_access_key.dart b/desktop_photo_search/fluent_ui/lib/unsplash_access_key.dart
@@ -2,7 +2,7 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-// TODO: Retrieve an API Access Key from https://unsplash.com/developers
+// To use this sample: Retrieve an API Access Key from https://unsplash.com/developers
 const String unsplashAccessKey = '';
 
 // The application name for the above API Access Key.
diff --git a/desktop_photo_search/material/lib/src/widgets/split.dart b/desktop_photo_search/material/lib/src/widgets/split.dart
@@ -19,8 +19,6 @@ import 'package:flutter/material.dart';
 ///
 /// The user can drag the widget with key [dividerKey] to change
 /// the space allocated between [firstChild] and [secondChild].
-// TODO(djshuckerow): introduce support for a minimum fraction a child
-//                    is allowed.
 class Split extends StatefulWidget {
   /// Builds a split oriented along [axis].
   const Split({
@@ -125,12 +123,6 @@ class _SplitState extends State<Split> {
       });
     }
 
-    // TODO(https://github.com/flutter/flutter/issues/43747): use an icon.
-    // The material icon for a drag handle is not currently available.
-    // For now, draw an indicator that is 3 lines running in the direction
-    // of the main axis, like a hamburger menu.
-    // TODO(https://github.com/flutter/devtools/issues/1265): update mouse
-    // to indicate that this is resizable.
     final dragIndicator = Flex(
       direction: isHorizontal ? Axis.vertical : Axis.horizontal,
       mainAxisSize: MainAxisSize.min,
diff --git a/desktop_photo_search/material/lib/unsplash_access_key.dart b/desktop_photo_search/material/lib/unsplash_access_key.dart
@@ -2,7 +2,7 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-// TODO: Retrieve an API Access Key from https://unsplash.com/developers
+// To use this sample: Retrieve an API Access Key from https://unsplash.com/developers
 const String unsplashAccessKey = '';
 
 // The application name for the above API Access Key.
diff --git a/form_app/lib/main.dart b/form_app/lib/main.dart
@@ -2,9 +2,6 @@
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
-import 'dart:io' show Platform;
-
-import 'package:flutter/foundation.dart' show kIsWeb;
 import 'package:flutter/material.dart';
 import 'package:go_router/go_router.dart';
 
@@ -19,7 +16,6 @@ void main() {
   runApp(const FormApp());
 }
 
-
 final demos = [
   Demo(
     name: 'Sign in with HTTP',
diff --git a/material_3_demo/lib/src/component_screen.dart b/material_3_demo/lib/src/component_screen.dart
diff --git a/navigation_and_routing/lib/main.dart b/navigation_and_routing/lib/main.dart
diff --git a/navigation_and_routing/lib/src/app.dart b/navigation_and_routing/lib/src/app.dart

Original file line number	Diff line number	Diff line change
`@@ -140,4 +140,3 @@ class _SettingsPageState extends State<SettingsPage> {`
`140`	`140`	`);`
`141`	`141`	`}`
`142`	`142`	`}`
`143`		`-`