diff --git a/dev/tools/gen_defaults/lib/checkbox_template.dart b/dev/tools/gen_defaults/lib/checkbox_template.dart index 5ee85c476a..74159d6673 100644 --- a/dev/tools/gen_defaults/lib/checkbox_template.dart +++ b/dev/tools/gen_defaults/lib/checkbox_template.dart @@ -134,7 +134,7 @@ class _${blockName}DefaultsM3 extends CheckboxThemeData { MaterialTapTargetSize get materialTapTargetSize => _theme.materialTapTargetSize; @override - VisualDensity get visualDensity => _theme.visualDensity; + VisualDensity get visualDensity => VisualDensity.standard; @override OutlinedBorder get shape => const RoundedRectangleBorder( diff --git a/packages/flutter/lib/src/material/checkbox.dart b/packages/flutter/lib/src/material/checkbox.dart index c0ab2fb62f..6eb51af0cb 100644 --- a/packages/flutter/lib/src/material/checkbox.dart +++ b/packages/flutter/lib/src/material/checkbox.dart @@ -284,7 +284,9 @@ class Checkbox extends StatefulWidget { /// {@macro flutter.material.themedata.visualDensity} /// /// If null, then the value of [CheckboxThemeData.visualDensity] is used. If - /// that is also null, then the value of [ThemeData.visualDensity] is used. + /// that is also null and if [ThemeData.useMaterial3] is false, then the + /// value of [ThemeData.visualDensity] is used. Otherwise, the default value + /// is [VisualDensity.standard]. /// /// See also: /// @@ -1041,7 +1043,7 @@ class _CheckboxDefaultsM3 extends CheckboxThemeData { MaterialTapTargetSize get materialTapTargetSize => _theme.materialTapTargetSize; @override - VisualDensity get visualDensity => _theme.visualDensity; + VisualDensity get visualDensity => VisualDensity.standard; @override OutlinedBorder get shape => const RoundedRectangleBorder( diff --git a/packages/flutter/lib/src/material/theme_data.dart b/packages/flutter/lib/src/material/theme_data.dart index c321509bf8..3597e8ad5c 100644 --- a/packages/flutter/lib/src/material/theme_data.dart +++ b/packages/flutter/lib/src/material/theme_data.dart @@ -1161,10 +1161,14 @@ class ThemeData with Diagnosticable { /// A larger value translates to a spacing increase (less dense), and a /// smaller value translates to a spacing decrease (more dense). /// - /// In Material Design 3, the [visualDensity] does not override the value of - /// [IconButton.visualDensity] which defaults to [VisualDensity.standard] - /// for all platforms. To override the default value of [IconButton.visualDensity], - /// use [ThemeData.iconButtonTheme] instead. + /// In Material Design 3, the [visualDensity] does not override the default + /// visual for the following components which are set to [VisualDensity.standard] + /// for all platforms: + /// + /// * [IconButton] - To override the default value of [IconButton.visualDensity], + /// use [ThemeData.iconButtonTheme] instead. + /// * [Checkbox] - To override the default value of [Checkbox.visualDensity], + /// use [ThemeData.checkboxTheme] instead. /// {@endtemplate} final VisualDensity visualDensity; diff --git a/packages/flutter/test/material/checkbox_test.dart b/packages/flutter/test/material/checkbox_test.dart index 9ea0d2381b..e05c02d5e2 100644 --- a/packages/flutter/test/material/checkbox_test.dart +++ b/packages/flutter/test/material/checkbox_test.dart @@ -1057,9 +1057,57 @@ void main() { expect(value, isTrue); }); - testWidgets('Checkbox responds to density changes.', (WidgetTester tester) async { + testWidgets( + 'Material3 - Checkbox visual density cannot be overriden by ThemeData.visualDensity', + (WidgetTester tester) async { + const Key key = Key('test'); + Widget buldCheckbox() { + return MaterialApp( + theme: theme.copyWith(visualDensity: VisualDensity.compact), + home: Material( + child: Center(child: Checkbox(key: key, value: true, onChanged: (bool? value) {})), + ), + ); + } + + await tester.pumpWidget(buldCheckbox()); + await tester.pumpAndSettle(); + final RenderBox box = tester.renderObject(find.byKey(key)); + expect(box.size, equals(const Size(48, 48))); + }, + ); + + testWidgets( + 'Material3 - Checkbox with MaterialTapTargetSize.padded meets Material Guidelines on desktop', + (WidgetTester tester) async { + const Key key = Key('test'); + Widget buldCheckbox() { + return MaterialApp( + theme: theme, + home: Material( + child: Center( + child: Checkbox( + key: key, + materialTapTargetSize: MaterialTapTargetSize.padded, + value: true, + onChanged: (bool? value) {}, + ), + ), + ), + ); + } + + await tester.pumpWidget(buldCheckbox()); + await tester.pumpAndSettle(); + final RenderBox box = tester.renderObject(find.byKey(key)); + expect(box.size, equals(const Size(48, 48))); + }, + variant: TargetPlatformVariant.desktop(), + ); + + testWidgets('Checkbox responds to density changes', (WidgetTester tester) async { const Key key = Key('test'); - Future buildTest(VisualDensity visualDensity) async { + Future buildTest({VisualDensity? visualDensity}) async { return tester.pumpWidget( MaterialApp( theme: theme, @@ -1077,20 +1125,29 @@ void main() { ); } - await buildTest(VisualDensity.standard); + // Test the default visual density. + await buildTest(); + await tester.pumpAndSettle(); final RenderBox box = tester.renderObject(find.byKey(key)); + expect(box.size, equals(const Size(48, 48))); + + await buildTest(visualDensity: VisualDensity.standard); await tester.pumpAndSettle(); expect(box.size, equals(const Size(48, 48))); - await buildTest(const VisualDensity(horizontal: 3.0, vertical: 3.0)); + await buildTest(visualDensity: VisualDensity.compact); + await tester.pumpAndSettle(); + expect(box.size, equals(const Size(40, 40))); + + await buildTest(visualDensity: const VisualDensity(horizontal: 3.0, vertical: 3.0)); await tester.pumpAndSettle(); expect(box.size, equals(const Size(60, 60))); - await buildTest(const VisualDensity(horizontal: -3.0, vertical: -3.0)); + await buildTest(visualDensity: const VisualDensity(horizontal: -3.0, vertical: -3.0)); await tester.pumpAndSettle(); expect(box.size, equals(const Size(36, 36))); - await buildTest(const VisualDensity(horizontal: 3.0, vertical: -3.0)); + await buildTest(visualDensity: const VisualDensity(horizontal: 3.0, vertical: -3.0)); await tester.pumpAndSettle(); expect(box.size, equals(const Size(60, 36))); });