From efa2c4cfe1ef42fc9e4a5241cbf58c1dcc57af17 Mon Sep 17 00:00:00 2001
From: hatz <hatz@hey.com>
Date: Thu, 1 May 2025 18:54:53 -0500
Subject: [PATCH] Use icon helper instead of lucide_icon helper

---
 app/views/rules/_form.html.erb | 8 ++++----
 app/views/rules/_rule.html.erb | 6 +++---
 app/views/rules/index.html.erb | 8 ++++----
 3 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/app/views/rules/_form.html.erb b/app/views/rules/_form.html.erb
index 346cddf5..998e719d 100644
--- a/app/views/rules/_form.html.erb
+++ b/app/views/rules/_form.html.erb
@@ -11,7 +11,7 @@
 
   <section class="space-y-4">
     <div class="flex items-center gap-1 text-secondary">
-      <%= lucide_icon "tag", class: "w-4 h-4 shrink-0" %>
+      <%= icon "tag", size: "sm" %>
       <h3 class="text-sm font-medium text-primary">Rule name (optional)</h3>
     </div>
     <div class="ml-6">
@@ -21,7 +21,7 @@
 
   <section class="space-y-4">
     <div class="flex items-center gap-1 text-secondary">
-      <%= lucide_icon "help-circle", class: "w-4 h-4 shrink-0" %>
+      <%= icon "help-circle", size: "sm" %>
       <h3 class="text-sm font-medium text-primary font-mono">IF</h3>
     </div>
 
@@ -58,7 +58,7 @@
 
   <section class="space-y-4">
     <div class="flex items-center gap-1 text-secondary">
-      <%= lucide_icon "arrow-right", class: "w-4 h-4 shrink-0" %>
+      <%= icon "arrow-right", size: "sm" %>
       <h3 class="text-sm font-medium text-primary font-mono">THEN</h3>
     </div>
 
@@ -81,7 +81,7 @@
 
   <section class="space-y-4">
     <div class="flex items-center gap-1 text-secondary">
-      <%= lucide_icon "calendar", class: "w-4 h-4 shrink-0" %>
+      <%= icon "calendar", size: "sm" %>
       <h3 class="text-sm font-medium text-primary font-mono">FOR</h3>
     </div>
 
diff --git a/app/views/rules/_rule.html.erb b/app/views/rules/_rule.html.erb
index 8455fd5c..c457f813 100644
--- a/app/views/rules/_rule.html.erb
+++ b/app/views/rules/_rule.html.erb
@@ -9,7 +9,7 @@
     <% if rule.conditions.any? %>
       <div class="flex items-center gap-2 mt-1">
         <div class="flex items-center gap-1 text-secondary w-16 shrink-0">
-          <%= lucide_icon "help-circle", class: "w-4 h-4 shrink-0" %>
+          <%= icon "help-circle", size: "sm" %>
           <span class="font-mono text-xs">IF</span>
         </div>
         <p class="flex items-center flex-wrap gap-1.5 m-0">
@@ -29,7 +29,7 @@
 
     <div class="flex items-center gap-2 mt-1">
       <div class="flex items-center gap-1 text-secondary w-16 shrink-0">
-        <%= lucide_icon "arrow-right", class: "w-4 h-4 shrink-0" %>
+        <%= icon "arrow-right", size: "sm" %>
         <span class="font-mono text-xs">THEN</span>
       </div>
       <p class="flex items-center flex-wrap gap-1.5 m-0">
@@ -48,7 +48,7 @@
 
     <div class="flex items-center gap-2 mt-1">
       <div class="flex items-center gap-1 text-secondary w-16 shrink-0">
-        <%= lucide_icon "calendar", class: "w-4 h-4 shrink-0" %>
+        <%= icon "calendar", size: "sm" %>
         <span class="font-mono text-xs">FOR</span>
       </div>
       <p class="flex items-center flex-wrap gap-1.5 m-0">
diff --git a/app/views/rules/index.html.erb b/app/views/rules/index.html.erb
index 4862f1b5..4f3f4faf 100644
--- a/app/views/rules/index.html.erb
+++ b/app/views/rules/index.html.erb
@@ -49,18 +49,18 @@
           <div class="flex items-center">
             <%= link_to rules_path(sort_by: "name", direction: (@sort_by == "name" && @direction == "asc") ? "desc" : "asc"), 
                   class: "px-2 py-1 rounded flex items-center hover:bg-gray-100 #{@sort_by == 'name' ? 'font-medium' : ''}" do %>
-              <span>Name</span>
+              <span class="mr-1">Name</span>
               <% if @sort_by == "name" %>
-                <%= lucide_icon(@direction == "asc" ? "arrow-up" : "arrow-down", class: "w-3 h-3 text-accent ml-1") %>
+                <%= icon(@direction == "asc" ? "arrow-up" : "arrow-down", size: "xs") %>
               <% end %>
             <% end %>
           </div>
           <div class="flex items-center">
             <%= link_to rules_path(sort_by: "created_at", direction: (@sort_by == "created_at" && @direction == "asc") ? "desc" : "asc"), 
                   class: "px-2 py-1 rounded flex items-center hover:bg-gray-100 #{@sort_by == 'created_at' ? 'font-medium' : ''}" do %>
-              <span>Date</span>
+              <span class="mr-1">Date</span>
               <% if @sort_by == "created_at" %>
-                <%= lucide_icon(@direction == "asc" ? "arrow-up" : "arrow-down", class: "w-3 h-3 text-accent ml-1") %>
+                <%= icon(@direction == "asc" ? "arrow-up" : "arrow-down", size: "xs") %>
               <% end %>
             <% end %>
           </div>