refactor(user_management): Align user action buttons with content actions

- Refactored `UserActionButtons` to display a primary action icon directly in the table row.
- Secondary actions (like "Copy User ID") are now in a `PopupMenuButton` (ellipsis icon).
- This change mimics the UI pattern of `ContentActionButtons`, creating a consistent user experience across the dashboard.
- The primary action is now an `IconButton` for "Promote" or "Demote", depending on the user's role.

Author: fulleni

lib/user_management/widgets/user_action_buttons.dart

@@ -1,20 +1,18 @@
 import 'package:core/core.dart';
 import 'package:flutter/material.dart';
 import 'package:flutter/services.dart';
-import 'package:flutter_bloc/flutter_bloc.dart';
-import 'package:flutter_news_app_web_dashboard_full_source_code/content_management/widgets/content_action_buttons.dart'
-    show ContentActionButtons;
+import 'package:flutter_bloc/flutter_bloc.dart' show ReadContext;
 import 'package:flutter_news_app_web_dashboard_full_source_code/l10n/app_localizations.dart';
 import 'package:flutter_news_app_web_dashboard_full_source_code/user_management/bloc/user_management_bloc.dart';
-import 'package:ui_kit/ui_kit.dart';
 
 /// {@template user_action_buttons}
 /// A widget that displays contextual action buttons for a user in the user
 /// management table.
 ///
 /// Actions are presented in a [PopupMenuButton] and are conditionally
 /// displayed based on the user's role and permissions. This follows a similar
-/// pattern to [ContentActionButtons] but is tailored for user management.
+/// pattern to the content management action buttons but is tailored for user
+/// management.
 /// {@endtemplate}
 class UserActionButtons extends StatelessWidget {
   /// {@macro user_action_buttons}
@@ -32,95 +30,95 @@ class UserActionButtons extends StatelessWidget {
 
   @override
   Widget build(BuildContext context) {
+    // visibleActions holds the primary action icons that are always visible.
+    final visibleActions = <Widget>[];
+    // overflowMenuItems holds actions that are in the popup menu.
     final overflowMenuItems = <PopupMenuEntry<String>>[];
 
     // Rule: Do not show any actions for admin users.
     if (user.dashboardRole == DashboardUserRole.admin) {
       return const SizedBox.shrink();
     }
-
-    // Add contextual actions based on the user's current dashboard role.
+    // Primary action: Copy User ID. This is always available for non-admins.
+    visibleActions.add(
+      IconButton(
+        visualDensity: VisualDensity.compact,
+        iconSize: 20,
+        icon: const Icon(Icons.copy),
+        tooltip: l10n.copyId,
+        onPressed: () => _onCopyId(context),
+      ),
+    );
+    // Add contextual actions to the overflow menu based on the user's role.
     switch (user.dashboardRole) {
       case DashboardUserRole.none:
         overflowMenuItems.add(
           PopupMenuItem<String>(
             value: 'promote',
-            child: Row(
-              children: [
-                const Icon(Icons.arrow_upward),
-                const SizedBox(width: AppSpacing.sm),
-                Text(l10n.promoteToPublisher),
-              ],
-            ),
+            child: Text(l10n.promoteToPublisher),
           ),
         );
       case DashboardUserRole.publisher:
         overflowMenuItems.add(
           PopupMenuItem<String>(
             value: 'demote',
-            child: Row(
-              children: [
-                const Icon(Icons.arrow_downward),
-                const SizedBox(width: AppSpacing.sm),
-                Text(l10n.demoteToUser),
-              ],
-            ),
+            child: Text(l10n.demoteToUser),
           ),
         );
       case DashboardUserRole.admin:
         // No actions for admins, handled by the initial check.
         break;
     }
-
-    // Add the "Copy User ID" action for all non-admin users.
-    overflowMenuItems.add(
-      PopupMenuItem<String>(
-        value: 'copy_id',
-        child: Row(
-          children: [
-            const Icon(Icons.copy),
-            const SizedBox(width: AppSpacing.sm),
-            Text(l10n.copyId),
-          ],
+    // Add the overflow menu button if there are items in it.
+    if (overflowMenuItems.isNotEmpty) {
+      visibleActions.add(
+        SizedBox(
+          width: 32,
+          child: PopupMenuButton<String>(
+            iconSize: 20,
+            icon: const Icon(Icons.more_vert),
+            tooltip: l10n.moreActions,
+            onSelected: (value) {
+              switch (value) {
+                case 'promote':
+                  _onPromote(context);
+                case 'demote':
+                  _onDemote(context);
+              }
+            },
+            itemBuilder: (context) => overflowMenuItems,
+          ),
         ),
-      ),
-    );
-
+      );
+    }
     return Row(
       mainAxisSize: MainAxisSize.min,
-      children: [
-        PopupMenuButton<String>(
-          iconSize: 20,
-          icon: const Icon(Icons.more_vert),
-          tooltip: l10n.moreActions,
-          onSelected: (value) {
-            switch (value) {
-              case 'promote':
-                context.read<UserManagementBloc>().add(
-                  UserDashboardRoleChanged(
-                    userId: user.id,
-                    dashboardRole: DashboardUserRole.publisher,
-                  ),
-                );
-              case 'demote':
-                context.read<UserManagementBloc>().add(
-                  UserDashboardRoleChanged(
-                    userId: user.id,
-                    dashboardRole: DashboardUserRole.none,
-                  ),
-                );
-              case 'copy_id':
-                Clipboard.setData(ClipboardData(text: user.id));
-                ScaffoldMessenger.of(context)
-                  ..hideCurrentSnackBar()
-                  ..showSnackBar(
-                    SnackBar(content: Text(l10n.idCopiedToClipboard(user.id))),
-                  );
-            }
-          },
-          itemBuilder: (context) => overflowMenuItems,
-        ),
-      ],
+      children: visibleActions,
     );
   }
+
+  void _onPromote(BuildContext context) =>
+      context.read<UserManagementBloc>().add(
+        UserDashboardRoleChanged(
+          userId: user.id,
+          dashboardRole: DashboardUserRole.publisher,
+        ),
+      );
+
+  void _onDemote(BuildContext context) =>
+      context.read<UserManagementBloc>().add(
+        UserDashboardRoleChanged(
+          userId: user.id,
+          dashboardRole: DashboardUserRole.none,
+        ),
+      );
+
+  void _onCopyId(BuildContext context) {
+    Clipboard.setData(ClipboardData(text: user.id));
+    ScaffoldMessenger.of(context)
+      ..hideCurrentSnackBar()
+      ..showSnackBar(
+        SnackBar(content: Text(l10n.idCopiedToClipboard(user.id))),
+      );
+  }
 }