Add missing widget key parameters

JonasWanke · Nov 7, 2023 · b06e5a3 · b06e5a3
1 parent 5c39e2a
commit b06e5a3
Show file tree

Hide file tree

Showing 8 changed files with 26 additions and 5 deletions.
diff --git a/lib/src/debug_helper.dart b/lib/src/debug_helper.dart
@@ -5,6 +5,7 @@ import 'utils/level_selector.dart';
 
 class DebugHelper extends StatelessWidget {
   const DebugHelper({
+    super.key,
     required this.title,
     this.actions = const [],
     required this.child,
@@ -49,6 +50,7 @@ class DebugHelper extends StatelessWidget {
 /// [DiagnosticsNode]s only works in debug mode.
 class DiagnosticsBasedDebugHelper extends StatefulWidget {
   const DiagnosticsBasedDebugHelper({
+    super.key,
     required this.title,
     required this.diagnosticsStream,
   });

diff --git a/lib/src/debug_overlay.dart b/lib/src/debug_overlay.dart
@@ -195,7 +195,7 @@ bool get _isInDebugMode {
 }
 
 class DebugOverlayContent extends StatelessWidget {
-  const DebugOverlayContent({this.scrollController, this.onClose});
+  const DebugOverlayContent({super.key, this.scrollController, this.onClose});
 
   final ScrollController? scrollController;
   final VoidCallback? onClose;

diff --git a/lib/src/helpers/device_info.dart b/lib/src/helpers/device_info.dart
@@ -4,7 +4,7 @@ import '../debug_helper.dart';
 import 'device_info_io.dart' if (dart.library.js) 'device_info_html.dart';
 
 class DeviceInfoDebugHelper extends StatelessWidget {
-  const DeviceInfoDebugHelper();
+  const DeviceInfoDebugHelper({super.key});
 
   @override
   Widget build(BuildContext context) {

diff --git a/lib/src/helpers/logs/widgets.dart b/lib/src/helpers/logs/widgets.dart
@@ -16,6 +16,7 @@ import 'data.dart';
 class LogsDebugHelper extends StatefulWidget {
   const LogsDebugHelper(
     this.logs, {
+    super.key,
     this.initialMinLevel = DiagnosticLevel.debug,
     this.title = const Text('Logs'),
   });

diff --git a/lib/src/helpers/media_override.dart b/lib/src/helpers/media_override.dart
@@ -26,7 +26,11 @@ class MediaOverrideState {
 }
 
 class MediaOverrideDebugHelper extends StatefulWidget {
-  const MediaOverrideDebugHelper(this.state, {this.supportedLocales});
+  const MediaOverrideDebugHelper(
+    this.state, {
+    super.key,
+    this.supportedLocales,
+  });
 
   final ValueNotifier<MediaOverrideState> state;
 

diff --git a/lib/src/helpers/media_query.dart b/lib/src/helpers/media_query.dart
@@ -4,7 +4,7 @@ import 'package:flutter/widgets.dart';
 import '../debug_helper.dart';
 
 class MediaQueryDebugHelper extends StatelessWidget {
-  const MediaQueryDebugHelper();
+  const MediaQueryDebugHelper({super.key});
 
   @override
   Widget build(BuildContext context) {

diff --git a/lib/src/helpers/package_info.dart b/lib/src/helpers/package_info.dart
@@ -5,7 +5,7 @@ import 'package:package_info_plus/package_info_plus.dart';
 import '../debug_helper.dart';
 
 class PackageInfoDebugHelper extends StatelessWidget {
-  const PackageInfoDebugHelper();
+  const PackageInfoDebugHelper({super.key});
 
   @override
   Widget build(BuildContext context) {

diff --git a/lib/src/utils/level_selector.dart b/lib/src/utils/level_selector.dart
@@ -2,6 +2,7 @@ import 'package:flutter/material.dart';
 
 class DiagnosticLevelSelector extends StatelessWidget {
   const DiagnosticLevelSelector({
+    super.key,
     required this.value,
     required this.onSelected,
   });
@@ -36,6 +37,19 @@ class DiagnosticLevelSelector extends StatelessWidget {
       DiagnosticLevel.off => Icons.not_interested_outlined,
     };
   }
+
+  @override
+  void debugFillProperties(DiagnosticPropertiesBuilder properties) {
+    super.debugFillProperties(properties);
+    properties.add(EnumProperty<DiagnosticLevel>('value', value));
+  }
+
+  @override
+  void debugFillProperties(DiagnosticPropertiesBuilder properties) {
+    super.debugFillProperties(properties);
+    properties.add(ObjectFlagProperty<ValueSetter<DiagnosticLevel>>.has(
+        'onSelected', onSelected));
+  }
 }
 
 extension on DiagnosticLevel {