Refactor/local ads management structure enhancement and UI overhaul

lib/app/view/app.dart

@@ -16,6 +16,8 @@ import 'package:flutter_news_app_web_dashboard_full_source_code/content_manageme
 import 'package:flutter_news_app_web_dashboard_full_source_code/content_management/bloc/sources_filter/sources_filter_bloc.dart';
 import 'package:flutter_news_app_web_dashboard_full_source_code/content_management/bloc/topics_filter/topics_filter_bloc.dart';
 import 'package:flutter_news_app_web_dashboard_full_source_code/l10n/app_localizations.dart';
+import 'package:flutter_news_app_web_dashboard_full_source_code/local_ads_management/bloc/filter_local_ads/filter_local_ads_bloc.dart';
+import 'package:flutter_news_app_web_dashboard_full_source_code/local_ads_management/bloc/local_ads_management_bloc.dart';
 import 'package:flutter_news_app_web_dashboard_full_source_code/overview/bloc/overview_bloc.dart';
 import 'package:flutter_news_app_web_dashboard_full_source_code/router/router.dart';
 import 'package:flutter_news_app_web_dashboard_full_source_code/shared/services/pending_deletions_service.dart';
@@ -131,6 +133,9 @@ class App extends StatelessWidget {
           BlocProvider(
             create: (context) => SourcesFilterBloc(),
           ),
+          BlocProvider(
+            create: (context) => FilterLocalAdsBloc(),
+          ),
           BlocProvider(
             create: (context) => ContentManagementBloc(
               headlinesRepository: context.read<DataRepository<Headline>>(),
@@ -142,6 +147,13 @@ class App extends StatelessWidget {
               pendingDeletionsService: context.read<PendingDeletionsService>(),
             ),
           ),
+          BlocProvider(
+            create: (context) => LocalAdsManagementBloc(
+              localAdsRepository: context.read<DataRepository<LocalAd>>(),
+              filterLocalAdsBloc: context.read<FilterLocalAdsBloc>(),
+              pendingDeletionsService: context.read<PendingDeletionsService>(),
+            ),
+          ),
           BlocProvider(
             create: (context) => OverviewBloc(
               dashboardSummaryRepository: context

lib/app/view/app_shell.dart

@@ -56,21 +56,25 @@ class AppShell extends StatelessWidget {
             label: l10n.appConfiguration,
           ),
         ],
-        leadingUnextendedNavRail: const Padding(
-          padding: EdgeInsets.symmetric(vertical: AppSpacing.lg),
-          child: Icon(Icons.newspaper_outlined),
+        leadingUnextendedNavRail: Padding(
+          padding: const EdgeInsets.symmetric(vertical: AppSpacing.lg),
+          child: Icon(
+            Icons.newspaper_outlined,
+            color: theme.colorScheme.primary,
+          ),
         ),
         leadingExtendedNavRail: Padding(
           padding: const EdgeInsets.all(AppSpacing.lg),
           child: Row(
             children: [
-              const Icon(Icons.newspaper_outlined),
+              Icon(
+                Icons.newspaper_outlined,
+                color: theme.colorScheme.primary,
+              ),
               const SizedBox(width: AppSpacing.md),
               Text(
                 l10n.dashboardTitle,
-                style: theme.textTheme.titleLarge?.copyWith(
-                  color: theme.colorScheme.primary,
-                ),
+                style: theme.textTheme.titleLarge,
               ),
             ],
           ),

lib/app_configuration/widgets/article_ad_settings_form.dart

@@ -202,7 +202,7 @@ class _ArticleAdSettingsFormState extends State<ArticleAdSettingsForm>
     return Column(
       children: [
         SwitchListTile(
-          title: Text(l10n.enableInArticleAdsForRoleLabel(role.l10n(context))),
+          title: Text(l10n.visibleToRoleLabel(role.l10n(context))),
           value: roleSlots != null,
           onChanged: (value) {
             final newVisibleTo =

lib/app_configuration/widgets/feed_decorator_form.dart

@@ -190,7 +190,6 @@ class _FeedDecoratorFormState extends State<FeedDecoratorForm>
             controller: _itemsToDisplayController,
           ),
         const SizedBox(height: AppSpacing.lg),
-        // Replaced SegmentedButton with TabBar for role selection
         Align(
           alignment: AlignmentDirectional.centerStart,
           child: SizedBox(

lib/app_configuration/widgets/user_preference_limits_form.dart

@@ -149,7 +149,6 @@ class _UserPreferenceLimitsFormState extends State<UserPreferenceLimitsForm>
           ),
         ),
         const SizedBox(height: AppSpacing.lg),
-        // Replaced SegmentedButton with TabBar for role selection
         Align(
           alignment: AlignmentDirectional.centerStart,
           child: SizedBox(

lib/content_management/view/headlines_page.dart

@@ -176,7 +176,7 @@ class _HeadlinesPageState extends State<HeadlinesPage> {
                       headingRowHeight: 56,
                       dataRowHeight: 56,
                       columnSpacing: AppSpacing.sm,
-                      horizontalMargin: AppSpacing.md,
+                      horizontalMargin: AppSpacing.sm,
                     );
                   },
                 ),

lib/content_management/view/sources_page.dart

@@ -126,10 +126,11 @@ class _SourcesPageState extends State<SourcesPage> {
                           label: Text(l10n.sourceName),
                           size: ColumnSize.L,
                         ),
-                        DataColumn2(
-                          label: Text(l10n.sourceType),
-                          size: ColumnSize.S,
-                        ),
+                        if (!isMobile)
+                          DataColumn2(
+                            label: Text(l10n.sourceType),
+                            size: ColumnSize.S,
+                          ),
                         DataColumn2(
                           label: Text(l10n.lastUpdated),
                           size: ColumnSize.S,
@@ -173,8 +174,8 @@ class _SourcesPageState extends State<SourcesPage> {
                       fit: FlexFit.tight,
                       headingRowHeight: 56,
                       dataRowHeight: 56,
-                      columnSpacing: AppSpacing.md,
-                      horizontalMargin: AppSpacing.md,
+                      columnSpacing: AppSpacing.sm,
+                      horizontalMargin: AppSpacing.sm,
                     );
                   },
                 ),
@@ -225,13 +226,14 @@ class _SourcesDataSource extends DataTableSource {
             overflow: TextOverflow.ellipsis,
           ),
         ),
-        DataCell(
-          Text(
-            source.sourceType.localizedName(l10n),
-            maxLines: 2,
-            overflow: TextOverflow.ellipsis,
+        if (!isMobile)
+          DataCell(
+            Text(
+              source.sourceType.localizedName(l10n),
+              maxLines: 2,
+              overflow: TextOverflow.ellipsis,
+            ),
           ),
-        ),
         DataCell(
           Text(
             DateFormat('dd-MM-yyyy').format(source.updatedAt.toLocal()),

lib/content_management/view/topics_page.dart

@@ -165,8 +165,8 @@ class _TopicPageState extends State<TopicPage> {
                       fit: FlexFit.tight,
                       headingRowHeight: 56,
                       dataRowHeight: 56,
-                      columnSpacing: AppSpacing.md,
-                      horizontalMargin: AppSpacing.md,
+                      columnSpacing: AppSpacing.sm,
+                      horizontalMargin: AppSpacing.sm,
                     );
                   },
                 ),

lib/l10n/app_localizations.dart

@@ -2725,6 +2725,108 @@ abstract class AppLocalizations {
   /// In en, this message translates to:
   /// **'Close'**
   String get closeButtonText;
+
+  /// Title for the filter dialog when filtering local ads.
+  ///
+  /// In en, this message translates to:
+  /// **'Filter Local Ads'**
+  String get filterLocalAds;
+
+  /// Hint text for the ad title or URL search field.
+  ///
+  /// In en, this message translates to:
+  /// **'Search by ad title or URL...'**
+  String get searchByAdTitleOrUrl;
+
+  /// Label for the ad type filter.
+  ///
+  /// In en, this message translates to:
+  /// **'Ad Type'**
+  String get adType;
+
+  /// Headline for loading state of native ads.
+  ///
+  /// In en, this message translates to:
+  /// **'Loading Native Ads'**
+  String get loadingNativeAds;
+
+  /// Message when no native ads are found.
+  ///
+  /// In en, this message translates to:
+  /// **'No native ads found.'**
+  String get noNativeAdsFound;
+
+  /// Headline for loading state of banner ads.
+  ///
+  /// In en, this message translates to:
+  /// **'Loading Banner Ads'**
+  String get loadingBannerAds;
+
+  /// Message when no banner ads are found.
+  ///
+  /// In en, this message translates to:
+  /// **'No banner ads found.'**
+  String get noBannerAdsFound;
+
+  /// Column header for ad image URL.
+  ///
+  /// In en, this message translates to:
+  /// **'Image URL'**
+  String get adImageUrl;
+
+  /// Headline for loading state of interstitial ads.
+  ///
+  /// In en, this message translates to:
+  /// **'Loading Interstitial Ads'**
+  String get loadingInterstitialAds;
+
+  /// Message when no interstitial ads are found.
+  ///
+  /// In en, this message translates to:
+  /// **'No interstitial ads found.'**
+  String get noInterstitialAdsFound;
+
+  /// Headline for loading state of video ads.
+  ///
+  /// In en, this message translates to:
+  /// **'Loading Video Ads'**
+  String get loadingVideoAds;
+
+  /// Message when no video ads are found.
+  ///
+  /// In en, this message translates to:
+  /// **'No video ads found.'**
+  String get noVideoAdsFound;
+
+  /// Column header for ad video URL.
+  ///
+  /// In en, this message translates to:
+  /// **'Video URL'**
+  String get adVideoUrl;
+
+  /// Snackbar message displayed when an ad ID is successfully copied to the clipboard.
+  ///
+  /// In en, this message translates to:
+  /// **'ID copied to clipboard'**
+  String get idCopied;
+
+  /// Tooltip for the copy ID action button.
+  ///
+  /// In en, this message translates to:
+  /// **'Copy Ad ID'**
+  String get copyIdTooltip;
+
+  /// Title for the dialog asking to save an ad
+  ///
+  /// In en, this message translates to:
+  /// **'Save Ad'**
+  String get saveAdTitle;
+
+  /// Message for the dialog asking to save an ad
+  ///
+  /// In en, this message translates to:
+  /// **'Do you want to publish this ad or save it as a draft?'**
+  String get saveAdMessage;
 }
 
 class _AppLocalizationsDelegate

lib/l10n/app_localizations_ar.dart

@@ -1454,4 +1454,55 @@ class AppLocalizationsAr extends AppLocalizations {
 
   @override
   String get closeButtonText => 'إغلاق';
+
+  @override
+  String get filterLocalAds => 'تصفية الإعلانات المحلية';
+
+  @override
+  String get searchByAdTitleOrUrl => 'البحث بعنوان الإعلان أو الرابط...';
+
+  @override
+  String get adType => 'نوع الإعلان';
+
+  @override
+  String get loadingNativeAds => 'جاري تحميل الإعلانات الأصلية';
+
+  @override
+  String get noNativeAdsFound => 'لم يتم العثور على إعلانات أصلية.';
+
+  @override
+  String get loadingBannerAds => 'جاري تحميل إعلانات البانر';
+
+  @override
+  String get noBannerAdsFound => 'لم يتم العثور على إعلانات بانر.';
+
+  @override
+  String get adImageUrl => 'رابط الصورة';
+
+  @override
+  String get loadingInterstitialAds => 'جاري تحميل الإعلانات البينية';
+
+  @override
+  String get noInterstitialAdsFound => 'لم يتم العثور على إعلانات بينية.';
+
+  @override
+  String get loadingVideoAds => 'جاري تحميل إعلانات الفيديو';
+
+  @override
+  String get noVideoAdsFound => 'لم يتم العثور على إعلانات فيديو.';
+
+  @override
+  String get adVideoUrl => 'رابط الفيديو';
+
+  @override
+  String get idCopied => 'تم نسخ المعرف إلى الحافظة';
+
+  @override
+  String get copyIdTooltip => 'نسخ معرف الإعلان';
+
+  @override
+  String get saveAdTitle => 'حفظ الإعلان';
+
+  @override
+  String get saveAdMessage => 'هل تريد نشر هذا الإعلان أم حفظه كمسودة؟';
 }

lib/l10n/app_localizations_en.dart

@@ -1459,4 +1459,56 @@ class AppLocalizationsEn extends AppLocalizations {
 
   @override
   String get closeButtonText => 'Close';
+
+  @override
+  String get filterLocalAds => 'Filter Local Ads';
+
+  @override
+  String get searchByAdTitleOrUrl => 'Search by ad title or URL...';
+
+  @override
+  String get adType => 'Ad Type';
+
+  @override
+  String get loadingNativeAds => 'Loading Native Ads';
+
+  @override
+  String get noNativeAdsFound => 'No native ads found.';
+
+  @override
+  String get loadingBannerAds => 'Loading Banner Ads';
+
+  @override
+  String get noBannerAdsFound => 'No banner ads found.';
+
+  @override
+  String get adImageUrl => 'Image URL';
+
+  @override
+  String get loadingInterstitialAds => 'Loading Interstitial Ads';
+
+  @override
+  String get noInterstitialAdsFound => 'No interstitial ads found.';
+
+  @override
+  String get loadingVideoAds => 'Loading Video Ads';
+
+  @override
+  String get noVideoAdsFound => 'No video ads found.';
+
+  @override
+  String get adVideoUrl => 'Video URL';
+
+  @override
+  String get idCopied => 'ID copied to clipboard';
+
+  @override
+  String get copyIdTooltip => 'Copy Ad ID';
+
+  @override
+  String get saveAdTitle => 'Save Ad';
+
+  @override
+  String get saveAdMessage =>
+      'Do you want to publish this ad or save it as a draft?';
 }