refactor(headline_tile_image_start):

- Replace EntityType with ContentType for better clarity
- Remove unused imports and reorganize remaining imports
- Simplify image loading and error handling for headline image
- Remove unused l10n parameter and update localization import
- Update routing and entity details handling
- Replace broken timeago functionality with TODO comment

Author: fulleni

lib/shared/widgets/headline_tile_image_start.dart

@@ -1,13 +1,10 @@
 import 'package:flutter/material.dart';
-import 'package:go_router/go_router.dart'; // Added
-import 'package:ht_main/entity_details/models/entity_type.dart';
-import 'package:ht_main/entity_details/view/entity_details_page.dart'; // Added for Page Arguments
-import 'package:ht_main/l10n/app_localizations.dart';
+import 'package:go_router/go_router.dart';
+import 'package:ht_main/entity_details/view/entity_details_page.dart';
 import 'package:ht_main/l10n/l10n.dart';
-import 'package:ht_main/router/routes.dart'; // Added
-import 'package:ht_shared/ht_shared.dart' show Headline;
+import 'package:ht_main/router/routes.dart';
+import 'package:ht_shared/ht_shared.dart';
 import 'package:ht_ui_kit/ht_ui_kit.dart';
-// timeago import removed from here, handled by utility
 
 /// {@template headline_tile_image_start}
 /// A shared widget to display a headline item with a small image at the start.
