TableBuilder: Make destroy link in gear menu render correctly

Previously we weren't correctly outputting the proper HTML to get the
:delete link rendering correctly with the icon and text. It should show:

    [TRASH CAN ICON] Supprimer

but instead was showing the href.

To get this working, I added a `@content` attribute to `Link`. This
allows us to optionally specify non-text content to display in the link.

We then have delete-link-specific handling when rendering the gear
menu's HTML. In that particular case, the `<li>` needs to have a
'delete-action' class in order for the styling to work properly.

Created a new `destroy_link` helper method to allow us to reuse the
destroy link content in the header bar.

TODO: Collect Link#content and Link#name into the same property.
TODO: Rename `#destroy_link` to `#destroy_link_content`

Refs #3479

2 files changed, 16 insertions, 7 deletions

diff --git a/app/helpers/links_helper.rb b/app/helpers/links_helper.rb
new file mode 100644
index 000000000..48cfe9ca4
--- /dev/null
+++ b/app/helpers/links_helper.rb
@@ -0,0 +1,5 @@
+module LinksHelper
+  def destroy_link
+    content_tag(:span, nil, class: 'fa fa-trash') + t('actions.destroy')
+  end
+end
diff --git a/app/helpers/table_builder_helper.rb b/app/helpers/table_builder_helper.rb
index 4537a0795..af05a7354 100644
--- a/app/helpers/table_builder_helper.rb
+++ b/app/helpers/table_builder_helper.rb
@@ -113,13 +113,17 @@ module TableBuilderHelper
         CustomLinks.new(item, pundit_user, links).links +
         item.action_links
       ).map do |link|
-        # TODO: ensure the Delete link is formatted correctly with the spacer,
-        # icon, and label
-        content_tag :li, link_to(
-          link.name,
-          link.href,
-          method: link.method,
-          data: link.data
+        # TODO: make a new method for this
+        content_tag(
+          :li,
+          link_to(
+            link.href,
+            method: link.method,
+            data: link.data
+          ) do
+            link.content || link.name
+          end,
+          class: link.method == :delete ? 'delete-action' : '',
         )
       end.join.html_safe