Skip to content

Commit 2ded90e

Browse files
committed
style(app_configuration): enhance advertisements config layout
- Adjust ExpansionTile children padding for better hierarchy visualization - Add top and bottom padding to ExpansionTile for improved layout - Set expandedCrossAxisAlignment to start for consistent content alignment
1 parent c7680eb commit 2ded90e

File tree

1 file changed

+12
-3
lines changed

1 file changed

+12
-3
lines changed

lib/app_configuration/view/tabs/advertisements_configuration_tab.dart

Lines changed: 12 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -36,8 +36,11 @@ class AdvertisementsConfigurationTab extends StatelessWidget {
3636
ExpansionTile(
3737
title: Text(l10n.adPlatformConfigurationTitle),
3838
childrenPadding: const EdgeInsetsDirectional.only(
39-
start: AppSpacing.xxl,
39+
start: AppSpacing.lg, // Adjusted padding for hierarchy
40+
top: AppSpacing.md,
41+
bottom: AppSpacing.md,
4042
),
43+
expandedCrossAxisAlignment: CrossAxisAlignment.start, // Align content to start
4144
children: [
4245
AdPlatformConfigForm(
4346
remoteConfig: remoteConfig,
@@ -50,8 +53,11 @@ class AdvertisementsConfigurationTab extends StatelessWidget {
5053
ExpansionTile(
5154
title: Text(l10n.feedAdSettingsTitle),
5255
childrenPadding: const EdgeInsetsDirectional.only(
53-
start: AppSpacing.xxl,
56+
start: AppSpacing.lg, // Adjusted padding for hierarchy
57+
top: AppSpacing.md,
58+
bottom: AppSpacing.md,
5459
),
60+
expandedCrossAxisAlignment: CrossAxisAlignment.start, // Align content to start
5561
children: [
5662
FeedAdSettingsForm(
5763
remoteConfig: remoteConfig,
@@ -64,8 +70,11 @@ class AdvertisementsConfigurationTab extends StatelessWidget {
6470
ExpansionTile(
6571
title: Text(l10n.articleAdSettingsTitle),
6672
childrenPadding: const EdgeInsetsDirectional.only(
67-
start: AppSpacing.xxl,
73+
start: AppSpacing.lg, // Adjusted padding for hierarchy
74+
top: AppSpacing.md,
75+
bottom: AppSpacing.md,
6876
),
77+
expandedCrossAxisAlignment: CrossAxisAlignment.start, // Align content to start
6978
children: [
7079
ArticleAdSettingsForm(
7180
remoteConfig: remoteConfig,

0 commit comments

Comments
 (0)