Table cells are always inputs

[ignaciosy]

Related to #15

There is no more hidding/showing text and inputs. Form inputs are now always shown and are styled to fit the table properly.

Screen.Recording.2025-01-15.at.16.37.04.mov

[simon-isler]

I'm not sure about this change. The readonly-attribute div was intended to be used for readonly attributes, that should not be displayed with a form input. #23 is related to this

[ignaciosy]

I'm not sure about this change. The readonly-attribute div was intended to be used for readonly attributes, that should not be displayed with a form input. #23 is related to this

Non-editable attributes are a separate feature, not implemented yet, and those will not be inputs of course. The point here is to avoid swapping between text blocks and inputs, which makes the screen flicker and other styling differences. This also makes it natively available to tab through the cells because the user can now tab through all inputs.

For read-only attributes I imagine table-cells to be conditionally rendered like:

<td>
    <% if attribute.editable? %>
        <%= render "editable_attribute", model: @model, record: record, attribute: attribute %>
    <% else %>
        <span><%= attribute %></span>
    <% end %>
</td>

[simon-isler]

We intended the readonly-attribute exactly for this feature. I wouldn't remove it now

Disregard, let's go with this simplified version for now and see 👍🏼