pa_link_to needs to support title option

openaustralia · Feb 22, 2024 · 57ec571 · 57ec571
1 parent a367434
commit 57ec571
Showing 1 changed file with 9 additions and 9 deletions.
diff --git a/app/helpers/application_helper.rb b/app/helpers/application_helper.rb
@@ -19,26 +19,26 @@ def pa_link_classes
 
   # Only to be used in tailwind theme
   # TODO: Generalise to support all the variants
-  sig { params(body: T.untyped, url: T.untyped, extra_classes: T.nilable(String)).returns(String) }
-  def pa_link_to(body, url, extra_classes: nil)
+  sig { params(body: T.untyped, url: T.untyped, extra_classes: T.nilable(String), title: T.nilable(String)).returns(String) }
+  def pa_link_to(body, url, extra_classes: nil, title: nil)
     # These extra classes can't override the default styling because they're at the end
-    link_to(body, url, class: "#{pa_link_classes} #{extra_classes}")
+    link_to(body, url, class: "#{pa_link_classes} #{extra_classes}", title:)
   end
 
   # Only to be used in tailwind theme
   # TODO: Generalise to support all the variants
-  sig { params(body: T.untyped, url: T.untyped, extra_classes: T.nilable(String)).returns(String) }
-  def pa_link_to_unless_current(body, url, extra_classes: nil)
+  sig { params(body: T.untyped, url: T.untyped, extra_classes: T.nilable(String), title: T.nilable(String)).returns(String) }
+  def pa_link_to_unless_current(body, url, extra_classes: nil, title: nil)
     # These extra classes can't override the default styling because they're at the end
-    link_to_unless_current(body, url, class: "#{pa_link_classes} #{extra_classes}")
+    link_to_unless_current(body, url, class: "#{pa_link_classes} #{extra_classes}", title:)
   end
 
   # Only to be used in tailwind theme
   # TODO: Generalise to support all the variants
-  sig { params(condition: T::Boolean, body: T.untyped, url: T.untyped, extra_classes: T.nilable(String)).returns(String) }
-  def pa_link_to_unless(condition, body, url, extra_classes: nil)
+  sig { params(condition: T::Boolean, body: T.untyped, url: T.untyped, extra_classes: T.nilable(String), title: T.nilable(String)).returns(String) }
+  def pa_link_to_unless(condition, body, url, extra_classes: nil, title: nil)
     # These extra classes can't override the default styling because they're at the end
-    link_to_unless(condition, body, url, class: "#{pa_link_classes} #{extra_classes}")
+    link_to_unless(condition, body, url, class: "#{pa_link_classes} #{extra_classes}", title:)
   end
 
   sig { params(path: String, extra_classes: T::Array[Symbol], block: T.untyped).returns(T.untyped) }