all_channels: Make row item at least 44px tall ConstrainedBox

Make sure every row is at least 44px in height by specifiying
BoxConstraints(minHeight: 44)

Author: chungwwei

lib/widgets/all_channels.dart

@@ -103,25 +103,26 @@ class AllChannelsListEntry extends StatelessWidget {
         MessageListPage.buildRoute(context: context,
           narrow: ChannelNarrow(channel.streamId))),
       onLongPress: () => showChannelActionSheet(context, channelId: channel.streamId),
-      child: Padding(
-        padding: EdgeInsetsDirectional.only(start: 8, end: 12),
-        child: Row(spacing: 6, children: [
-          Icon(
-            size: 20,
-            color: colorSwatchFor(context, subscription).iconOnPlainBackground,
-            iconDataForStream(channel)),
-          Expanded(
-            child: Text(
-              maxLines: 1,
-              overflow: TextOverflow.ellipsis,
-              style: TextStyle(
-                color: designVariables.textMessage,
-                fontSize: 17,
-                height: 20 / 17,
-              ).merge(weightVariableTextStyle(context, wght: 600)),
-              channel.name)),
-          if (hasContentAccess) _SubscribeToggle(channel: channel),
-        ])));
+      child: ConstrainedBox(constraints: const BoxConstraints(minHeight: 44),
+        child: Padding(
+          padding: EdgeInsetsDirectional.only(start: 8, end: 12),
+          child: Row(spacing: 6, children: [
+            Icon(
+              size: 20,
+              color: colorSwatchFor(context, subscription).iconOnPlainBackground,
+              iconDataForStream(channel)),
+            Expanded(
+              child: Text(
+                maxLines: 1,
+                overflow: TextOverflow.ellipsis,
+                style: TextStyle(
+                  color: designVariables.textMessage,
+                  fontSize: 17,
+                  height: 20 / 17,
+                ).merge(weightVariableTextStyle(context, wght: 600)),
+                channel.name)),
+            hasContentAccess ? _SubscribeToggle(channel: channel) : const SizedBox(height: 40, width: 52)
+          ]))));
   }
 }
 

test/widgets/all_channels_test.dart

@@ -203,6 +203,9 @@ void main() {
       } else {
         check(maybeToggle).isNull();
       }
+
+      final touchTargetSize = tester.getSize(findElement);
+      check(touchTargetSize.height).equals(44);
     }
   });