From cbff90ec5b9b808ea4402afdf7818c420182ede1 Mon Sep 17 00:00:00 2001
From: armada-lookout <armada-lookout@users.noreply.github.com>
Date: Tue, 21 Jul 2026 11:10:06 +0000
Subject: [PATCH] Use YakShaver icon and align "What we collect" icons in
 consent dialog

The "Help Us Improve YakShaver" telemetry consent dialog used a generic
BarChart3 analyze icon next to its heading, and the leading icons for
each "What we collect" item (Error reports, Workflow performance,
Usage metrics) didn't line up with their checkbox/label row because of
an ad-hoc mt-0.5 offset.

- Replace the heading icon with the YakShaver (small Red Bull) logo,
  reusing the same SQ-YakShaver-LogoIcon-Red.svg asset used elsewhere
  in the app.
- Wrap each list icon in a min-h-11 flex container matching the height
  of its checkbox/label row so all three icons align consistently with
  each other and with their text.

Closes #964

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
---
 .../settings/telemetry/TelemetryConsentDialog.tsx | 15 +++++++++++----
 1 file changed, 11 insertions(+), 4 deletions(-)

diff --git a/src/ui/src/components/settings/telemetry/TelemetryConsentDialog.tsx b/src/ui/src/components/settings/telemetry/TelemetryConsentDialog.tsx
index 849c8f9ea..d6fa98a11 100644
--- a/src/ui/src/components/settings/telemetry/TelemetryConsentDialog.tsx
+++ b/src/ui/src/components/settings/telemetry/TelemetryConsentDialog.tsx
@@ -11,6 +11,7 @@ import {
   DialogTitle,
 } from "@/components/ui/dialog";
 import { Label } from "@/components/ui/label";
+import yakShaverLogo from "/logos/SQ-YakShaver-LogoIcon-Red.svg?url";
 
 interface TelemetryConsentDialogProps {
   open: boolean;
@@ -46,7 +47,7 @@ export function TelemetryConsentDialog({
       <DialogContent className="sm:max-w-[500px]">
         <DialogHeader>
           <DialogTitle className="flex items-center gap-2">
-            <BarChart3 className="h-5 w-5" />
+            <img src={yakShaverLogo} alt="" className="h-5 w-5" />
             Help Us Improve YakShaver
           </DialogTitle>
           <DialogDescription className="pt-2">
@@ -60,7 +61,9 @@ export function TelemetryConsentDialog({
             <h4 className="text-sm font-medium">What we collect:</h4>
 
             <div className="flex items-start space-x-3">
-              <Bug className="h-4 w-4 mt-0.5 text-muted-foreground" />
+              <div className="flex items-center min-h-11">
+                <Bug className="h-4 w-4 text-muted-foreground" />
+              </div>
               <div className="flex-1">
                 <div className="flex items-center space-x-2 min-h-11">
                   <Checkbox
@@ -82,7 +85,9 @@ export function TelemetryConsentDialog({
             </div>
 
             <div className="flex items-start space-x-3">
-              <Activity className="h-4 w-4 mt-0.5 text-muted-foreground" />
+              <div className="flex items-center min-h-11">
+                <Activity className="h-4 w-4 text-muted-foreground" />
+              </div>
               <div className="flex-1">
                 <div className="flex items-center space-x-2 min-h-11">
                   <Checkbox
@@ -104,7 +109,9 @@ export function TelemetryConsentDialog({
             </div>
 
             <div className="flex items-start space-x-3">
-              <BarChart3 className="h-4 w-4 mt-0.5 text-muted-foreground" />
+              <div className="flex items-center min-h-11">
+                <BarChart3 className="h-4 w-4 text-muted-foreground" />
+              </div>
               <div className="flex-1">
                 <div className="flex items-center space-x-2 min-h-11">
                   <Checkbox