backstage 0.1.14 → 0.1.16

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: af33e498a31b4178f2565526576a747f3b40f79fe869d0c9f02e7b9147f250ed
-  data.tar.gz: e4a561394fe10ec3ac30a8bcbc3492bd13285881c9e1e1cf2d16a2ef3302e259
+  metadata.gz: c93addd91e80f76d095d7fd56542ebea748d27be3043cad7affcee189143718e
+  data.tar.gz: a9df1684acf09fb155e449e612e1e92c117874b581d4161f4b9c4b3749f610b4
 SHA512:
-  metadata.gz: faf0d674103b06bffd2625b114aec195037f8ab3c08ff6a5c8786abb8b8fd5c53938c6f394a934e10820459089af5c0fcafda82d86ddae9c584081918287ceef
-  data.tar.gz: 1b2ee32beb0c16edfd8a7d2b08f86671ead990cc131272a60b9fd3a027fd8b9b1c5d37314ee7eccec221e6cd037fb9e6706d60dc8f97041dbfd1f11470e1a2ac
+  metadata.gz: 48c81f6f435f100053c573b886ad40f6df175ae871302bbc381c161ade4f99bbe2062505de73133702d8a7c05e679d721e31d0a95f76b6555d3d1923df0f328a
+  data.tar.gz: 6653ba7832ea040c05081c5f46e9b4f8504503ed7bf117a49c7d111e95d144c56edbf9262f7864eed7d12094c482574e31ad0505a33323ea6141601e9d13d4e3

data/CHANGELOG.md

@@ -7,6 +7,26 @@ This project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.htm
 
 ## [Unreleased]
 
+## [0.1.16] — 2026-06-02
+
+### Fixed
+
+- Nested table now uses a `<template>` element instead of a static new row, so unsaved blank rows are never submitted unintentionally
+- New-row template includes readonly fields as plain inputs so new records can be fully populated via the Add button
+- Readonly nested fields are now permitted in Strong Parameters, so new records with readonly fields save correctly
+- `belongs_to :assoc, readonly: true` now propagates `readonly:` to the generated FK field
+
+## [0.1.15] — 2026-05-27
+
+### Added
+
+- `c.nested :assoc, fields: [...]` DSL renders existing `has_many` records as an inline editable table using `accepts_nested_attributes_for`
+- Nested rows include a destroy button (×) and a `_destroy` hidden field; clicking hides the row and marks it for deletion on save
+- A blank add row is always rendered at the bottom of the nested table for creating new records
+- `image_url` fields on index and dashboard tables now render as `<img>` tags instead of raw URLs
+- `belongs_to` fields now render correctly on dashboard tables (was missing)
+- Shared `backstage/shared/_cell` partial extracted for consistent field rendering across index and dashboard views
+
 ## [0.1.14] — 2026-05-26
 
 ### Fixed

data/app/controllers/backstage/resources_controller.rb

@@ -99,8 +99,7 @@ module Backstage
         elsif field.has_many?
           [{field.name => []}]
         elsif field.nested?
-          writable = field.nested_fields - field.nested_readonly_fields
-          [{"#{field.name}_attributes": [:id, *writable]}]
+          [{"#{field.name}_attributes": [:id, :_destroy, *field.nested_fields]}]
         else
           [field.name]
         end

data/app/views/backstage/dashboards/show.html.erb

@@ -13,13 +13,7 @@
     <% @records.each do |record| %>
       <tr id="<%= @resource_config.model_name_param %>_<%= record.id %>_row">
         <% @resource_config.index_fields.each do |field| %>
-          <td>
-            <% if field.enum? %>
-              <%= record.public_send(field.name).to_s.humanize %>
-            <% else %>
-              <%= record.public_send(field.name) %>
-            <% end %>
-          </td>
+          <td><%= render "backstage/shared/cell", field: field, record: record %></td>
         <% end %>
         <td><%= link_to "Edit", edit_resource_path(resource: @resource_config.model_name_param, id: record.id) %></td>
       </tr>

data/app/views/backstage/fields/_nested.html.erb

@@ -1,30 +1,55 @@
 <% records = record.public_send(field.name) %>
-<% if records.any? %>
-  <table>
-    <thead>
+<table>
+  <thead>
+    <tr>
+      <% field.nested_fields.each do |col| %>
+        <th><%= col.to_s.humanize %></th>
+      <% end %>
+      <th></th>
+    </tr>
+  </thead>
+  <tbody>
+    <%= f.fields_for field.name do |nf| %>
       <tr>
+        <%= nf.hidden_field :id %>
+        <%= nf.hidden_field :_destroy, value: "0", data: {nested_destroy_flag: true} %>
         <% field.nested_fields.each do |col| %>
-          <th><%= col.to_s.humanize %></th>
+          <td>
+            <% if field.nested_readonly_fields.include?(col) %>
+              <%= nf.object.public_send(col) %>
+            <% else %>
+              <%= nf.text_field col %>
+            <% end %>
+          </td>
         <% end %>
+        <td>
+          <button type="button" data-nested-destroy aria-label="Remove">&times;</button>
+        </td>
       </tr>
