Use view action in resource toolbar

Author: rsmithlal

app/views/shared/_resource_toolbar.html.erb

@@ -2,8 +2,8 @@
     Locals:
       edit_path: path for edit action (optional)
       edit_aria_label: aria-label for edit button (defaults to t('globals.edit'))
-      preview_path: path for preview action (optional)
-      preview_aria_label: aria-label for preview button (defaults to t('globals.preview'))
+      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'))
@@ -14,9 +14,9 @@
       <i class="fas fa-edit"></i> <%= t('globals.edit') %>
     <% end %>
   <% end %>
-  <% if local_assigns[:preview_path] %>
-    <%= link_to preview_path, class: 'btn btn-outline-secondary btn-sm me-2', 'aria-label' => (local_assigns[:preview_aria_label] || t('globals.preview')) do %>
-      <i class="fas fa-eye"></i> <%= t('globals.preview') %>
+  <% 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] %>

config/locales/en.yml

@@ -1273,7 +1273,6 @@ en:
     destroy: Destroy
     draft: Draft
     edit: Edit
-    preview: Preview
     'false': 'No'
     forms:
       save: Save

config/locales/es.yml

@@ -1259,7 +1259,6 @@ es:
     destroy: Destruir
     draft: Borrador
     edit: Editar
-    preview: Vista previa
     'false': 'No'
     forms:
       save: Guardar

config/locales/fr.yml

@@ -1286,7 +1286,6 @@ fr:
     destroy: Détruire
     draft: Brouillon
     edit: Modifier
-    preview: Aperçu
     'false': Non
     forms:
       save: Enregistrer

docs/resource_toolbar.md

@@ -1,26 +1,26 @@
 # Resource Toolbar
 
-A shared partial for rendering edit, preview, and destroy buttons for a resource.
+A shared partial for rendering edit, view, and destroy buttons for a resource.
 
 ## Usage
 
 ```
 <%= render 'shared/resource_toolbar',
            edit_path: edit_post_path(@post),
-           preview_path: preview_post_path(@post),
+           view_path: post_path(@post),
            destroy_path: post_path(@post),
            destroy_confirm: t('globals.confirm_delete'),
            edit_aria_label: 'Edit Post',
-           preview_aria_label: 'Preview Post',
+           view_aria_label: 'View Post',
            destroy_aria_label: 'Delete Post' %>
 ```
 
 ## Locals
 
 - `edit_path` – link for the edit action (optional)
-- `preview_path` – link for the preview 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`, `preview_aria_label`, `destroy_aria_label` – ARIA labels for accessibility.
+- `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