Skip to content

Commit c9cf9e3

Browse files
committed
feat(dialog): improve layout and styling of detail dialogs
- Add max width and height constraints to dialog content - Enhance styling for no reason provided text - Adjust font styles for report details
1 parent e667796 commit c9cf9e3

File tree

3 files changed

+20
-9
lines changed

3 files changed

+20
-9
lines changed

lib/community_management/widgets/app_review_details_dialog.dart

Lines changed: 12 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -13,14 +13,23 @@ class AppReviewDetailsDialog extends StatelessWidget {
1313
@override
1414
Widget build(BuildContext context) {
1515
final l10n = AppLocalizations.of(context);
16+
final theme = Theme.of(context);
1617
final details = appReview.feedbackDetails;
1718

1819
return AlertDialog(
1920
title: Text(l10n.feedbackDetails),
20-
content: SizedBox(
21-
width: double.maxFinite,
21+
content: ConstrainedBox(
22+
constraints: const BoxConstraints(maxWidth: 400, maxHeight: 300),
2223
child: SingleChildScrollView(
23-
child: Text(details ?? l10n.noReasonProvided),
24+
child: Text(
25+
details ?? l10n.noReasonProvided,
26+
style: (details == null)
27+
? theme.textTheme.bodyLarge?.copyWith(
28+
fontStyle: FontStyle.italic,
29+
color: theme.colorScheme.onSurface.withOpacity(0.6),
30+
)
31+
: theme.textTheme.bodyLarge,
32+
),
2433
),
2534
),
2635
actions: [

lib/community_management/widgets/engagement_details_dialog.dart

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,8 +24,8 @@ class EngagementDetailsDialog extends StatelessWidget {
2424

2525
return AlertDialog(
2626
title: Text(l10n.commentDetails),
27-
content: SizedBox(
28-
width: double.maxFinite,
27+
content: ConstrainedBox(
28+
constraints: const BoxConstraints(maxWidth: 400, maxHeight: 300),
2929
child: SingleChildScrollView(
3030
child: Text(
3131
engagement.comment?.content ?? l10n.noReasonProvided,

lib/community_management/widgets/report_details_dialog.dart

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -23,23 +23,25 @@ class ReportDetailsDialog extends StatelessWidget {
2323

2424
return AlertDialog(
2525
title: Text(l10n.reportDetails),
26-
content: SizedBox(
27-
width: double.maxFinite,
26+
content: ConstrainedBox(
27+
constraints: const BoxConstraints(maxWidth: 400, maxHeight: 300),
2828
child: SingleChildScrollView(
2929
child: Column(
3030
crossAxisAlignment: CrossAxisAlignment.start,
3131
mainAxisSize: MainAxisSize.min,
3232
children: [
3333
Text(
3434
'${l10n.reason}: ${report.reason.l10n(context)}',
35-
style: theme.textTheme.titleMedium,
35+
style: theme.textTheme.titleSmall?.copyWith(
36+
fontWeight: FontWeight.bold,
37+
),
3638
),
3739
if (report.additionalComments != null &&
3840
report.additionalComments!.isNotEmpty) ...[
3941
const SizedBox(height: AppSpacing.md),
4042
Text(
4143
l10n.comment,
42-
style: theme.textTheme.titleSmall,
44+
style: theme.textTheme.labelMedium,
4345
),
4446
const SizedBox(height: AppSpacing.xs),
4547
Text(

0 commit comments

Comments
 (0)