feat(account): improve followed sources list

- Refactor UI for clarity
- Add loading and error states
- Improve navigation
- Use shared widgets

Author: fulleni

lib/account/view/manage_followed_items/sources/followed_sources_list_page.dart

@@ -2,15 +2,14 @@ import 'package:flutter/material.dart';
 import 'package:flutter_bloc/flutter_bloc.dart';
 import 'package:go_router/go_router.dart';
 import 'package:ht_main/account/bloc/account_bloc.dart';
-import 'package:ht_main/entity_details/view/entity_details_page.dart'; // Added
+import 'package:ht_main/entity_details/view/entity_details_page.dart'; // Import for Arguments
 import 'package:ht_main/l10n/l10n.dart';
 import 'package:ht_main/router/routes.dart';
-import 'package:ht_main/shared/constants/app_spacing.dart';
-import 'package:ht_main/shared/widgets/widgets.dart';
+import 'package:ht_main/shared/widgets/widgets.dart'; // For common widgets
+import 'package:ht_shared/ht_shared.dart';
 
 /// {@template followed_sources_list_page}
-/// Displays a list of sources the user is currently following.
-/// Allows unfollowing and navigating to add more sources.
+/// Page to display and manage sources followed by the user.
 /// {@endtemplate}
 class FollowedSourcesListPage extends StatelessWidget {
   /// {@macro followed_sources_list_page}
@@ -19,14 +18,16 @@ class FollowedSourcesListPage extends StatelessWidget {
   @override
   Widget build(BuildContext context) {
     final l10n = context.l10n;
+    final followedSources =
+        context.watch<AccountBloc>().state.preferences?.followedSources ?? [];
 
     return Scaffold(
       appBar: AppBar(
-        title: Text(l10n.followedSourcesPageTitle),
+        title: const Text('Followed Sources'), // Placeholder
         actions: [
           IconButton(
             icon: const Icon(Icons.add_circle_outline),
-            tooltip: l10n.addSourcesTooltip,
+            tooltip: 'Add Source to Follow', // Placeholder
             onPressed: () {
               context.goNamed(Routes.addSourceToFollowName);
             },
@@ -35,87 +36,68 @@ class FollowedSourcesListPage extends StatelessWidget {
       ),
       body: BlocBuilder<AccountBloc, AccountState>(
         builder: (context, state) {
-          if (state.status == AccountStatus.initial ||
-              (state.status == AccountStatus.loading &&
-                  state.preferences == null)) {
-            return const Center(child: CircularProgressIndicator());
+          if (state.status == AccountStatus.loading &&
+              state.preferences == null) {
+            return LoadingStateWidget(
+              icon: Icons.source_outlined,
+              headline: 'Loading Followed Sources...', // Placeholder
+              subheadline: l10n.pleaseWait, // Assuming this exists
+            );
           }
 
           if (state.status == AccountStatus.failure &&
               state.preferences == null) {
             return FailureStateWidget(
-              message: state.errorMessage ?? l10n.unknownError,
+              message: state.errorMessage ?? 'Could not load followed sources.', // Placeholder
               onRetry: () {
                 if (state.user?.id != null) {
                   context.read<AccountBloc>().add(
-                    AccountLoadContentPreferencesRequested(
-                      userId: state.user!.id,
-                    ),
-                  );
+                        AccountLoadUserPreferences( // Corrected event name
+                          userId: state.user!.id,
+                        ),
+                      );
                 }
               },
             );
           }
 
-          final followedSources = state.preferences?.followedSources;
-
-          if (followedSources == null || followedSources.isEmpty) {
-            return Center(
-              child: Padding(
-                padding: const EdgeInsets.all(AppSpacing.lg),
-                child: Column(
-                  mainAxisAlignment: MainAxisAlignment.center,
-                  children: [
-                    const Icon(Icons.source_outlined, size: 48),
-                    const SizedBox(height: AppSpacing.md),
-                    Text(
-                      l10n.noFollowedSourcesMessage,
-                      style: Theme.of(context).textTheme.titleMedium,
-                      textAlign: TextAlign.center,
-                    ),
-                    const SizedBox(height: AppSpacing.lg),
-                    ElevatedButton.icon(
-                      icon: const Icon(Icons.add_circle_outline),
-                      label: Text(l10n.addSourcesButtonLabel),
-                      onPressed: () {
-                        context.goNamed(Routes.addSourceToFollowName);
-                      },
-                    ),
-                  ],
-                ),
-              ),
+          if (followedSources.isEmpty) {
+            return const InitialStateWidget(
+              icon: Icons.no_sim_outlined, // Placeholder icon
+              headline: 'No Followed Sources', // Placeholder
+              subheadline: 'Start following sources to see them here.', // Placeholder
             );
           }
 
           return ListView.builder(
-            padding: const EdgeInsets.all(AppSpacing.md),
             itemCount: followedSources.length,
             itemBuilder: (context, index) {
               final source = followedSources[index];
-              return Card(
-                margin: const EdgeInsets.only(bottom: AppSpacing.sm),
-                child: ListTile(
-                  title: Text(source.name),
-                  onTap: () {
-                    // Added onTap for navigation
-                    context.push(
-                      Routes.sourceDetails,
-                      extra: EntityDetailsPageArguments(entity: source),
-                    );
+              return ListTile(
+                leading: const Icon(Icons.source_outlined), // Generic icon
+                title: Text(source.name),
+                subtitle: source.description != null
+                    ? Text(
+                        source.description!,
+                        maxLines: 1,
+                        overflow: TextOverflow.ellipsis,
+                      )
+                    : null,
+                trailing: IconButton(
+                  icon: const Icon(Icons.remove_circle_outline, color: Colors.red),
+                  tooltip: 'Unfollow Source', // Placeholder
+                  onPressed: () {
+                    context.read<AccountBloc>().add(
+                          AccountFollowSourceToggled(source: source),
+                        );
                   },
-                  trailing: IconButton(
-                    icon: Icon(
-                      Icons.remove_circle_outline,
-                      color: Theme.of(context).colorScheme.error,
-                    ),
-                    tooltip: l10n.unfollowSourceTooltip(source.name),
-                    onPressed: () {
-                      context.read<AccountBloc>().add(
-                        AccountFollowSourceToggled(source: source),
-                      );
-                    },
-                  ),
                 ),
+                onTap: () {
+                  context.push(
+                    Routes.sourceDetails, // Navigate to source details
+                    extra: EntityDetailsPageArguments(entity: source),
+                  );
+                },
               );
             },
           );