fix(breadcrumbs): rails resolve has_back_icon bug (#1891)

* fix(breadcrumbs): rails resolve has back icon prop bug when value is false * chore(breadcrumbs): add comment for code clarity
Kajabi · May 29, 2024 · ea31830 · ea31830
1 parent 4cd1b72
commit ea31830
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/docs/lib/sage_rails/app/views/sage_components/_sage_breadcrumbs.html.erb b/docs/lib/sage_rails/app/views/sage_components/_sage_breadcrumbs.html.erb
@@ -1,5 +1,6 @@
 <%
-has_back_icon = component.has_back_icon.present? ? component.has_back_icon : component.items.length() == 1
+# The intent is to set this prop to false unless the has_back_icon prop is explicitly set
+has_back_icon = component.has_back_icon.nil? ? component.items.length() == 1 : component.has_back_icon
 is_progressbar = component.is_progressbar.present? && component.is_progressbar
 %>
 <nav
@@ -38,7 +39,7 @@ is_progressbar = component.is_progressbar.present? && component.is_progressbar
         <i class="sage-breadcrumbs__icon sage-icon-arrow-left" aria-hidden="true"></i>
       <% end %>
       <%= component.items[0][:text] %>
-    </a>  
+    </a>
   </p>
   <% end %>
 </nav>