phlex-forms 0.2.9 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f6247172989a97967a471140547a44b29058f77827ce8d699b2f8ae9d0aa60ce
-  data.tar.gz: 5aaeccec4ad938988ee09ad31f8b42ca92bc29f55fd3a5c481a1d1b21f17d1b8
+  metadata.gz: f09c916a2ff918b4abceddb58267e59d2a7584090aabb8daeacfd2112bf36d9f
+  data.tar.gz: b19313a3517a7289a7dc46575e47965ce62849c712cdf27cc6e3fd0cbed13db8
 SHA512:
-  metadata.gz: 4dd96934af47c850597cd9f750bf03f97052301e6d9850239cb864f13fe914d17767b9c00f4b30fb23dcb0ea44c92b4e3d9f463d90e3832823d00dffd6700f70
-  data.tar.gz: 0a8acefb7a3bb47d002633482592aec0aa425b4b5df99d8a360efeb37455e1b9ea4ff9ba8dd918c4b15cdf08aa72ac8ab8bcbda6f37ef5c8d9f6091e966a540e
+  metadata.gz: 557674f0906795a1f9a09e9d815c6b398b1cd8e8dfd60481512815c85cf276dc3372543272771a1b5dd5cb1a20f45c475bf2254c89d3ab303b1f997b9a1df067
+  data.tar.gz: 3994c3583c2c091b0f035df86f3ad9f0428ad7b955cd03bdb8c649756e0840cb474cbba290c916445919fff6afcb73f84f6dca948294a4b7970d2ba43e8b6b30

data/CHANGELOG.md