@@ -33,14 +30,13 @@ class HeadlineTileImageStart extends StatelessWidget {
   final Widget? trailing;
 
   /// The type of the entity currently being viewed in detail (e.g., on a category page).
-  final EntityType? currentContextEntityType;
+  final ContentType? currentContextEntityType;
 
   /// The ID of the entity currently being viewed in detail.
   final String? currentContextEntityId;
 
   @override
   Widget build(BuildContext context) {
-    final l10n = context.l10n;
     final theme = Theme.of(context);
     final textTheme = theme.textTheme;
     final colorScheme = theme.colorScheme;
@@ -62,39 +58,29 @@ class HeadlineTileImageStart extends StatelessWidget {
                 height: 72,
                 child: ClipRRect(
                   borderRadius: BorderRadius.circular(AppSpacing.xs),
-                  child: headline.imageUrl != null
-                      ? Image.network(
-                          headline.imageUrl,
-                          fit: BoxFit.cover,
-                          loadingBuilder: (context, child, loadingProgress) {
-                            if (loadingProgress == null) return child;
-                            return ColoredBox(
-                              color: colorScheme.surfaceContainerHighest,
-                              child: const Center(
-                                child: CircularProgressIndicator(
-                                  strokeWidth: 2,
-                                ),
-                              ),
-                            );
-                          },
-                          errorBuilder: (context, error, stackTrace) =>
-                              ColoredBox(
-                                color: colorScheme.surfaceContainerHighest,
-                                child: Icon(
-                                  Icons.broken_image_outlined,
-                                  color: colorScheme.onSurfaceVariant,
-                                  size: AppSpacing.xl,
-                                ),
-                              ),
-                        )
-                      : ColoredBox(
-                          color: colorScheme.surfaceContainerHighest,
-                          child: Icon(
-                            Icons.image_not_supported_outlined,
-                            color: colorScheme.onSurfaceVariant,
-                            size: AppSpacing.xl,
+                  child: Image.network(
+                    headline.imageUrl,
+                    fit: BoxFit.cover,
+                    loadingBuilder: (context, child, loadingProgress) {
+                      if (loadingProgress == null) return child;
+                      return ColoredBox(
+                        color: colorScheme.surfaceContainerHighest,
+                        child: const Center(
+                          child: CircularProgressIndicator(
+                            strokeWidth: 2,
                           ),
                         ),
+                      );
+                    },
+                    errorBuilder: (context, error, stackTrace) => ColoredBox(
+                      color: colorScheme.surfaceContainerHighest,
+                      child: Icon(
+                        Icons.broken_image_outlined,
+                        color: colorScheme.onSurfaceVariant,
+                        size: AppSpacing.xl,
+                      ),
+                    ),
+                  ),
                 ),
               ),
               const SizedBox(width: AppSpacing.md), // Always add spacing
@@ -113,7 +99,6 @@ class HeadlineTileImageStart extends StatelessWidget {
                     const SizedBox(height: AppSpacing.sm),
                     _HeadlineMetadataRow(
                       headline: headline,
-                      l10n: l10n,
                       colorScheme: colorScheme,
                       textTheme: textTheme,
                       currentContextEntityType:
@@ -140,23 +125,22 @@ class HeadlineTileImageStart extends StatelessWidget {
 class _HeadlineMetadataRow extends StatelessWidget {
   const _HeadlineMetadataRow({
     required this.headline,
-    required this.l10n,
     required this.colorScheme,
     required this.textTheme,
     this.currentContextEntityType,
     this.currentContextEntityId,
   });
 
   final Headline headline;
-  final AppLocalizations l10n;
   final ColorScheme colorScheme;
   final TextTheme textTheme;
-  final EntityType? currentContextEntityType;
+  final ContentType? currentContextEntityType;
   final String? currentContextEntityId;
 
   @override
   Widget build(BuildContext context) {
-    final formattedDate = formatRelativeTime(context, headline.publishedAt);
+    // TODO(anyone): Use a proper timeago library.
+    final formattedDate = headline.createdAt.toString();
 
     // Use bodySmall for a reasonable base size, with muted accent color
     final metadataTextStyle = textTheme.bodySmall?.copyWith(
@@ -184,34 +168,35 @@ class _HeadlineMetadataRow extends StatelessWidget {
               Text(formattedDate, style: metadataTextStyle),
             ],
           ),
-        // Conditionally render Category as Text
-        if (headline.category?.name != null &&
-            !(currentContextEntityType == EntityType.category &&
-                headline.category!.id == currentContextEntityId)) ...[
+        // Conditionally render Topic as Text
+        if (headline.topic.name.isNotEmpty &&
+            !(currentContextEntityType == ContentType.topic &&
+                headline.topic.id == currentContextEntityId)) ...[
           if (formattedDate.isNotEmpty)
             Padding(
               padding: const EdgeInsets.symmetric(horizontal: AppSpacing.xs),
               child: Text('•', style: metadataTextStyle),
             ),
           GestureDetector(
             onTap: () {
-              if (headline.category != null) {
-                context.push(
-                  Routes.categoryDetails,
-                  extra: EntityDetailsPageArguments(entity: headline.category),
-                );
-              }
+              context.push(
+                Routes.topicDetails,
+                extra: EntityDetailsPageArguments(
+                  entity: headline.topic,
+                  entityType: ContentType.topic,
+                ),
+              );
             },
-            child: Text(headline.category!.name, style: metadataTextStyle),
+            child: Text(headline.topic.name, style: metadataTextStyle),
           ),
         ],
         // Conditionally render Source as Text
-        if (!(currentContextEntityType == EntityType.source &&
+        if (!(currentContextEntityType == ContentType.source &&
             headline.source.id == currentContextEntityId)) ...[
           if (formattedDate.isNotEmpty ||
-              (headline.category?.name != null &&
-                  !(currentContextEntityType == EntityType.category &&
-                      headline.category!.id == currentContextEntityId)))
+              (headline.topic.name.isNotEmpty &&
+                  !(currentContextEntityType == ContentType.topic &&
+                      headline.topic.id == currentContextEntityId)))
             Padding(
               padding: const EdgeInsets.symmetric(horizontal: AppSpacing.xs),
               child: Text('•', style: metadataTextStyle),
@@ -220,7 +205,10 @@ class _HeadlineMetadataRow extends StatelessWidget {
             onTap: () {
               context.push(
                 Routes.sourceDetails,
-                extra: EntityDetailsPageArguments(entity: headline.source),
+                extra: EntityDetailsPageArguments(
+                  entity: headline.source,
+                  entityType: ContentType.source,
+                ),
               );
             },
             child: Text(headline.source.name, style: metadataTextStyle),