refactor(app_configuration): update article ad settings form

- Replace AdType with BannerAdShape for in-article ad type selection
- Update localization keys to reflect new banner ad shape selection
- Remove unnecessary comments and adjust padding for better readability
- Simplify segmented button configuration for banner ad shape

Author: fulleni

lib/app_configuration/widgets/article_ad_settings_form.dart

@@ -1,7 +1,7 @@
 import 'package:core/core.dart';
 import 'package:flutter/material.dart';
 import 'package:flutter_news_app_web_dashboard_full_source_code/l10n/l10n.dart';
-import 'package:flutter_news_app_web_dashboard_full_source_code/shared/extensions/ad_type_l10n.dart';
+import 'package:flutter_news_app_web_dashboard_full_source_code/shared/extensions/banner_ad_shape_l10n.dart';
 import 'package:flutter_news_app_web_dashboard_full_source_code/shared/extensions/in_article_ad_slot_type_l10n.dart';
 import 'package:ui_kit/ui_kit.dart';
 
@@ -64,51 +64,47 @@ class _ArticleAdSettingsFormState extends State<ArticleAdSettingsForm>
         ),
         const SizedBox(height: AppSpacing.lg),
         ExpansionTile(
-          title: Text(l10n.defaultInArticleAdTypeSelectionTitle),
+          title: Text(l10n.bannerAdShapeSelectionTitle),
           childrenPadding: const EdgeInsetsDirectional.only(
-            start: AppSpacing.lg, // Adjusted padding for hierarchy
+            start: AppSpacing.lg,
             top: AppSpacing.md,
             bottom: AppSpacing.md,
           ),
-          expandedCrossAxisAlignment:
-              CrossAxisAlignment.start, // Align content to start
+          expandedCrossAxisAlignment: CrossAxisAlignment.start,
           children: [
             Text(
-              l10n.defaultInArticleAdTypeSelectionDescription,
+              l10n.bannerAdShapeSelectionDescription,
               style: Theme.of(context).textTheme.bodySmall?.copyWith(
                 color: Theme.of(
                   context,
                 ).colorScheme.onSurface.withOpacity(0.7),
               ),
-              textAlign: TextAlign.start, // Ensure text aligns to start
+              textAlign: TextAlign.start,
             ),
             const SizedBox(height: AppSpacing.lg),
             Align(
               alignment: AlignmentDirectional.centerStart,
-              child: SegmentedButton<AdType>(
+              child: SegmentedButton<BannerAdShape>(
                 style: SegmentedButton.styleFrom(
                   shape: const RoundedRectangleBorder(
                     borderRadius: BorderRadius.zero,
                   ),
                 ),
-                segments: AdType.values
-                    .where(
-                      (type) => type == AdType.native || type == AdType.banner,
-                    )
+                segments: BannerAdShape.values
                     .map(
-                      (type) => ButtonSegment<AdType>(
+                      (type) => ButtonSegment<BannerAdShape>(
                         value: type,
                         label: Text(type.l10n(context)),
                       ),
                     )
                     .toList(),
-                selected: {articleAdConfig.defaultInArticleAdType},
+                selected: {articleAdConfig.bannerAdShape},
                 onSelectionChanged: (newSelection) {
                   widget.onConfigChanged(
                     widget.remoteConfig.copyWith(
                       adConfig: adConfig.copyWith(
                         articleAdConfiguration: articleAdConfig.copyWith(
-                          defaultInArticleAdType: newSelection.first,
+                          bannerAdShape: newSelection.first,
                         ),
                       ),
                     ),
@@ -122,12 +118,11 @@ class _ArticleAdSettingsFormState extends State<ArticleAdSettingsForm>
         ExpansionTile(
           title: Text(l10n.inArticleAdSlotPlacementsTitle),
           childrenPadding: const EdgeInsetsDirectional.only(
-            start: AppSpacing.lg, // Adjusted padding for hierarchy
+            start: AppSpacing.lg,
             top: AppSpacing.md,
             bottom: AppSpacing.md,
           ),
-          expandedCrossAxisAlignment:
-              CrossAxisAlignment.start, // Align content to start
+          expandedCrossAxisAlignment: CrossAxisAlignment.start,
           children: [
             Text(
               l10n.inArticleAdSlotPlacementsDescription,
@@ -136,7 +131,7 @@ class _ArticleAdSettingsFormState extends State<ArticleAdSettingsForm>
                   context,
                 ).colorScheme.onSurface.withOpacity(0.7),
               ),
-              textAlign: TextAlign.start, // Ensure text aligns to start
+              textAlign: TextAlign.start,
             ),
             const SizedBox(height: AppSpacing.lg),
             ...articleAdConfig.inArticleAdSlotConfigurations.map(