be56a9139f
Some arguments for UI components are optional, so we don't have to provide all of them when using them. I had added a `defined?(name)` check to use default values when an argument is not provided. Jeremy pointed out that it was working by accident. In the code `name = defined?(name)`, `defined?(name)` always returns `local-variable` because it’s on the right-hand side of the assignment, while the local variable is on the left-hand side. When we remove the `defined?` calls, the remaining code is equivalent to `||=`.
27 lines
1.0 KiB
Plaintext
27 lines
1.0 KiB
Plaintext
<% name ||= nil
|
|
value = flash.dig("old", name) || value
|
|
label ||= nil
|
|
error ||= flash.dig("errors", name)
|
|
description ||= nil
|
|
attributes ||= {} %>
|
|
|
|
<div class="space-y-2">
|
|
<% if label %>
|
|
<label for="<%= name %>" class="block text-sm font-medium leading-6 text-gray-900"><%= label %></label>
|
|
<% end %>
|
|
<textarea
|
|
id="<%= name %>"
|
|
name="<%= name %>"
|
|
class="block w-full rounded-md border-0 sm:py-1.5 shadow-sm ring-1 ring-inset focus:ring-2 focus:ring-inset sm:text-sm sm:leading-6 <%= error ? "text-red-900 ring-red-300 placeholder:text-red-300 focus:ring-red-500" : "text-gray-900 ring-gray-300 placeholder:text-gray-400 focus:ring-orange-600"%>"
|
|
<% attributes.each do |atr_key, atr_value| %>
|
|
<%= atr_key %>="<%= atr_value %>"
|
|
<% end%>
|
|
><%= value %></textarea>
|
|
<% if error %>
|
|
<p class="text-sm text-red-600 leading-6" id="<%= name %>-error"><%= error %></p>
|
|
<% end %>
|
|
<% if description %>
|
|
<p class="text-sm text-gray-500 leading-6" id="<%= name %>-description"><%= description %></p>
|
|
<% end %>
|
|
</div>
|