-    </thead>
-    <tbody>
-      <%= f.fields_for field.name do |nf| %>
-        <%= nf.hidden_field :id %>
-        <tr>
-          <% field.nested_fields.each do |col| %>
-            <td>
-              <% if field.nested_readonly_fields.include?(col) %>
-                <%= nf.object.public_send(col) %>
-              <% else %>
-                <%= nf.text_field col %>
-              <% end %>
-            </td>
-          <% end %>
-        </tr>
-      <% end %>
-    </tbody>
-  </table>
-<% else %>
-  <em>None</em>
-<% end %>
+    <% end %>
+    <%# Template is not part of the DOM — never submitted until the user clicks Add %>
+    <template data-nested-template data-next-index="<%= records.size %>">
+      <tr>
+        <% field.nested_fields.each do |col| %>
+          <td>
+            <input type="text"
+                   name="<%= "#{f.object_name}[#{field.name}_attributes][__INDEX__][#{col}]" %>"
+                   value="" />
+          </td>
+        <% end %>
+        <td>
+          <input type="hidden"
+                 name="<%= "#{f.object_name}[#{field.name}_attributes][__INDEX__][_destroy]" %>"
+                 value="0"
+                 data-nested-destroy-flag="true" />
+          <button type="button" data-nested-destroy aria-label="Remove">&times;</button>
+        </td>
+      </tr>
+    </template>
+    <tr>
+      <td colspan="<%= field.nested_fields.size + 1 %>">
+        <button type="button" data-nested-add>+ Add</button>
+      </td>
+    </tr>
+  </tbody>
+</table>

data/app/views/backstage/resources/index.html.erb

@@ -32,20 +32,7 @@
     <% @records.each do |record| %>
       <tr>
         <% @resource_config.index_fields.each do |field| %>
-          <td>
-            <% if field.enum? %>
-              <%= record.public_send(field.name).to_s.humanize %>
-            <% elsif field.belongs_to? %>
-              <% related = record.public_send(field.association.name)
-                 col     = field.association.display_column
-                 label   = related&.public_send(col) %>
-              <% if related %>
-                <%= link_to label, edit_resource_path(resource: related.class.model_name.plural, id: related.id) %>
-              <% end %>
-            <% else %>
-              <%= record.public_send(field.name) %>
-            <% end %>
-          </td>
+          <td><%= render "backstage/shared/cell", field: field, record: record %></td>
         <% end %>
         <td><%= link_to "Edit", edit_resource_path(resource: params[:resource], id: record.id) %></td>
       </tr>

data/app/views/backstage/shared/_cell.html.erb

@@ -0,0 +1,17 @@
+<% if field.image_url? %>
+  <% url = record.public_send(field.name) %>
+  <% if url.present? %>
+    <%= image_tag url, style: "max-height:60px;" %>
+  <% end %>
+<% elsif field.enum? %>
+  <%= record.public_send(field.name).to_s.humanize %>
+<% elsif field.belongs_to? %>
+  <% related = record.public_send(field.association.name)
+     col     = field.association.display_column
+     label   = related&.public_send(col) %>
+  <% if related %>
+    <%= link_to label, edit_resource_path(resource: related.class.model_name.plural, id: related.id) %>
+  <% end %>
+<% else %>
+  <%= record.public_send(field.name) %>
+<% end %>

data/app/views/layouts/backstage/backstage.html.erb

@@ -26,6 +26,24 @@
                  })
           })
         })
+        document.addEventListener("click", function(e) {
+          var destroyBtn = e.target.closest("[data-nested-destroy]")
+          if (destroyBtn) {
+            var row = destroyBtn.closest("tr")
+            var flag = row.querySelector("[data-nested-destroy-flag]")
+            if (flag) flag.value = "1"
+            row.hidden = true
+          }
+          var addBtn = e.target.closest("[data-nested-add]")
+          if (addBtn) {
+            var table = addBtn.closest("table")
+            var tmpl = table.querySelector("[data-nested-template]")
+            var index = parseInt(tmpl.dataset.nextIndex, 10)
+            tmpl.dataset.nextIndex = index + 1
+            var html = tmpl.innerHTML.replace(/__INDEX__/g, index)
+            addBtn.closest("tr").insertAdjacentHTML("beforebegin", html)
+          }
+        })
       })
     </script>
   </head>

data/lib/backstage/field.rb

@@ -36,6 +36,10 @@ module Backstage
       type == :nested
     end
 
+    def image_url?
+      type == :image_url
+    end
+
     def nested_fields
       options[:nested_fields] || []
     end

data/lib/backstage/resource_config.rb

@@ -63,7 +63,7 @@ module Backstage
     def belongs_to(name, **opts)
       assoc = AssociationConfig.new(name, :belongs_to, opts)
       @associations << assoc
-      fk_field = Field.new(assoc.foreign_key, :belongs_to, association: assoc)
+      fk_field = Field.new(assoc.foreign_key, :belongs_to, association: assoc, readonly: opts[:readonly])
       index_field = Field.new(name.to_sym, :belongs_to, association: assoc)
       @edit_fields.reject! { |f| f.name == fk_field.name }
       @index_fields.reject! { |f| f.name == fk_field.name || f.name == index_field.name }

data/lib/backstage/version.rb

@@ -1,3 +1,3 @@
 module Backstage
-  VERSION = "0.1.14"
+  VERSION = "0.1.16"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: backstage
 version: !ruby/object:Gem::Version
-  version: 0.1.14
+  version: 0.1.16
 platform: ruby
 authors:
 - Gareth James
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-05-26 00:00:00.000000000 Z
+date: 2026-06-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: railties
@@ -61,6 +61,7 @@ files:
 - app/views/backstage/resources/edit.html.erb
 - app/views/backstage/resources/index.html.erb
 - app/views/backstage/resources/new.html.erb
+- app/views/backstage/shared/_cell.html.erb
 - app/views/backstage/shared/_pagination.html.erb
 - app/views/layouts/backstage/backstage.html.erb
 - config/routes.rb