Extract shared resource toolbar partial (#972)

## Summary
- add `shared/resource_toolbar` partial for edit, preview, and destroy
actions
- replace duplicated action buttons in show views with the shared
toolbar
- document toolbar usage and add translation for preview action

## Testing
- `bundle exec rubocop` *(fails: bundler: command not found: rubocop)*
- `bundle exec brakeman -q -w2` *(fails: bundler: command not found:
brakeman)*
- `bundle exec bundler-audit --update` *(fails: bundler: command not
found: bundler-audit)*
- `bin/codex_style_guard` *(fails: bundler: command not found: rubocop)*
- `bin/ci` *(fails: bundler: command not found: rails)*

------
https://chatgpt.com/codex/tasks/task_e_689a55855c448321b8d8ff076a916415

Author: rsmithlal

app/views/better_together/calls_for_interest/show.html.erb

@@ -22,16 +22,12 @@
         <% end %>
       </div>
       <div class="col-auto">
-        <% if policy(@resource).edit? %>
-          <%= link_to edit_call_for_interest_path(@resource), class: 'btn btn-outline-primary btn-sm me-2', 'aria-label' => 'Edit Partner' do %>
-            <i class="fas fa-edit"></i> <%= t('globals.edit') %>
-          <% end %>
-        <% end %>
-        <% if policy(@resource).destroy? %>
-          <%= link_to call_for_interest_path(@resource), data: { turbo_method: :delete, turbo_confirm: t('globals.confirm_delete') }, class: 'btn btn-outline-danger btn-sm', 'aria-label' => 'Delete Record' do %>
-            <i class="fas fa-trash-alt"></i> <%= t('globals.delete') %>
-          <% end %>
-        <% end %>
+        <%= render 'shared/resource_toolbar',
+                   edit_path: policy(@resource).edit? ? edit_call_for_interest_path(@resource) : nil,
+                   edit_aria_label: 'Edit Partner',
+                   destroy_path: policy(@resource).destroy? ? call_for_interest_path(@resource) : nil,
+                   destroy_confirm: t('globals.confirm_delete'),
+                   destroy_aria_label: 'Delete Record' %>
       </div>
     </div>
   </div>

app/views/better_together/categories/show.html.erb

@@ -12,15 +12,11 @@
     <% if policy(resource_class).index? %>
       <%= link_to t('better_together.categories.back_to_categories'), categories_path, class: 'btn btn-sm btn-outline-secondary' %>
     <% end %>
-    <% if policy(@resource).edit? %>
-      <%= link_to edit_category_path(@resource), class: 'btn btn-outline-primary btn-sm me-2', 'aria-label' => 'Edit Partner' do %>
-        <i class="fas fa-edit"></i> <%= t('globals.edit') %>
-      <% end %>
-    <% end %>
-    <% if policy(@resource).destroy? %>
-      <%= link_to category_path(@resource), data: { turbo_method: :delete, turbo_confirm: t('globals.confirm_delete') }, class: 'btn btn-outline-danger btn-sm', 'aria-label' => 'Delete Record' do %>
-        <i class="fas fa-trash-alt"></i> <%= t('globals.delete') %>
-      <% end %>
-    <% end %>
+    <%= render 'shared/resource_toolbar',
+               edit_path: policy(@resource).edit? ? edit_category_path(@resource) : nil,
+               edit_aria_label: 'Edit Partner',
+               destroy_path: policy(@resource).destroy? ? category_path(@resource) : nil,
+               destroy_confirm: t('globals.confirm_delete'),
+               destroy_aria_label: 'Delete Record' %>
   </div>
 </div>

app/views/better_together/communities/show.html.erb

@@ -27,16 +27,12 @@
         <%= privacy_badge(@resource) %>
       </div>
       <div class="col-auto">
-        <% if policy(@resource).edit? %>
-          <%= link_to edit_community_path(@resource), class: 'btn btn-outline-primary btn-sm me-2', 'aria-label' => 'Edit Partner' do %>
-            <i class="fas fa-edit"></i> <%= t('globals.edit') %>
-          <% end %>
-        <% end %>
-        <% if policy(@resource).destroy? %>
-          <%= link_to community_path(@resource), data: { turbo_method: :delete, turbo_confirm: t('partners.confirm_delete') }, class: 'btn btn-outline-danger btn-sm', 'aria-label' => 'Delete Partner' do %>
-            <i class="fas fa-trash-alt"></i> <%= t('globals.delete') %>
-          <% end %>
-        <% end %>
+        <%= render 'shared/resource_toolbar',
+                   edit_path: policy(@resource).edit? ? edit_community_path(@resource) : nil,
+                   edit_aria_label: 'Edit Partner',
+                   destroy_path: policy(@resource).destroy? ? community_path(@resource) : nil,
+                   destroy_confirm: t('partners.confirm_delete'),
+                   destroy_aria_label: 'Delete Partner' %>
       </div>
     </div>
   </div>

app/views/better_together/events/show.html.erb

@@ -29,16 +29,12 @@
         <% end %>
       </div>
       <div class="col-auto">
-        <% if policy(@resource).edit? %>
-          <%= link_to edit_event_path(@resource), class: 'btn btn-outline-primary btn-sm me-2', 'aria-label' => 'Edit Partner' do %>
-            <i class="fas fa-edit"></i> <%= t('globals.edit') %>
-          <% end %>
-        <% end %>
-        <% if policy(@resource).destroy? %>
-          <%= link_to event_path(@resource), data: { turbo_method: :delete, turbo_confirm: t('globals.confirm_delete') }, class: 'btn btn-outline-danger btn-sm', 'aria-label' => 'Delete Record' do %>
-            <i class="fas fa-trash-alt"></i> <%= t('globals.delete') %>
-          <% end %>
-        <% end %>
+        <%= render 'shared/resource_toolbar',
+                   edit_path: policy(@resource).edit? ? edit_event_path(@resource) : nil,
+                   edit_aria_label: 'Edit Partner',
+                   destroy_path: policy(@resource).destroy? ? event_path(@resource) : nil,
+                   destroy_confirm: t('globals.confirm_delete'),
+                   destroy_aria_label: 'Delete Record' %>
       </div>
     </div>
   </div>

app/views/better_together/people/show.html.erb

@@ -26,16 +26,12 @@
         <%= privacy_badge(@person) %>
       </div>
       <div class="col-auto">
-        <% if policy(@person).edit? %>
-          <%= link_to edit_person_path(@person), class: 'btn btn-outline-primary btn-sm me-2', 'aria-label' => 'Edit Profile' do %>
-            <i class="fas fa-edit"></i> <%= t('globals.edit') %>
-          <% end %>
-        <% end %>
-        <% if policy(@person).destroy? %>
-          <%= link_to person_path(@person), method: :delete, data: { turbo_method: :delete, confirm: t('people.confirm_delete') }, class: 'btn btn-outline-danger btn-sm', 'aria-label' => 'Delete Profile' do %>
-            <i class="fas fa-trash-alt"></i> <%= t('globals.delete') %>
-          <% end %>
-        <% end %>
+        <%= render 'shared/resource_toolbar',
+                   edit_path: policy(@person).edit? ? edit_person_path(@person) : nil,
+                   edit_aria_label: 'Edit Profile',
+                   destroy_path: policy(@person).destroy? ? person_path(@person) : nil,
+                   destroy_confirm: t('people.confirm_delete'),
+                   destroy_aria_label: 'Delete Profile' %>
       </div>
     </div>
   </div>

app/views/better_together/platforms/show.html.erb

@@ -24,16 +24,12 @@
         <p class="profile-description"><%= @platform.description.presence || t('globals.no_description') %></p>
       </div>
       <div class="col-auto">
-        <% if policy(@platform).edit? %>
-          <%= link_to edit_platform_path(@platform), class: 'btn btn-outline-primary btn-sm me-2', 'aria-label' => 'Edit Platform' do %>
-            <i class="fas fa-edit"></i> <%= t('globals.edit') %>
-          <% end %>
-        <% end %>
-        <% if policy(@platform).destroy? %>
-          <%= link_to platform_path(@platform), method: :delete, data: { turbo_method: :delete, confirm: t('platforms.confirm_delete') }, class: 'btn btn-outline-danger btn-sm', 'aria-label' => 'Delete Platform' do %>
-            <i class="fas fa-trash-alt"></i> <%= t('globals.delete') %>
-          <% end %>
-        <% end %>
+        <%= render 'shared/resource_toolbar',
+                   edit_path: policy(@platform).edit? ? edit_platform_path(@platform) : nil,
+                   edit_aria_label: 'Edit Platform',
+                   destroy_path: policy(@platform).destroy? ? platform_path(@platform) : nil,
+                   destroy_confirm: t('platforms.confirm_delete'),
+                   destroy_aria_label: 'Delete Platform' %>
       </div>
     </div>
   </div>

app/views/better_together/posts/show.html.erb

@@ -13,15 +13,11 @@
     <% if policy(resource_class).index? %>
       <%= link_to t('better_together.posts.back_to_posts', default: 'Back to Posts'), posts_path, class: 'btn btn-sm btn-outline-secondary' %>
     <% end %>
-    <% if policy(@post).edit? %>
-      <%= link_to edit_post_path(@post), class: 'btn btn-outline-primary btn-sm me-2', 'aria-label' => 'Edit Post' do %>
-        <i class="fas fa-edit"></i> <%= t('globals.edit') %>
-      <% end %>
-    <% end %>
-    <% if policy(@post).destroy? %>
-      <%= link_to post_path(@post), data: { turbo_method: :delete, turbo_confirm: t('globals.confirm_delete') }, class: 'btn btn-outline-danger btn-sm', 'aria-label' => 'Delete Record' do %>
-        <i class="fas fa-trash-alt"></i> <%= t('globals.delete') %>
-      <% end %>
-    <% end %>
+    <%= render 'shared/resource_toolbar',
+               edit_path: policy(@post).edit? ? edit_post_path(@post) : nil,
+               edit_aria_label: 'Edit Post',
+               destroy_path: policy(@post).destroy? ? post_path(@post) : nil,
+               destroy_confirm: t('globals.confirm_delete'),
+               destroy_aria_label: 'Delete Record' %>
   </div>
 </div>

app/views/shared/_resource_toolbar.html.erb

@@ -0,0 +1,27 @@
+<%# Renders a toolbar with optional action buttons for a resource.
+    Locals:
+      edit_path: path for edit action (optional)
+      edit_aria_label: aria-label for edit button (defaults to t('globals.edit'))
+      view_path: path for view action (optional)
+      view_aria_label: aria-label for view button (defaults to t('globals.view'))
+      destroy_path: path for destroy action (optional)
+      destroy_confirm: confirmation message (defaults to t('globals.confirm_delete'))
+      destroy_aria_label: aria-label for destroy button (defaults to t('globals.delete'))
+%>
+<div class="resource-toolbar">
+  <% if local_assigns[:edit_path] %>
+    <%= link_to edit_path, class: 'btn btn-outline-primary btn-sm me-2', 'aria-label' => (local_assigns[:edit_aria_label] || t('globals.edit')) do %>
+      <i class="fas fa-edit"></i> <%= t('globals.edit') %>
+    <% end %>
+  <% end %>
+  <% if local_assigns[:view_path] %>
+    <%= link_to view_path, class: 'btn btn-outline-secondary btn-sm me-2', 'aria-label' => (local_assigns[:view_aria_label] || t('globals.view')) do %>
+      <i class="fas fa-eye"></i> <%= t('globals.view') %>
+    <% end %>
+  <% end %>
+  <% if local_assigns[:destroy_path] %>
+    <%= link_to destroy_path, data: { turbo_method: :delete, turbo_confirm: (local_assigns[:destroy_confirm] || t('globals.confirm_delete')) }, class: 'btn btn-outline-danger btn-sm', 'aria-label' => (local_assigns[:destroy_aria_label] || t('globals.delete')) do %>
+      <i class="fas fa-trash-alt"></i> <%= t('globals.delete') %>
+    <% end %>
+  <% end %>
+</div>

docs/resource_toolbar.md

@@ -0,0 +1,26 @@
+# Resource Toolbar
+
+A shared partial for rendering edit, view, and destroy buttons for a resource.
+
+## Usage
+
+```
+<%= render 'shared/resource_toolbar',
+           edit_path: edit_post_path(@post),
+           view_path: post_path(@post),
+           destroy_path: post_path(@post),
+           destroy_confirm: t('globals.confirm_delete'),
+           edit_aria_label: 'Edit Post',
+           view_aria_label: 'View Post',
+           destroy_aria_label: 'Delete Post' %>
+```
+
+## Locals
+
+- `edit_path` – link for the edit action (optional)
+- `view_path` – link for the view action (optional)
+- `destroy_path` – link for the destroy action (optional)
+- `destroy_confirm` – confirmation text for destroy (defaults to `t('globals.confirm_delete')`)
+- `edit_aria_label`, `view_aria_label`, `destroy_aria_label` – ARIA labels for accessibility.
+
+Buttons render only when the corresponding path is provided. Defaults use the global translations for button text and ARIA labels.

spec/views/shared/resource_toolbar.html.erb_spec.rb

@@ -0,0 +1,29 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+RSpec.describe 'shared/resource_toolbar', type: :view do
+  it 'renders provided action buttons' do
+    render partial: 'shared/resource_toolbar', locals: {
+      edit_path: '/edit',
+      view_path: '/view',
+      destroy_path: '/destroy',
+      destroy_confirm: 'Are you sure?'
+    }
+
+    expect(rendered).to include(t('globals.edit'))
+    expect(rendered).to include(t('globals.view'))
+    expect(rendered).to include(t('globals.delete'))
+    expect(rendered).to include('href="/edit"')
+    expect(rendered).to include('href="/view"')
+    expect(rendered).to include('href="/destroy"')
+  end
+
+  it 'omits buttons when paths are missing' do
+    render partial: 'shared/resource_toolbar'
+
+    expect(rendered).not_to include(t('globals.edit'))
+    expect(rendered).not_to include(t('globals.view'))
+    expect(rendered).not_to include(t('globals.delete'))
+  end
+end