NOMERGE read_receipts [nfc]: Inline _ReadReceiptsUserList

Author: chrisbobbe

lib/widgets/read_receipts.dart

@@ -80,15 +80,33 @@ class _ReadReceiptsState extends State<ReadReceipts> with PerAccountStoreAwareSt
 
   @override
   Widget build(BuildContext context) {
+    final zulipLocalizations = ZulipLocalizations.of(context);
     // TODO could pull out this layout/appearance code,
     //   focusing this widget only on state management
 
+    final content = switch (status) {
+      FetchStatus.loading => BottomSheetEmptyContentPlaceholder(loading: true),
+      FetchStatus.error   => BottomSheetEmptyContentPlaceholder(
+        message: zulipLocalizations.actionSheetReadReceiptsErrorReadCount),
+      FetchStatus.success => userIds.isEmpty
+        ? BottomSheetEmptyContentPlaceholder(
+            message: zulipLocalizations.actionSheetReadReceiptsZeroReadCount)
+        : InsetShadowBox(
+            top: 8, bottom: 8,
+            color: DesignVariables.of(context).bgContextMenu,
+            child: ListView.builder(
+              padding: EdgeInsets.symmetric(vertical: 8),
+              itemCount: userIds.length,
+              itemBuilder: (context, index) =>
+                ReadReceiptsUserItem(userId: userIds[index])))
+    };
+
     return SizedBox(
       height: 500, // TODO(design) tune
       child: Column(
         children: [
           _ReadReceiptsHeader(receiptCount: userIds.length, status: status),
-          Expanded(child: _ReadReceiptsUserList(userIds: userIds, status: status)),
+          Expanded(child: content),
           Padding(
             padding: const EdgeInsets.symmetric(horizontal: 16),
             child: const BottomSheetDismissButton(style: BottomSheetDismissButtonStyle.close))
@@ -129,35 +147,6 @@ class _ReadReceiptsHeader extends StatelessWidget {
   }
 }
 
-class _ReadReceiptsUserList extends StatelessWidget {
-  const _ReadReceiptsUserList({required this.userIds, required this.status});
-
-  final List<int> userIds;
-  final FetchStatus status;
-
-  @override
-  Widget build(BuildContext context) {
-    final zulipLocalizations = ZulipLocalizations.of(context);
-
-    return switch(status) {
-      FetchStatus.loading => BottomSheetEmptyContentPlaceholder(loading: true),
-      FetchStatus.error   => BottomSheetEmptyContentPlaceholder(
-        message: zulipLocalizations.actionSheetReadReceiptsErrorReadCount),
-      FetchStatus.success => userIds.isEmpty
-        ? BottomSheetEmptyContentPlaceholder(
-            message: zulipLocalizations.actionSheetReadReceiptsZeroReadCount)
-        : InsetShadowBox(
-            top: 8, bottom: 8,
-            color: DesignVariables.of(context).bgContextMenu,
-            child: ListView.builder(
-              padding: EdgeInsets.symmetric(vertical: 8),
-              itemCount: userIds.length,
-              itemBuilder: (context, index) =>
-                ReadReceiptsUserItem(userId: userIds[index])))
-    };
-  }
-}
-
 // TODO: deduplicate the code with [ViewReactionsUserItem]
 @visibleForTesting
 class ReadReceiptsUserItem extends StatelessWidget {