Skip to content

F
Front-End
Unverified Commit 9def8f6b authored by Seiya Kokushi's avatar Seiya Kokushi Committed by GitHub
Browse files
Options

Proposal to add barrier configs for showDatePicker, showTimePicker and showAboutDialog. (#130484) 

### Overview

Add `barrierDismissible`, `barrierColor` and `barrierLabel` parameters to  `showDatePicker`, `showTimePicker` and `showAboutDialog` which calls `showDialog` internally.
We can change these parameters with `showDialog` and Dialog widgets (like `DatePickerDialog`, `TimePickerDialog` or `AboutDialog`) directly. But, I think it is prefer to provide interfaces same as `showDialog` to keep application wide unified looks if it is used internally.

Fixes #130971
parent c8b9b15e
Hide whitespace changes
Inline Side-by-side
Showing with 564 additions and 14 deletions
packages/flutter/lib/src/material/about.dart
View file @ 9def8f6b
...@@ -170,9 +170,9 @@ class AboutListTile extends StatelessWidget { ...@@ -170,9 +170,9 @@ class AboutListTile extends StatelessWidget {
/// The licenses shown on the [LicensePage] are those returned by the /// The licenses shown on the [LicensePage] are those returned by the
/// [LicenseRegistry] API, which can be used to add more licenses to the list. /// [LicenseRegistry] API, which can be used to add more licenses to the list.
/// ///
/// The [context], [useRootNavigator], [routeSettings] and [anchorPoint] /// The [context], [barrierDismissible], [barrierColor], [barrierLabel],
/// arguments are passed to [showDialog], the documentation for which discusses /// [useRootNavigator], [routeSettings] and [anchorPoint] arguments are
/// how it is used. /// passed to [showDialog], the documentation for which discusses how it is used.
void showAboutDialog({ void showAboutDialog({
required BuildContext context, required BuildContext context,
String? applicationName, String? applicationName,
...@@ -180,12 +180,18 @@ void showAboutDialog({ ...@@ -180,12 +180,18 @@ void showAboutDialog({
Widget? applicationIcon, Widget? applicationIcon,
String? applicationLegalese, String? applicationLegalese,
List<Widget>? children, List<Widget>? children,
bool barrierDismissible = true,
Color? barrierColor,
String? barrierLabel,
bool useRootNavigator = true, bool useRootNavigator = true,
RouteSettings? routeSettings, RouteSettings? routeSettings,
Offset? anchorPoint, Offset? anchorPoint,
}) { }) {
showDialog<void>( showDialog<void>(
context: context, context: context,
barrierDismissible: barrierDismissible,
barrierColor: barrierColor,
barrierLabel: barrierLabel,
useRootNavigator: useRootNavigator, useRootNavigator: useRootNavigator,
builder: (BuildContext context) { builder: (BuildContext context) {
return AboutDialog( return AboutDialog(
......
packages/flutter/lib/src/material/date_picker.dart
View file @ 9def8f6b
...@@ -113,9 +113,10 @@ const double _kMaxTextScaleFactor = 1.3; ...@@ -113,9 +113,10 @@ const double _kMaxTextScaleFactor = 1.3;
/// [locale] and [textDirection] are non-null, [textDirection] overrides the /// [locale] and [textDirection] are non-null, [textDirection] overrides the
/// direction chosen for the [locale]. /// direction chosen for the [locale].
/// ///
/// The [context], [useRootNavigator] and [routeSettings] arguments are passed to /// The [context], [barrierDismissible], [barrierColor], [barrierLabel],
/// [showDialog], the documentation for which discusses how it is used. [context] /// [useRootNavigator] and [routeSettings] arguments are passed to [showDialog],
/// and [useRootNavigator] must be non-null. /// the documentation for which discusses how it is used.
/// [context], [barrierDismissible] and [useRootNavigator] must be non-null.
/// ///
/// The [builder] parameter can be used to wrap the dialog widget /// The [builder] parameter can be used to wrap the dialog widget
/// to add inherited widgets like [Theme]. /// to add inherited widgets like [Theme].
...@@ -169,6 +170,9 @@ Future<DateTime?> showDatePicker({ ...@@ -169,6 +170,9 @@ Future<DateTime?> showDatePicker({
String? cancelText, String? cancelText,
String? confirmText, String? confirmText,
Locale? locale, Locale? locale,
bool barrierDismissible = true,
Color? barrierColor,
String? barrierLabel,
bool useRootNavigator = true, bool useRootNavigator = true,
RouteSettings? routeSettings, RouteSettings? routeSettings,
TextDirection? textDirection, TextDirection? textDirection,
...@@ -243,6 +247,9 @@ Future<DateTime?> showDatePicker({ ...@@ -243,6 +247,9 @@ Future<DateTime?> showDatePicker({
return showDialog<DateTime>( return showDialog<DateTime>(
context: context, context: context,
barrierDismissible: barrierDismissible,
barrierColor: barrierColor,
barrierLabel: barrierLabel,
useRootNavigator: useRootNavigator, useRootNavigator: useRootNavigator,
routeSettings: routeSettings, routeSettings: routeSettings,
builder: (BuildContext context) { builder: (BuildContext context) {
...@@ -967,9 +974,10 @@ class _DatePickerHeader extends StatelessWidget { ...@@ -967,9 +974,10 @@ class _DatePickerHeader extends StatelessWidget {
/// [locale] and [textDirection] are non-null, [textDirection] overrides the /// [locale] and [textDirection] are non-null, [textDirection] overrides the
/// direction chosen for the [locale]. /// direction chosen for the [locale].
/// ///
/// The [context], [useRootNavigator] and [routeSettings] arguments are passed /// The [context], [barrierDismissible], [barrierColor], [barrierLabel],
/// to [showDialog], the documentation for which discusses how it is used. /// [useRootNavigator] and [routeSettings] arguments are passed to [showDialog],
/// [context] and [useRootNavigator] must be non-null. /// the documentation for which discusses how it is used.
/// [context], [barrierDismissible] and [useRootNavigator] must be non-null.
/// ///
/// The [builder] parameter can be used to wrap the dialog widget /// The [builder] parameter can be used to wrap the dialog widget
/// to add inherited widgets like [Theme]. /// to add inherited widgets like [Theme].
...@@ -1022,6 +1030,9 @@ Future<DateTimeRange?> showDateRangePicker({ ...@@ -1022,6 +1030,9 @@ Future<DateTimeRange?> showDateRangePicker({
String? fieldStartLabelText, String? fieldStartLabelText,
String? fieldEndLabelText, String? fieldEndLabelText,
Locale? locale, Locale? locale,
bool barrierDismissible = true,
Color? barrierColor,
String? barrierLabel,
bool useRootNavigator = true, bool useRootNavigator = true,
RouteSettings? routeSettings, RouteSettings? routeSettings,
TextDirection? textDirection, TextDirection? textDirection,
...@@ -1100,6 +1111,9 @@ Future<DateTimeRange?> showDateRangePicker({ ...@@ -1100,6 +1111,9 @@ Future<DateTimeRange?> showDateRangePicker({
return showDialog<DateTimeRange>( return showDialog<DateTimeRange>(
context: context, context: context,
barrierDismissible: barrierDismissible,
barrierColor: barrierColor,
barrierLabel: barrierLabel,
useRootNavigator: useRootNavigator, useRootNavigator: useRootNavigator,
routeSettings: routeSettings, routeSettings: routeSettings,
useSafeArea: false, useSafeArea: false,
......
packages/flutter/lib/src/material/dialog.dart
View file @ 9def8f6b
...@@ -1404,7 +1404,7 @@ Future<T?> showDialog<T>({ ...@@ -1404,7 +1404,7 @@ Future<T?> showDialog<T>({
required BuildContext context, required BuildContext context,
required WidgetBuilder builder, required WidgetBuilder builder,
bool barrierDismissible = true, bool barrierDismissible = true,
Color? barrierColor = Colors.black54, Color? barrierColor,
String? barrierLabel, String? barrierLabel,
bool useSafeArea = true, bool useSafeArea = true,
bool useRootNavigator = true, bool useRootNavigator = true,
...@@ -1426,7 +1426,7 @@ Future<T?> showDialog<T>({ ...@@ -1426,7 +1426,7 @@ Future<T?> showDialog<T>({
return Navigator.of(context, rootNavigator: useRootNavigator).push<T>(DialogRoute<T>( return Navigator.of(context, rootNavigator: useRootNavigator).push<T>(DialogRoute<T>(
context: context, context: context,
builder: builder, builder: builder,
barrierColor: barrierColor, barrierColor: barrierColor ?? Colors.black54,
barrierDismissible: barrierDismissible, barrierDismissible: barrierDismissible,
barrierLabel: barrierLabel, barrierLabel: barrierLabel,
useSafeArea: useSafeArea, useSafeArea: useSafeArea,
...@@ -1449,7 +1449,7 @@ Future<T?> showAdaptiveDialog<T>({ ...@@ -1449,7 +1449,7 @@ Future<T?> showAdaptiveDialog<T>({
required BuildContext context, required BuildContext context,
required WidgetBuilder builder, required WidgetBuilder builder,
bool? barrierDismissible, bool? barrierDismissible,
Color? barrierColor = Colors.black54, Color? barrierColor,
String? barrierLabel, String? barrierLabel,
bool useSafeArea = true, bool useSafeArea = true,
bool useRootNavigator = true, bool useRootNavigator = true,
......
packages/flutter/lib/src/material/time_picker.dart
View file @ 9def8f6b
...@@ -2874,8 +2874,9 @@ class _TimePickerState extends State<_TimePicker> with RestorationMixin { ...@@ -2874,8 +2874,9 @@ class _TimePickerState extends State<_TimePicker> with RestorationMixin {
/// ``` /// ```
/// {@end-tool} /// {@end-tool}
/// ///
/// The [context], [useRootNavigator] and [routeSettings] arguments are passed /// The [context], [barrierDismissible], [barrierColor], [barrierLabel],
/// to [showDialog], the documentation for which discusses how it is used. /// [useRootNavigator] and [routeSettings] arguments are passed to [showDialog],
/// the documentation for which discusses how it is used.
/// ///
/// The [builder] parameter can be used to wrap the dialog widget to add /// The [builder] parameter can be used to wrap the dialog widget to add
/// inherited widgets like [Localizations.override], [Directionality], or /// inherited widgets like [Localizations.override], [Directionality], or
...@@ -2954,6 +2955,9 @@ Future<TimeOfDay?> showTimePicker({ ...@@ -2954,6 +2955,9 @@ Future<TimeOfDay?> showTimePicker({
required BuildContext context, required BuildContext context,
required TimeOfDay initialTime, required TimeOfDay initialTime,
TransitionBuilder? builder, TransitionBuilder? builder,
bool barrierDismissible = true,
Color? barrierColor,
String? barrierLabel,
bool useRootNavigator = true, bool useRootNavigator = true,
TimePickerEntryMode initialEntryMode = TimePickerEntryMode.dial, TimePickerEntryMode initialEntryMode = TimePickerEntryMode.dial,
String? cancelText, String? cancelText,
...@@ -2983,6 +2987,9 @@ Future<TimeOfDay?> showTimePicker({ ...@@ -2983,6 +2987,9 @@ Future<TimeOfDay?> showTimePicker({
); );
return showDialog<TimeOfDay>( return showDialog<TimeOfDay>(
context: context, context: context,
barrierDismissible: barrierDismissible,
barrierColor: barrierColor,
barrierLabel: barrierLabel,
useRootNavigator: useRootNavigator, useRootNavigator: useRootNavigator,
builder: (BuildContext context) { builder: (BuildContext context) {
return builder == null ? dialog : builder(context, dialog); return builder == null ? dialog : builder(context, dialog);
......
packages/flutter/test/material/about_test.dart
View file @ 9def8f6b
...@@ -742,6 +742,160 @@ void main() { ...@@ -742,6 +742,160 @@ void main() {
expect(nestedObserver.licensePageCount, 0); expect(nestedObserver.licensePageCount, 0);
}); });
group('Barrier dismissible', () {
late AboutDialogObserver rootObserver;
setUpAll(() {
rootObserver = AboutDialogObserver();
});
testWidgets('Barrier is dismissible with default parameter', (WidgetTester tester) async {
await tester.pumpWidget(
MaterialApp(
navigatorObservers: <NavigatorObserver>[rootObserver],
home: Material(
child: Center(
child: Builder(
builder: (BuildContext context) {
return ElevatedButton(
child: const Text('X'),
onPressed: () => showAboutDialog(
context: context,
),
);
},
),
),
),
),
);
// Open the dialog.
await tester.tap(find.byType(ElevatedButton));
await tester.pumpAndSettle();
expect(rootObserver.dialogCount, 1);
// Tap on the barrier.
await tester.tapAt(const Offset(10.0, 10.0));
await tester.pumpAndSettle();
expect(rootObserver.dialogCount, 0);
});
testWidgets('Barrier is not dismissible with barrierDismissible is false', (WidgetTester tester) async {
await tester.pumpWidget(
MaterialApp(
navigatorObservers: <NavigatorObserver>[rootObserver],
home: Material(
child: Center(
child: Builder(
builder: (BuildContext context) {
return ElevatedButton(
child: const Text('X'),
onPressed: () => showAboutDialog(
context: context,
barrierDismissible: false
),
);
},
),
),
),
),
);
// Open the dialog.
await tester.tap(find.byType(ElevatedButton));
await tester.pumpAndSettle();
expect(rootObserver.dialogCount, 1);
// Tap on the barrier, which shouldn't do anything this time.
await tester.tapAt(const Offset(10.0, 10.0));
await tester.pumpAndSettle();
expect(rootObserver.dialogCount, 1);
});
});
testWidgets('Barrier color', (WidgetTester tester) async {
await tester.pumpWidget(
MaterialApp(
home: Material(
child: Center(
child: Builder(
builder: (BuildContext context) {
return ElevatedButton(
child: const Text('X'),
onPressed: () => showAboutDialog(
context: context,
),
);
},
),
),
),
),
);
// Open the dialog.
await tester.tap(find.byType(ElevatedButton));
await tester.pumpAndSettle();
expect(tester.widget<ModalBarrier>(find.byType(ModalBarrier).last).color, Colors.black54);
// Dismiss the dialog.
await tester.tapAt(const Offset(10.0, 10.0));
await tester.pumpWidget(
MaterialApp(
home: Material(
child: Center(
child: Builder(
builder: (BuildContext context) {
return ElevatedButton(
child: const Text('X'),
onPressed: () => showAboutDialog(
context: context,
barrierColor: Colors.pink,
),
);
},
),
),
),
),
);
// Open the dialog.
await tester.tap(find.byType(ElevatedButton));
await tester.pumpAndSettle();
expect(tester.widget<ModalBarrier>(find.byType(ModalBarrier).last).color, Colors.pink);
});
testWidgets('Barrier Label', (WidgetTester tester) async {
await tester.pumpWidget(
MaterialApp(
home: Material(
child: Center(
child: Builder(
builder: (BuildContext context) {
return ElevatedButton(
child: const Text('X'),
onPressed: () => showAboutDialog(
context: context,
barrierLabel: 'Custom Label',
),
);
},
),
),
),
),
);
// Open the dialog.
await tester.tap(find.byType(ElevatedButton));
await tester.pumpAndSettle();
expect(tester.widget<ModalBarrier>(find.byType(ModalBarrier).last).semanticsLabel, 'Custom Label');
});
testWidgetsWithLeakTracking('showAboutDialog uses root navigator by default', (WidgetTester tester) async { testWidgetsWithLeakTracking('showAboutDialog uses root navigator by default', (WidgetTester tester) async {
final AboutDialogObserver rootObserver = AboutDialogObserver(); final AboutDialogObserver rootObserver = AboutDialogObserver();
final AboutDialogObserver nestedObserver = AboutDialogObserver(); final AboutDialogObserver nestedObserver = AboutDialogObserver();
...@@ -1741,4 +1895,12 @@ class AboutDialogObserver extends NavigatorObserver { ...@@ -1741,4 +1895,12 @@ class AboutDialogObserver extends NavigatorObserver {
} }
super.didPush(route, previousRoute); super.didPush(route, previousRoute);
} }
@override
void didPop(Route<dynamic> route, Route<dynamic>? previousRoute) {
if (route is DialogRoute) {
dialogCount--;
}
super.didPop(route, previousRoute);
}
} }
packages/flutter/test/material/date_picker_test.dart
View file @ 9def8f6b
...@@ -335,6 +335,190 @@ void main() { ...@@ -335,6 +335,190 @@ void main() {
expect(tester.getBottomLeft(find.text('OK')).dx, 800 - ltrOkRight); expect(tester.getBottomLeft(find.text('OK')).dx, 800 - ltrOkRight);
}); });
group('Barrier dismissible', () {
late _DatePickerObserver rootObserver;
setUpAll(() {
rootObserver = _DatePickerObserver();
});
testWidgets('Barrier is dismissible with default parameter', (WidgetTester tester) async {
await tester.pumpWidget(
MaterialApp(
navigatorObservers: <NavigatorObserver>[rootObserver],
home: Material(
child: Center(
child: Builder(
builder: (BuildContext context) {
return ElevatedButton(
child: const Text('X'),
onPressed: () =>
showDatePicker(
context: context,
initialDate: DateTime.now(),
firstDate: DateTime(2018),
lastDate: DateTime(2030),
builder: (BuildContext context,
Widget? child) => const SizedBox(),
),
);
},
),
),
),
),
);
// Open the dialog.
await tester.tap(find.byType(ElevatedButton));
await tester.pumpAndSettle();
expect(rootObserver.datePickerCount, 1);
// Tap on the barrier.
await tester.tapAt(const Offset(10.0, 10.0));
await tester.pumpAndSettle();
expect(rootObserver.datePickerCount, 0);
});
testWidgets('Barrier is not dismissible with barrierDismissible is false', (WidgetTester tester) async {
await tester.pumpWidget(
MaterialApp(
navigatorObservers: <NavigatorObserver>[rootObserver],
home: Material(
child: Center(
child: Builder(
builder: (BuildContext context) {
return ElevatedButton(
child: const Text('X'),
onPressed: () =>
showDatePicker(
context: context,
initialDate: DateTime.now(),
firstDate: DateTime(2018),
lastDate: DateTime(2030),
barrierDismissible: false,
builder: (BuildContext context,
Widget? child) => const SizedBox(),
),
);
},
),
),
),
),
);
// Open the dialog.
await tester.tap(find.byType(ElevatedButton));
await tester.pumpAndSettle();
expect(rootObserver.datePickerCount, 1);
// Tap on the barrier, which shouldn't do anything this time.
await tester.tapAt(const Offset(10.0, 10.0));
await tester.pumpAndSettle();
expect(rootObserver.datePickerCount, 1);
});
});
testWidgets('Barrier color', (WidgetTester tester) async {
await tester.pumpWidget(
MaterialApp(
home: Material(
child: Center(
child: Builder(
builder: (BuildContext context) {
return ElevatedButton(
child: const Text('X'),
onPressed: () =>
showDatePicker(
context: context,
initialDate: DateTime.now(),
firstDate: DateTime(2018),
lastDate: DateTime(2030),
builder: (BuildContext context,
Widget? child) => const SizedBox(),
),
);
},
),
),
),
),
);
// Open the dialog.
await tester.tap(find.byType(ElevatedButton));
await tester.pumpAndSettle();
expect(tester.widget<ModalBarrier>(find.byType(ModalBarrier).last).color, Colors.black54);
// Dismiss the dialog.
await tester.tapAt(const Offset(10.0, 10.0));
await tester.pumpWidget(
MaterialApp(
home: Material(
child: Center(
child: Builder(
builder: (BuildContext context) {
return ElevatedButton(
child: const Text('X'),
onPressed: () =>
showDatePicker(
context: context,
barrierColor: Colors.pink,
initialDate: DateTime.now(),
firstDate: DateTime(2018),
lastDate: DateTime(2030),
builder: (BuildContext context,
Widget? child) => const SizedBox(),
),
);
},
),
),
),
),
);
// Open the dialog.
await tester.tap(find.byType(ElevatedButton));
await tester.pumpAndSettle();
expect(tester.widget<ModalBarrier>(find.byType(ModalBarrier).last).color, Colors.pink);
});
testWidgets('Barrier Label', (WidgetTester tester) async {
await tester.pumpWidget(
MaterialApp(
home: Material(
child: Center(
child: Builder(
builder: (BuildContext context) {
return ElevatedButton(
child: const Text('X'),
onPressed: () =>
showDatePicker(
context: context,
barrierLabel: 'Custom Label',
initialDate: DateTime.now(),
firstDate: DateTime(2018),
lastDate: DateTime(2030),
builder: (BuildContext context,
Widget? child) => const SizedBox(),
),
);
},
),
),
),
),
);
// Open the dialog.
await tester.tap(find.byType(ElevatedButton));
await tester.pumpAndSettle();
expect(tester.widget<ModalBarrier>(find.byType(ModalBarrier).last).semanticsLabel, 'Custom Label');
});
testWidgets('uses nested navigator if useRootNavigator is false', (WidgetTester tester) async { testWidgets('uses nested navigator if useRootNavigator is false', (WidgetTester tester) async {
final _DatePickerObserver rootObserver = _DatePickerObserver(); final _DatePickerObserver rootObserver = _DatePickerObserver();
final _DatePickerObserver nestedObserver = _DatePickerObserver(); final _DatePickerObserver nestedObserver = _DatePickerObserver();
...@@ -2035,4 +2219,12 @@ class _DatePickerObserver extends NavigatorObserver { ...@@ -2035,4 +2219,12 @@ class _DatePickerObserver extends NavigatorObserver {
} }
super.didPush(route, previousRoute); super.didPush(route, previousRoute);
} }
@override
void didPop(Route<dynamic> route, Route<dynamic>? previousRoute) {
if (route is DialogRoute) {
datePickerCount--;
}
super.didPop(route, previousRoute);
}
} }
packages/flutter/test/material/time_picker_test.dart
View file @ 9def8f6b
...@@ -668,6 +668,167 @@ void main() { ...@@ -668,6 +668,167 @@ void main() {
expect(tester.getBottomLeft(find.text(okString)).dx, 800 - ltrOkRight); expect(tester.getBottomLeft(find.text(okString)).dx, 800 - ltrOkRight);
}); });
group('Barrier dismissible', () {
late PickerObserver rootObserver;
setUpAll(() {
rootObserver = PickerObserver();
});
testWidgets('Barrier is dismissible with default parameter', (WidgetTester tester) async {
await tester.pumpWidget(
MaterialApp(
navigatorObservers: <NavigatorObserver>[rootObserver],
home: Material(
child: Center(
child: Builder(
builder: (BuildContext context) {
return ElevatedButton(
child: const Text('X'),
onPressed: () =>
showTimePicker(
context: context,
initialTime: const TimeOfDay(hour: 7, minute: 0),
),
);
},
),
),
),
),
);
// Open the dialog.
await tester.tap(find.byType(ElevatedButton));
await tester.pumpAndSettle();
expect(rootObserver.pickerCount, 1);
// Tap on the barrier.
await tester.tapAt(const Offset(10.0, 10.0));
await tester.pumpAndSettle();
expect(rootObserver.pickerCount, 0);
});
testWidgets('Barrier is not dismissible with barrierDismissible is false', (WidgetTester tester) async {
await tester.pumpWidget(
MaterialApp(
navigatorObservers: <NavigatorObserver>[rootObserver],
home: Material(
child: Center(
child: Builder(
builder: (BuildContext context) {
return ElevatedButton(
child: const Text('X'),
onPressed: () =>
showTimePicker(
context: context,
barrierDismissible: false,
initialTime: const TimeOfDay(hour: 7, minute: 0),
),
);
},
),
),
),
),
);
// Open the dialog.
await tester.tap(find.byType(ElevatedButton));
await tester.pumpAndSettle();
expect(rootObserver.pickerCount, 1);
// Tap on the barrier, which shouldn't do anything this time.
await tester.tapAt(const Offset(10.0, 10.0));
await tester.pumpAndSettle();
expect(rootObserver.pickerCount, 1);
});
});
testWidgets('Barrier color', (WidgetTester tester) async {
await tester.pumpWidget(
MaterialApp(
home: Material(
child: Center(
child: Builder(
builder: (BuildContext context) {
return ElevatedButton(
child: const Text('X'),
onPressed: () => showTimePicker(
context: context,
initialTime: const TimeOfDay(hour: 7, minute: 0),
),
);
},
),
),
),
),
);
// Open the dialog.
await tester.tap(find.byType(ElevatedButton));
await tester.pumpAndSettle();
expect(tester.widget<ModalBarrier>(find.byType(ModalBarrier).last).color, Colors.black54);
// Dismiss the dialog.
await tester.tapAt(const Offset(10.0, 10.0));
await tester.pumpWidget(
MaterialApp(
home: Material(
child: Center(
child: Builder(
builder: (BuildContext context) {
return ElevatedButton(
child: const Text('X'),
onPressed: () => showTimePicker(
context: context,
barrierColor: Colors.pink,
initialTime: const TimeOfDay(hour: 7, minute: 0),
),
);
},
),
),
),
),
);
// Open the dialog.
await tester.tap(find.byType(ElevatedButton));
await tester.pumpAndSettle();
expect(tester.widget<ModalBarrier>(find.byType(ModalBarrier).last).color, Colors.pink);
});
testWidgets('Barrier Label', (WidgetTester tester) async {
await tester.pumpWidget(
MaterialApp(
home: Material(
child: Center(
child: Builder(
builder: (BuildContext context) {
return ElevatedButton(
child: const Text('X'),
onPressed: () => showTimePicker(
context: context,
barrierLabel: 'Custom Label',
initialTime: const TimeOfDay(hour: 7, minute: 0),
),
);
},
),
),
),
),
);
// Open the dialog.
await tester.tap(find.byType(ElevatedButton));
await tester.pumpAndSettle();
expect(tester.widget<ModalBarrier>(find.byType(ModalBarrier).last).semanticsLabel, 'Custom Label');
});
testWidgets('uses root navigator by default', (WidgetTester tester) async { testWidgets('uses root navigator by default', (WidgetTester tester) async {
final PickerObserver rootObserver = PickerObserver(); final PickerObserver rootObserver = PickerObserver();
final PickerObserver nestedObserver = PickerObserver(); final PickerObserver nestedObserver = PickerObserver();
...@@ -1807,6 +1968,14 @@ class PickerObserver extends NavigatorObserver { ...@@ -1807,6 +1968,14 @@ class PickerObserver extends NavigatorObserver {
} }
super.didPush(route, previousRoute); super.didPush(route, previousRoute);
} }
@override
void didPop(Route<dynamic> route, Route<dynamic>? previousRoute) {
if (route is DialogRoute) {
pickerCount--;
}
super.didPop(route, previousRoute);
}
} }
Future<void> mediaQueryBoilerplate( Future<void> mediaQueryBoilerplate(
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment

HIAST - GitLab