diff options
| author | erdgeist <erdgeist@erdgeist.org> | 2026-07-12 23:43:02 +0200 |
|---|---|---|
| committer | erdgeist <erdgeist@erdgeist.org> | 2026-07-12 23:43:02 +0200 |
| commit | 848ce18303f927bfadaf0965a769cae2c62492a0 (patch) | |
| tree | 3cb14888a0df92178cd20037e7fe94f407d5ad2e /app/views/menu_items | |
| parent | b271648f89cba7cafafa1b73b1658b1c1bc0e4b0 (diff) | |
Normalize action-button wording and add icons app-wide
"New X" becomes "Create X" throughout (users, events, assets, menu
items, nodes), matching the verb-first pattern the dashboard's own
signposts already established, with a shared plus icon rather than a
document-flavored one that only made sense next to "post". The
"Destroy"/"destroy"/"Delete" family is normalized to "Destroy"
everywhere, with a shared trash icon; occurrences#index and
pages#index also pick up the destructive button class they'd been
silently missing. Filter and Search convert from submit_tag to
button_tag, the only way either can hold an icon alongside its label.
Edit and the node editor's three dynamic labels (Continue Editing /
Edit Draft / Lock + Edit) share one icon without touching their
wording -- unlike Destroy's family, the state nuance in the text is
real information, not just inconsistent phrasing.
Diffstat (limited to 'app/views/menu_items')
| -rw-r--r-- | app/views/menu_items/index.html.erb | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/app/views/menu_items/index.html.erb b/app/views/menu_items/index.html.erb index e84199b..591eb68 100644 --- a/app/views/menu_items/index.html.erb +++ b/app/views/menu_items/index.html.erb | |||
| @@ -1,6 +1,8 @@ | |||
| 1 | <h1>Menu Items</h1> | 1 | <h1>Menu Items</h1> |
| 2 | 2 | ||
| 3 | <%= link_to 'New menu item', new_menu_item_path, class: 'action_button' %> | 3 | <%= link_to new_menu_item_path, class: 'action_button' do %> |
| 4 | <%= icon("plus", library: "tabler", "aria-hidden": true) %> Create menu item | ||
| 5 | <% end %> | ||
| 4 | 6 | ||
| 5 | <table id="menu_item_list"> | 7 | <table id="menu_item_list"> |
| 6 | <% @menu_items.each do |menu_item| %> | 8 | <% @menu_items.each do |menu_item| %> |
| @@ -11,9 +13,10 @@ | |||
| 11 | <td class="menu_item_title"><%= menu_item.title %></td> | 13 | <td class="menu_item_title"><%= menu_item.title %></td> |
| 12 | <td><%= link_to "Edit", edit_menu_item_path(menu_item) %></td> | 14 | <td><%= link_to "Edit", edit_menu_item_path(menu_item) %></td> |
| 13 | <td> | 15 | <td> |
| 14 | <%= button_to "Delete", menu_item_path(menu_item), | 16 | <%= button_to menu_item_path(menu_item), method: :delete, |
| 15 | method: :delete, | 17 | form: { data: { confirm: "Do you really want to destroy the menu entry?" }, class: 'button_to destructive' } do %> |
| 16 | form: { data: { confirm: "Are you sure?" }, class: 'button_to destructive' } %> | 18 | <%= icon("trash", library: "tabler", "aria-hidden": true) %> Destroy |
| 19 | <% end %> | ||
| 17 | </td> | 20 | </td> |
| 18 | </tr> | 21 | </tr> |
| 19 | <% end %> | 22 | <% end %> |
