action_sheet [nfc]: Move an 8px top padding into BottomSheetHeader

Author: chrisbobbe

lib/widgets/action_sheet.dart

@@ -114,8 +114,6 @@ typedef WidgetBuilderFromTextStyle = Widget Function(TextStyle);
 /// The "build" params support richer content, such as [TextWithLink],
 /// and the callback is passed a [TextStyle] which is the base style.
 ///
-/// Assumes 8px padding below the top of the bottom sheet.
-///
 /// Figma; just message no title:
 ///   https://www.figma.com/design/1JTNtYo9memgW7vV6d0ygq/Zulip-Mobile?node-id=3481-26993&m=dev
 ///
@@ -171,7 +169,7 @@ class BottomSheetHeader extends StatelessWidget {
     };
 
     return Padding(
-      padding: EdgeInsets.fromLTRB(16, 8, 16, 4),
+      padding: EdgeInsets.fromLTRB(16, 16, 16, 4),
       child: Column(
         crossAxisAlignment: CrossAxisAlignment.stretch,
         spacing: 8,

lib/widgets/emoji_reaction.dart

@@ -804,10 +804,8 @@ class ViewReactionsHeader extends StatelessWidget {
     final reactions = message?.reactions;
 
     if (reactions == null || reactions.aggregated.isEmpty) {
-      return Padding(
-        padding: const EdgeInsets.only(top: 8),
-        child: BottomSheetHeader(message: zulipLocalizations.seeWhoReactedSheetNoReactions),
-      );
+      return BottomSheetHeader(
+        message: zulipLocalizations.seeWhoReactedSheetNoReactions);
     }
 
     return Padding(

lib/widgets/read_receipts.dart

@@ -128,11 +128,9 @@ class _ReadReceiptsHeader extends StatelessWidget {
         markup: zulipLocalizations.actionSheetReadReceiptsReadCount(receiptCount));
     }
 
-    return Padding(
-      padding: const EdgeInsets.only(top: 8),
-      child: BottomSheetHeader(
-        title: zulipLocalizations.actionSheetReadReceipts,
-        buildMessage: headerMessageBuilder));
+    return BottomSheetHeader(
+      title: zulipLocalizations.actionSheetReadReceipts,
+      buildMessage: headerMessageBuilder);
   }
 }