fix: address review feedback on kind chip filter

Author: crackedhandle

packages/devtools_app/lib/src/screens/logging/_message_column.dart

@@ -94,11 +94,9 @@ class MessageColumn extends ColumnData<LogData>
                 builder: (context, constraints) {
                   return MetadataChips(
                     data: data,
-                    onKindTapped: (kind) {
-                      final controller = screenControllers
-                          .lookup<LoggingController>();
-                      controller.setActiveFilter(query: 'k:$kind');
-                    },
+                    onKindTapped: (kind) => screenControllers
+                        .lookup<LoggingController>()
+                        .setActiveFilter(query: 'k:"$kind"'),
                   );
                 },
               ),

packages/devtools_app/lib/src/screens/logging/metadata.dart

@@ -152,9 +152,6 @@ abstract class MetadataChip extends StatelessWidget {
             ? Border.all(color: theme.colorScheme.subtleTextColor)
             : null,
       ),
-      margin: includeLeadingMargin
-          ? const EdgeInsets.only(left: denseSpacing)
-          : null,
       padding: const EdgeInsets.symmetric(
         horizontal: horizontalPadding,
         vertical: verticalPadding,
@@ -187,14 +184,26 @@ abstract class MetadataChip extends StatelessWidget {
     );
 
     if (onTap != null) {
-      chip = MouseRegion(
-        cursor: SystemMouseCursors.click,
-        child: GestureDetector(
-          onTap: onTap,
-          behavior: HitTestBehavior.opaque,
-          child: chip,
+      chip = Padding(
+        padding: includeLeadingMargin
+            ? const EdgeInsets.only(left: denseSpacing)
+            : EdgeInsets.zero,
+        child: MouseRegion(
+          cursor: SystemMouseCursors.click,
+          child: GestureDetector(
+            onTap: onTap,
+            behavior: HitTestBehavior.opaque,
+            child: chip,
+          ),
         ),
       );
+    } else {
+      chip = Padding(
+        padding: includeLeadingMargin
+            ? const EdgeInsets.only(left: denseSpacing)
+            : EdgeInsets.zero,
+        child: chip,
+      );
     }
 
     return maybeWrapWithTooltip(tooltip: tooltip, child: chip);