Skip to content

Commit

Permalink
Add missing semantic header and close button on search provider dialog.
Browse files Browse the repository at this point in the history
  • Loading branch information
amugofjava committed Jul 21, 2024
1 parent 4ca974b commit 0c55da3
Showing 1 changed file with 19 additions and 2 deletions.
21 changes: 19 additions & 2 deletions lib/ui/settings/search_provider.dart
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@
import 'package:anytime/bloc/settings/settings_bloc.dart';
import 'package:anytime/entities/app_settings.dart';
import 'package:anytime/l10n/L.dart';
import 'package:anytime/ui/widgets/action_text.dart';
import 'package:flutter/material.dart';
import 'package:flutter_dialogs/flutter_dialogs.dart';
import 'package:provider/provider.dart';
Expand Down Expand Up @@ -45,8 +46,11 @@ class _SearchProviderWidgetState extends State<SearchProviderWidget> {
useRootNavigator: false,
builder: (BuildContext context) {
return AlertDialog(
title: Text(L.of(context)!.search_provider_label,
style: Theme.of(context).textTheme.titleMedium, textAlign: TextAlign.center),
title: Semantics(
header: true,
child: Text(L.of(context)!.search_provider_label,
style: Theme.of(context).textTheme.titleMedium, textAlign: TextAlign.center),
),
content: StatefulBuilder(
builder: (BuildContext context, StateSetter setState) {
return Column(mainAxisSize: MainAxisSize.min, children: <Widget>[
Expand Down Expand Up @@ -86,6 +90,19 @@ class _SearchProviderWidgetState extends State<SearchProviderWidget> {
});
},
),
SimpleDialogOption(
padding: const EdgeInsets.symmetric(vertical: 16.0, horizontal: 24.0),
// child: Text(L.of(context)!.close_button_label),
child: Align(
alignment: Alignment.centerRight,
child: TextButton(
child: ActionText(L.of(context)!.close_button_label),
onPressed: () {
Navigator.pop(context, '');
},
),
),
),
]);
},
));
Expand Down

0 comments on commit 0c55da3

Please sign in to comment.