Skip to content

Commit a574df5

Browse files
authored
[mob][photos] Fix changelog dialog layout (#7851)
## Description Shrink Changelog if needed. ## Tests - [x] Tested on simulator <img width="200" src="https://github.com/user-attachments/assets/fc50ceb4-7bd4-4ee4-a30d-04efbb5521f9" /> <img width="200" src="https://github.com/user-attachments/assets/9254a604-2b92-4329-8485-30e7edac469e" />
2 parents f5a8560 + 2fc68e6 commit a574df5

File tree

1 file changed

+60
-66
lines changed

1 file changed

+60
-66
lines changed

mobile/apps/photos/lib/ui/notification/update/change_log_page.dart

Lines changed: 60 additions & 66 deletions
Original file line numberDiff line numberDiff line change
@@ -22,76 +22,69 @@ class _ChangeLogPageState extends State<ChangeLogPage> {
2222
@override
2323
Widget build(BuildContext context) {
2424
final enteColorScheme = getEnteColorScheme(context);
25-
return Scaffold(
26-
appBar: null,
27-
body: Container(
28-
color: enteColorScheme.backgroundElevated,
29-
child: Column(
30-
mainAxisAlignment: MainAxisAlignment.center,
31-
children: [
32-
const SizedBox(
33-
height: 36,
34-
),
35-
Container(
36-
alignment: Alignment.centerLeft,
37-
child: Padding(
38-
padding: const EdgeInsets.symmetric(horizontal: 16.0),
39-
child: TitleBarTitleWidget(
40-
title: AppLocalizations.of(context).whatsNew,
41-
),
25+
return Material(
26+
color: enteColorScheme.backgroundElevated,
27+
child: Column(
28+
mainAxisSize: MainAxisSize.min,
29+
mainAxisAlignment: MainAxisAlignment.center,
30+
children: [
31+
const SizedBox(height: 36),
32+
Container(
33+
alignment: Alignment.centerLeft,
34+
child: Padding(
35+
padding: const EdgeInsets.symmetric(horizontal: 16.0),
36+
child: TitleBarTitleWidget(
37+
title: AppLocalizations.of(context).whatsNew,
4238
),
4339
),
44-
const SizedBox(
45-
height: 24,
46-
),
47-
Expanded(child: _getChangeLog(context)),
48-
const DividerWidget(
49-
dividerType: DividerType.solid,
50-
),
51-
SafeArea(
52-
child: Padding(
53-
padding: const EdgeInsets.only(
54-
left: 16.0,
55-
right: 16,
56-
top: 16,
57-
bottom: 8,
58-
),
59-
child: Column(
60-
crossAxisAlignment: CrossAxisAlignment.start,
61-
children: [
62-
ButtonWidget(
63-
buttonType: ButtonType.trailingIconPrimary,
64-
buttonSize: ButtonSize.large,
65-
labelText: AppLocalizations.of(context).continueLabel,
66-
icon: Icons.arrow_forward_outlined,
67-
onTap: () async {
68-
await updateService.hideChangeLog();
69-
if (mounted && Navigator.of(context).canPop()) {
70-
Navigator.of(context).pop();
71-
}
72-
},
73-
),
74-
const SizedBox(
75-
height: 8,
76-
),
77-
ButtonWidget(
78-
buttonType: ButtonType.trailingIconSecondary,
79-
buttonSize: ButtonSize.large,
80-
labelText: AppLocalizations.of(context).rateUs,
81-
icon: Icons.favorite_rounded,
82-
iconColor: enteColorScheme.primary500,
83-
onTap: () async {
84-
await updateService.launchReviewUrl();
85-
},
86-
),
87-
const SizedBox(height: 8),
88-
const SizedBox(height: 8),
89-
],
90-
),
40+
),
41+
const SizedBox(
42+
height: 24,
43+
),
44+
Flexible(child: _getChangeLog(context)),
45+
const DividerWidget(
46+
dividerType: DividerType.solid,
47+
),
48+
SafeArea(
49+
child: Padding(
50+
padding: const EdgeInsets.only(
51+
left: 16.0,
52+
right: 16,
53+
top: 16,
54+
bottom: 8,
55+
),
56+
child: Column(
57+
crossAxisAlignment: CrossAxisAlignment.start,
58+
children: [
59+
ButtonWidget(
60+
buttonType: ButtonType.trailingIconPrimary,
61+
buttonSize: ButtonSize.large,
62+
labelText: AppLocalizations.of(context).continueLabel,
63+
icon: Icons.arrow_forward_outlined,
64+
onTap: () async {
65+
await updateService.hideChangeLog();
66+
if (mounted && Navigator.of(context).canPop()) {
67+
Navigator.of(context).pop();
68+
}
69+
},
70+
),
71+
const SizedBox(height: 8),
72+
ButtonWidget(
73+
buttonType: ButtonType.trailingIconSecondary,
74+
buttonSize: ButtonSize.large,
75+
labelText: AppLocalizations.of(context).rateUs,
76+
icon: Icons.favorite_rounded,
77+
iconColor: enteColorScheme.primary500,
78+
onTap: () async {
79+
await updateService.launchReviewUrl();
80+
},
81+
),
82+
const SizedBox(height: 16),
83+
],
9184
),
9285
),
93-
],
94-
),
86+
),
87+
],
9588
),
9689
);
9790
}
@@ -118,6 +111,7 @@ class _ChangeLogPageState extends State<ChangeLogPage> {
118111
thumbVisibility: true,
119112
thickness: 2.0,
120113
child: ListView.builder(
114+
shrinkWrap: true,
121115
physics: const BouncingScrollPhysics(),
122116
itemBuilder: (context, index) {
123117
return Padding(

0 commit comments

Comments
 (0)