refactor(headline): Improve metadata display

- Use utils for relative time
- Compacted metadata info
- Use chips for category/source

Author: fulleni

lib/shared/widgets/headline_tile_text_only.dart

@@ -1,8 +1,9 @@
 import 'package:flutter/material.dart';
 import 'package:ht_main/l10n/l10n.dart';
 import 'package:ht_main/shared/constants/app_spacing.dart';
+import 'package:ht_main/shared/utils/utils.dart'; // Import the new utility
 import 'package:ht_shared/ht_shared.dart' show Headline;
-import 'package:timeago/timeago.dart' as timeago;
+// timeago import removed from here, handled by utility
 
 /// {@template headline_tile_text_only}
 /// A widget to display a headline item with text only.
@@ -96,109 +97,100 @@ class _HeadlineMetadataRow extends StatelessWidget {
 
   @override
   Widget build(BuildContext context) {
-    final String formattedDate;
-    if (headline.publishedAt != null) {
-      formattedDate = timeago.format(
-        headline.publishedAt!,
-        locale: Localizations.localeOf(context).languageCode,
-      );
-    } else {
-      formattedDate = '';
-    }
+    final formattedDate = formatRelativeTime(context, headline.publishedAt);
 
     final metadataStyle = textTheme.bodySmall?.copyWith(
       color: colorScheme.onSurfaceVariant,
     );
-    const iconSize = 12.0;
+    final chipLabelStyle = textTheme.labelSmall?.copyWith(
+      color: colorScheme.onSurfaceVariant,
+    );
+    final chipBackgroundColor =
+        colorScheme.surfaceContainerHighest.withOpacity(0.5);
+    const iconSize = 12.0; // Kept for date icon
 
     return Wrap(
-      spacing: AppSpacing.md, // Spacing between items in the row
-      runSpacing: AppSpacing.xs, // Spacing if items wrap
+      spacing: AppSpacing.sm, // Reduced spacing for more compactness
+      runSpacing: AppSpacing.xs,
       crossAxisAlignment: WrapCrossAlignment.center,
       children: [
-        if (headline.category?.name != null)
+        if (formattedDate.isNotEmpty)
           GestureDetector(
             onTap: () {
               ScaffoldMessenger.of(context)
                 ..hideCurrentSnackBar()
                 ..showSnackBar(
-                  SnackBar(
-                    content: Text(
-                      'Tapped Category: ${headline.category!.name}',
-                    ),
-                  ),
+                  SnackBar(content: Text('Tapped Date: $formattedDate')),
                 );
             },
-            child: Chip(
-              avatar: Icon(
-                Icons.label_outline,
-                size: iconSize,
-                color: colorScheme.onSurfaceVariant, // Changed color
-              ),
-              label: Text(headline.category!.name),
-              labelStyle: textTheme.labelSmall?.copyWith(
-                color: colorScheme.onSurfaceVariant, // Changed color
-              ),
-              // backgroundColor removed
-              padding: const EdgeInsets.symmetric(horizontal: AppSpacing.sm),
-              visualDensity: VisualDensity.compact,
-              materialTapTargetSize: MaterialTapTargetSize.shrinkWrap,
+            child: Row(
+              mainAxisSize: MainAxisSize.min,
+              children: [
+                Icon(
+                  Icons.calendar_today_outlined,
+                  size: iconSize,
+                  color: colorScheme.onSurfaceVariant,
+                ),
+                const SizedBox(width: AppSpacing.xs),
+                Text(formattedDate, style: metadataStyle),
+              ],
             ),
           ),
-        if (headline.source?.name != null)
+        if (headline.category?.name != null) ...[
+          if (formattedDate.isNotEmpty)
+            Padding(
+              padding:
+                  const EdgeInsets.symmetric(horizontal: AppSpacing.xs / 2),
+              child: Text('•', style: metadataStyle),
+            ),
           GestureDetector(
             onTap: () {
               ScaffoldMessenger.of(context)
                 ..hideCurrentSnackBar()
                 ..showSnackBar(
                   SnackBar(
-                    content: Text('Tapped Source: ${headline.source!.name}'),
+                    content:
+                        Text('Tapped Category: ${headline.category!.name}'),
                   ),
                 );
             },
-            child: Row(
-              mainAxisSize: MainAxisSize.min,
-              children: [
-                Icon(
-                  Icons.source_outlined,
-                  size: iconSize,
-                  color: colorScheme.onSurfaceVariant,
-                ),
-                const SizedBox(width: AppSpacing.xs),
-                Flexible(
-                  child: Text(
-                    headline.source!.name,
-                    style: metadataStyle,
-                    overflow: TextOverflow.ellipsis,
-                  ),
-                ),
-              ],
+            child: Chip(
+              label: Text(headline.category!.name),
+              labelStyle: chipLabelStyle,
+              backgroundColor: chipBackgroundColor,
+              padding: const EdgeInsets.symmetric(horizontal: AppSpacing.sm),
+              visualDensity: VisualDensity.compact,
+              materialTapTargetSize: MaterialTapTargetSize.shrinkWrap,
             ),
           ),
-        if (formattedDate.isNotEmpty)
+        ],
+        if (headline.source?.name != null) ...[
+          if (formattedDate.isNotEmpty || headline.category?.name != null)
+            Padding(
+              padding:
+                  const EdgeInsets.symmetric(horizontal: AppSpacing.xs / 2),
+              child: Text('•', style: metadataStyle),
+            ),
           GestureDetector(
             onTap: () {
               ScaffoldMessenger.of(context)
                 ..hideCurrentSnackBar()
                 ..showSnackBar(
                   SnackBar(
-                    content: Text('Tapped Date: $formattedDate'),
+                    content: Text('Tapped Source: ${headline.source!.name}'),
                   ),
                 );
             },
-            child: Row(
-              mainAxisSize: MainAxisSize.min,
-              children: [
-                Icon(
-                  Icons.calendar_today_outlined,
-                  size: iconSize,
-                  color: colorScheme.onSurfaceVariant,
-                ),
-                const SizedBox(width: AppSpacing.xs),
-                Text(formattedDate, style: metadataStyle),
-              ],
+            child: Chip(
+              label: Text(headline.source!.name),
+              labelStyle: chipLabelStyle,
+              backgroundColor: chipBackgroundColor,
+              padding: const EdgeInsets.symmetric(horizontal: AppSpacing.sm),
+              visualDensity: VisualDensity.compact,
+              materialTapTargetSize: MaterialTapTargetSize.shrinkWrap,
             ),
           ),
+        ],
       ],
     );
   }