rejigger icons for clarity

dfl · dfl · commit 711802224325 · 2026-05-01T13:50:14.000-07:00
diff --git a/app/assets/images/icons/alert_round.svg b/app/assets/images/icons/alert_round.svg
diff --git a/app/assets/images/icons/alert_triangle.svg b/app/assets/images/icons/alert_triangle.svg
diff --git a/app/components/base_component.rb b/app/components/base_component.rb
@@ -34,12 +34,12 @@ class BaseComponent < ViewComponent::Base
       path: "icons/arrow_open.svg",
       alt: "chevron up"
     },
-    "alert" => {
-      path: "icons/alert.svg",
-      alt: "alert"
+    "warn" => {
+      path: "icons/alert_triangle.svg",
+      alt: "warning"
     },
     "error" => {
-      path: "icons/error_icon.svg",
+      path: "icons/alert_round.svg",
       alt: "error"
     },
     "delete" => {
diff --git a/app/components/checkboxes_component.html.erb b/app/components/checkboxes_component.html.erb
@@ -30,7 +30,7 @@
     <%= render(FormErrorsComponent.new(form: @form, method: @method, small: @small)) %>
     <% if has_warning? %>
       <p class="form_warning flex items-center gap-cfa-sm text-text-body-secondary">
-        <%= inline_icon(:alert, size: icon_size, css_class: "text-warning") %>
+        <%= inline_icon(:warn, size: icon_size, css_class: "text-warning") %>
         <span><%= @warning_message %></span>
       </p>
     <% end %>
diff --git a/app/components/radio_buttons_component.html.erb b/app/components/radio_buttons_component.html.erb
@@ -29,7 +29,7 @@
     <%= render(FormErrorsComponent.new(form: @form, method: @method, small: @small)) %>
     <% if has_warning? %>
       <p class="form_warning flex items-center gap-cfa-sm text-text-body-secondary">
-        <%= inline_icon(:alert, size: icon_size, css_class: "text-warning") %>
+        <%= inline_icon(:warn, size: icon_size, css_class: "text-warning") %>
         <span><%= @warning_message %></span>
       </p>
     <% end %>
diff --git a/test/components/previews/icon_component_preview.rb b/test/components/previews/icon_component_preview.rb
@@ -20,8 +20,8 @@ def xmark
     render(IconComponent.new(icon: "xmark"))
   end
 
-  def alert
-    render(IconComponent.new(icon: "alert"))
+  def warn
+    render(IconComponent.new(icon: "warn"))
   end
 
   def delete