@@ -7,6 +7,15 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Fixed
+
+- **`FileInput` multiple uploads submit an array** (Rails `file_field` parity):
+  with `multiple:` set, the input name now gets `[]` appended (unless already
+  present), for the standalone component, the `Plain` variant, and the form
+  builder path. Without it the browser collapsed a multi-file selection into
+  ONE scalar param, which a host app's `params.expect(attr: [])` silently
+  discarded — uploads no-opped with a success response.
+
 ### Added
 
 - **`checkbox_group` — batched checkbox group for array-valued fields** (the

data/lib/forms/field.rb

@@ -30,6 +30,12 @@ module Forms
     # PhlexForms::Theme), so the same field renders daisy or plain.
 
     def input(*modifiers, type: :text, **)
+      # type: :hidden reroutes to #hidden so EVERY path to a hidden field lands on
+      # the bare leaf — `f.Input(:token, :hidden)` and `f.Input(:token, type:
+      # :hidden)` would otherwise still emit the styled `input w-full`. Positional
+      # modifiers are dropped with it: a hidden field has no visual variants.
+      return hidden(**) if type.to_s == "hidden"
+
       theme[:input].new(*modifiers, type:, **field_attributes, **)
     end
 
@@ -43,8 +49,13 @@ module Forms
     end
     alias rich_text_area rich_textarea
 
+    # Bare <input type="hidden">, never the styled :input leaf — daisyui's
+    # `.input` beats WebKit's non-!important `input[type=hidden] { display: none }`
+    # and turns the field into a focusable phantom tab stop in Safari. Built from
+    # the explicit accessors (not field_attributes) so the meaningless
+    # `error: invalid?` is never introduced.
     def hidden(**)
-      theme[:input].new(type: :hidden, **field_attributes, **)
+      theme[:hidden].new(name: field_name, id: field_id, value: field_value, **)
     end
 
     # daisyui v5 "icon/text inside the field" wrapper. The block renders the

data/lib/forms/file_input.rb

@@ -8,7 +8,10 @@ module Forms
     def initialize(*modifiers, name: nil, id: nil, multiple: false, accept: nil,
                    error: false, disabled: false, required: false, full_width: true, **attributes)
       @modifiers = normalize_modifiers(modifiers)
-      @name = name
+      # Rails file_field parity: a multiple input needs an array param name, or
+      # the browser collapses the selection into one scalar file — which a host
+      # app's `params.expect(attr: [])` then silently discards.
+      @name = multiple && name && !name.to_s.end_with?("[]") ? "#{name}[]" : name
       @id = id
       @multiple = multiple
       @accept = accept

data/lib/forms/hidden.rb

@@ -0,0 +1,30 @@
+# frozen_string_literal: true
+
+module Forms
+  # A model-bound hidden field: a bare <input type="hidden"> carrying name/id/
+  # value plus whatever the caller passes through. Deliberately NOT a
+  # DelegatedField/DaisyUI::Input like Forms::Input — a hidden field must never
+  # carry visual styling. daisyui's `.input { display: inline-flex }` overrides
+  # WebKit's *non*-!important UA rule `input[type=hidden] { display: none }`, so a
+  # styled hidden field renders as an empty box and joins the keyboard tab order
+  # in Safari (Chromium's UA rule is !important, hiding the bug there).
+  #
+  #   Forms::Hidden.new(name: "user[token]", id: "user_token", value: "abc123")
+  #
+  # Same component under both themes (:hidden role) — there are no styling
+  # classes for a Plain twin to strip. It takes no positional variants and no
+  # error:: a hidden field has no visual or error state.
+  class Hidden < Phlex::HTML
+    def initialize(name: nil, id: nil, value: nil, **attributes)
+      @name = name
+      @id = id
+      @value = value
+      @attributes = attributes
+      super()
+    end
+
+    def view_template
+      input(type: "hidden", name: @name, id: @id, value: @value.to_s, **@attributes)
+    end
+  end
+end

data/lib/phlex_forms/theme.rb

@@ -4,7 +4,9 @@ module PhlexForms
   # A theme maps component roles (:input, :select, :control, ...) to component
   # classes. The binding contract — the leaf initializer signatures
   # (*modifiers, name:, id:, value:, error:, required:, ...) — is the stable
-  # interface, so the same form class renders under any theme.
+  # interface, so the same form class renders under any theme. The one exception
+  # is :hidden, which takes name:/id:/value: only — a hidden field has neither a
+  # visual variant nor an error state.
   #
   # Built-ins:
   #   Theme.daisy — the DaisyUI-delegating Forms::* leaves (default when the
@@ -56,7 +58,7 @@ module PhlexForms
           textarea: Forms::Textarea, rich_textarea: Forms::RichTextarea,
           checkbox: Forms::Checkbox, toggle: Forms::Toggle, radio: Forms::Radio,
           checkbox_group: Forms::CheckboxGroup,
-          file: Forms::FileInput, wrapped_input: Forms::WrappedInput,
+          file: Forms::FileInput, wrapped_input: Forms::WrappedInput, hidden: Forms::Hidden,
           control: Forms::FormControl, label: Forms::Label,
           field_error: Forms::FieldError, field_hint: Forms::FieldHint,
           submit: Forms::Submit, row: Forms::Row, group: Forms::Group,
@@ -74,7 +76,7 @@ module PhlexForms
           textarea: Forms::Plain::Textarea, rich_textarea: Forms::Plain::Textarea,
           checkbox: Forms::Plain::Checkbox, toggle: Forms::Plain::Checkbox, radio: Forms::Plain::Radio,
           checkbox_group: Forms::Plain::CheckboxGroup,
-          file: Forms::Plain::FileInput, wrapped_input: Forms::Plain::WrappedInput,
+          file: Forms::Plain::FileInput, wrapped_input: Forms::Plain::WrappedInput, hidden: Forms::Hidden,
           control: Forms::Plain::Control, label: Forms::Plain::Label,
           field_error: Forms::Plain::FieldError, field_hint: Forms::Plain::FieldHint,
           submit: Forms::Plain::Submit, row: Forms::Plain::Row, group: Forms::Plain::Group,

data/lib/phlex_forms/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module PhlexForms
-  VERSION = "0.2.9"
+  VERSION = "0.3.0"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: phlex-forms
 version: !ruby/object:Gem::Version
-  version: 0.2.9
+  version: 0.3.0
 platform: ruby
 authors:
 - Mikael Henriksson
@@ -128,6 +128,7 @@ files:
 - lib/forms/form.rb
 - lib/forms/form_control.rb
 - lib/forms/group.rb
+- lib/forms/hidden.rb
 - lib/forms/input.rb
 - lib/forms/label.rb
 - lib/forms/live.rb