fix(sidebar): tighten recent session rows

Author: verseles

lib/presentation/pages/chat_page/chat_page_scaffold.dart

@@ -680,6 +680,7 @@ extension _ChatPageScaffold on _ChatPageState {
     );
     return SidebarSelectionIndicator(
       selected: isCurrentSession,
+      padding: const EdgeInsetsDirectional.only(start: 3),
       child: SessionContextMenuRegion(
         session: session,
         actions: menuActions,
@@ -752,6 +753,7 @@ extension _ChatPageScaffold on _ChatPageState {
                     session: session,
                     actions: menuActions,
                     surface: 'recent',
+                    compact: !isMobileLayout,
                     iconColor: isCurrentSession
                         ? selectedForeground
                         : colorScheme.onSurfaceVariant,

lib/presentation/widgets/session_context_menu.dart

@@ -49,18 +49,31 @@ class SessionContextMenuButton extends StatelessWidget {
     required this.actions,
     required this.surface,
     this.iconColor,
+    this.compact = false,
   });
 
   final ChatSession session;
   final SessionContextMenuActions actions;
   final String surface;
   final Color? iconColor;
+  final bool compact;
 
   @override
   Widget build(BuildContext context) {
     return PopupMenuButton<String>(
       tooltip: context.l10n.chatSessionActions,
-      icon: Icon(Symbols.more_vert, color: iconColor),
+      padding: compact ? EdgeInsets.zero : const EdgeInsets.all(8),
+      icon: compact ? null : Icon(Symbols.more_vert, color: iconColor),
+      child: compact
+          ? Semantics(
+              button: true,
+              label: context.l10n.chatSessionActions,
+              child: SizedBox.square(
+                dimension: 32,
+                child: Icon(Symbols.more_vert, size: 20, color: iconColor),
+              ),
+            )
+          : null,
       onOpened: () =>
           logSessionContextMenuOpen(surface: surface, sessionId: session.id),
       onSelected: (value) => unawaited(

test/widget/chat_page_test.dart

@@ -2717,17 +2717,49 @@ void main() {
           find.byKey(const ValueKey<String>('recent_session_tile_ses_recent')),
           findsOneWidget,
         );
-        expect(
-          find.descendant(
-            of: find.byKey(
-              const ValueKey<String>('recent_session_tile_ses_recent'),
-            ),
-            matching: find.byType(SessionContextMenuButton),
+        final recentHeaderLeft = tester
+            .getTopLeft(find.text('Recent sessions'))
+            .dx;
+        final recentTitleLeft = tester
+            .getTopLeft(
+              find.byKey(
+                const ValueKey<String>('recent_session_title_ses_recent'),
+              ),
+            )
+            .dx;
+        final recentTileHeight = tester
+            .getSize(
+              find.byKey(
+                const ValueKey<String>('recent_session_tile_ses_recent'),
+              ),
+            )
+            .height;
+        final recentMenuButton = find.descendant(
+          of: find.byKey(
+            const ValueKey<String>('recent_session_tile_ses_recent'),
           ),
-          findsOneWidget,
+          matching: find.byType(SessionContextMenuButton),
         );
+
+        expect(recentTitleLeft - recentHeaderLeft, greaterThanOrEqualTo(0));
+        expect(recentTitleLeft - recentHeaderLeft, lessThanOrEqualTo(6));
+        expect(recentTileHeight, lessThanOrEqualTo(50));
+        expect(recentMenuButton, findsOneWidget);
         expect(find.text('Project A'), findsWidgets);
 
+        await tester.tap(recentMenuButton);
+        await tester.pumpAndSettle();
+
+        expect(find.text('Pin'), findsOneWidget);
+        expect(find.text('Rename'), findsOneWidget);
+        expect(find.text('Share'), findsOneWidget);
+        expect(find.text('Archive'), findsOneWidget);
+        expect(find.text('Fork'), findsOneWidget);
+        expect(find.text('Delete'), findsOneWidget);
+
+        await tester.tapAt(const Offset(10, 10));
+        await tester.pumpAndSettle();
+
         await tester.tap(
           find.byKey(const ValueKey<String>('recent_session_tile_ses_recent')),
           buttons: kSecondaryMouseButton,