Next Version Relese (#1892)

* fix(dropdown): add check for el (#1885) * chore: update browserslist dependency * chore: remove empty package.json dependencies block * chore: update node engines to support v20 (#1888) Co-authored-by: Julian Skinner <[email protected]> * 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 --------- Co-authored-by: Phillip Lovelace <[email protected]> Co-authored-by: Monica Wheeler <[email protected]> Co-authored-by: Julian Skinner <[email protected]> Co-authored-by: Julian Skinner <[email protected]>
Kajabi · May 31, 2024 · 4d579a8 · 4d579a8
1 parent 9fbb17f
commit 4d579a8
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>