glib-web 7.0.0 → 7.0.1.beta1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8ea8c829dd6dc5164f98cf51bfed43321128774d9ac1bbc36e5dc317be1dbee5
-  data.tar.gz: 8a4667dcde7336f585c98759b52995cd97f8c125855858edd36bd129aa987d01
+  metadata.gz: 7a88117fbaf57f748a21abf1ea95fc18fdec09171c7b93e87ac8ef5e02850f93
+  data.tar.gz: d37adcb5e04c61400b5f073ada5a65d0cd5e5f39b62df49c3a28c93c93b06a41
 SHA512:
-  metadata.gz: 367ea48dd6c7f0b8168d0fb097d504f756fad8b92834d3e5d285d05c01d6f9dcdac00478b964b83c7105359b2450638a000ab5adb36848864d7997f1bb07bb2d
-  data.tar.gz: 401c7ebfc8b3e0dc38cac9bc4630d2eb14764130305592ca3b8cebac26532a980f0d67e28af094c02971a8e90f776ac25282bd4d53c147904e24ee8d9699784f
+  metadata.gz: a47b8e5826f8ac505d0e38b4530a9294140dcd8e5280178c636e2abe78d38d4c323a04d02e0915aec9bcaf73441fecb90d5d0f89650059e7cdc9a7658f737b56
+  data.tar.gz: 7c8ce8df263b855e5a09cd4e42d5e0bf039c0b24d9d08b1f6f6767e5d8857f4cd33d1f9b5a272f3a4c8b8aa16291ff26c57f4dfa350d8881d5b79129dd77f2be

data/app/controllers/concerns/glib/json/libs.rb

@@ -96,18 +96,6 @@ module Glib::Json::Libs
 
         json_ui_redirect_to url, status: status
       end
-      # Any other request format (e.g. a browser tab opening a protected file link
-      # `/temp_blobs/:id/report.pdf`, where the filename sets the format) used to raise
-      # ActionController::UnknownFormat -> a bare 406 with an empty body. Mirror the html
-      # branch instead: whatever opened the URL (browser tab, iframe, img) is redirected
-      # to the target, which the browser follows natively.
-      #
-      # MUST stay the LAST block. `negotiate_mime` answers `Accept: */*` requests with
-      # the FIRST declared format, so an `any` block placed before html would capture
-      # ordinary browser page loads and leave request.format as `*/*` for the action.
-      format.any do
-        redirect_to url
-      end
     end
   end
 
@@ -154,17 +142,6 @@ module Glib::Json::Libs
           redirect_to sign_in_url, **redirect_options
         end
       end
-      # Other non-HTML/JSON formats (a protected file link whose filename sets the
-      # request format, an XML export, ...) used to raise UnknownFormat -> a bare 406.
-      # Mirrors the csv branch: a clean gate status in test, a redirect in production.
-      # MUST stay the LAST block -- see glib_redirect_to.
-      format.any do
-        if Rails.env.test?
-          head :unauthorized
-        else
-          redirect_to sign_in_url, **redirect_options
-        end
-      end
     end
   end
 
@@ -208,16 +185,6 @@ module Glib::Json::Libs
           redirect_to user_default_url
         end
       end
-      # Other non-HTML/JSON formats -- mirrors the csv branch (clean gate status in
-      # test, redirect in production); previously raised UnknownFormat -> a bare 406.
-      # MUST stay the LAST block -- see glib_redirect_to.
-      format.any do
-        if Rails.env.test?
-          head :forbidden
-        else
-          redirect_to user_default_url
-        end
-      end
     end
   end
 

data/app/helpers/glib/json_ui/view_builder/panels.rb

@@ -493,23 +493,129 @@ class Glib::JsonUi::ViewBuilder
       end
     end
 
-    # The generic bulk-edit table: form-peer submission, per-row POSTs,
-    # viewCells/viewHeaders, plus the full parse/gate/dedup config so any
-    # consumer can compose specialized import behavior (it replaced the
-    # retired bulkEdit2 and bulkEditSg -- see the component's header comment
-    # for the full mapping).
-    class BulkEdit3 < View
+    class BulkEdit2 < View
       hash :import, required: [:submitUrl, :paramName]
 
       # Override
       def created
-        # bulkEdit3 is a form PEER, not a form child: it answers the same
+        # bulkEdit2 is a form PEER, not a form child: it answers the same
         # forms_submit action a form does (forms/directSubmit) and submits its
         # rows itself to `import.submitUrl`. Inside a form, its repeated
         # per-row input names would collide with the form's FormData scans
         # (duplicate-name warnings, dirty-baseline pollution) while adding
         # nothing to the form's submission -- so, like a nested form, reject
         # the arrangement at render time.
+        if page.current_form
+          raise 'panels_bulkEdit2 is not allowed inside a form; place it as a sibling and target it with forms_submit targetId'
+        end
+      end
+
+      # Maximum number of data rows an uploaded file may contain; files with
+      # more rows are rejected with an error dialog. No limit when unset.
+      int :maxRows
+
+      # string :statusViewIdPrefix
+      array :viewHeaders
+
+      # string :paramNameForFormData
+      string :paramNameForRowId
+      action :onCellChange
+      action :onRowsLoad
+      action :onRowsSelect
+      action :onRowsSucceed
+      action :onSubmitStart
+      action :onSubmitEnd
+
+      # Experimental
+      # panels_builder :accessory, :header, :footer
+
+      def viewCells(values)
+        json.set! 'viewCells' do
+          values.each do |value|
+            value.call(page.view_builder)
+          end
+        end
+      end
+
+      def dataRows(value)
+        json.dataRows value
+      end
+
+      def viewRows(value)
+        json.viewRows value
+      end
+
+    end
+
+    # The Surfguard members-import table: a fork of bulkEdit2's builder for
+    # glib-web-npm's bulkEditSg component (same skeleton — form-peer
+    # submission, per-row POSTs, viewCells/viewHeaders — plus the Surfguard
+    # parse inputs). Deliberately SG-specific; consolidate with BulkEdit2 when
+    # a second consumer proves which props generalize.
+    class BulkEditSg < View
+      hash :import, required: [:submitUrl, :paramName]
+
+      # Override
+      def created
+        # Same form-peer rule as bulkEdit2 (see its comment).
+        if page.current_form
+          raise 'panels_bulkEditSg is not allowed inside a form; place it as a sibling and target it with forms_submit targetId'
+        end
+      end
+
+      # Maximum number of data rows an uploaded file may contain; files with
+      # more rows are rejected with an error dialog. No limit when unset.
+      int :maxRows
+
+      array :viewHeaders
+
+      # string :paramNameForFormData
+      string :paramNameForRowId
+      action :onCellChange
+      action :onRowsLoad
+      action :onRowsSelect
+      action :onRowsSucceed
+      # Fires when non-importable rows drain out of the table on save (they
+      # never POST), so the page can keep its Loaded/Uploaded/Skipped
+      # arithmetic honest.
+      action :onRowsSkip
+      action :onSubmitStart
+      action :onSubmitEnd
+
+      # Surfguard file shape: CSV header => form field. The parser follows
+      # this map, so a header-spelling fix is a one-line server change.
+      hash :headerMap
+      array :requiredHeaders
+      string :statusFilter
+
+      def viewCells(values)
+        json.set! 'viewCells' do
+          values.each do |value|
+            value.call(page.view_builder)
+          end
+        end
+      end
+
+      def dataRows(value)
+        json.dataRows value
+      end
+
+      def viewRows(value)
+        json.viewRows value
+      end
+
+    end
+
+    # The generic bulk-edit table: bulkEdit2's skeleton plus everything
+    # bulkEditSg hard-coded for the Surfguard imports, expressed as config so
+    # any consumer can compose the same behavior (bulkEditSg's replacement --
+    # see the component's header comment for the full mapping).
+    class BulkEdit3 < View
+      hash :import, required: [:submitUrl, :paramName]
+
+      # Override
+      def created
+        # Same form-peer rule as bulkEdit2 (see its comment).
         if page.current_form
           raise 'panels_bulkEdit3 is not allowed inside a form; place it as a sibling and target it with forms_submit targetId'
         end
@@ -535,7 +641,7 @@ class Glib::JsonUi::ViewBuilder
       action :onSubmitEnd
 
       # The file shape. Without it the table takes plain CSVs (header row
-      # first, header names = field ids):
+      # first, header names = field ids -- the bulkEdit2 shape):
       #   detectHeader: { allOf: [...], anyOf: [...], scanRows: N } -- find
       #     the header row BY CONTENT (report preambles survive);
       #   headerMap: { csvHeader => field } -- header-keyed, order-independent
@@ -610,6 +716,36 @@ class Glib::JsonUi::ViewBuilder
 
     end
 
+    # deprecated
+    class BulkEdit < View
+      hash :nextPage
+      hash :export, required: [:label, :fileName]
+      hash :import, required: [:submitUrl, :paramName]
+      action :onScrollToTop
+      action :onScrollToBottom
+      string :statusViewIdPrefix
+      action :onLoadRows
+
+      # Experimental
+      panels_builder :accessory, :header, :footer
+
+      def firstSection(block)
+        json.sections [1] do
+          block.call page.table_section_builder
+        end
+      end
+
+      def sections(blocks)
+        json.sections do
+          blocks.each do |block|
+            json.child! do
+              block.call page.table_section_builder
+            end
+          end
+        end
+      end
+    end
+
     # Scrollable container panel for content that exceeds viewport height.
     #
     # Scroll panels automatically add scrollbars when content overflows.

data/app/models/concerns/glib/snapshot_v2.rb

@@ -12,9 +12,7 @@ require 'hashdiff'
 #   4. ActiveStorage in diff_associations (convert Attached::One/Many to attachment records).
 #   5. same_as_before? derives from the computed diff instead of the instance-local
 #      `snapshot_changed` flag (mapping-web's flag-gating silently dropped a target's
-#      snapshot when taken after the owner's -- see "Snapshot ordering" below, issue #481),
-#      with the flag re-entering as a NEGATIVE signal for first snapshots only (see
-#      `same_as_before?` for why an empty association baseline cannot decide those).
+#      snapshot when taken after the owner's -- see "Snapshot ordering" below, issue #481).
 #
 # ## Snapshot ordering: owner vs association targets
 #
@@ -29,8 +27,7 @@ require 'hashdiff'
 # reading the target back through the association snapshots the persisted state instead.
 #
 # `snapshot_changed` / `attributes_before_save` remain writable instance accessors for
-# call-site use; `snapshot_changed` is consulted by the skip decision only as a NEGATIVE
-# signal on first snapshots (see `same_as_before?`).
+# call-site use, but they are no longer consulted for the skip decision.
 #
 # To get all records of a model, you can use the following query:
 # MyModel.joins(:snapshots).distinct
@@ -63,16 +60,7 @@ module Glib
         # Only set snapshot_changed if not already set (handles case where before_save
         # is called multiple times - e.g., when associated records are saved before parent)
         if snapshot_changed.nil?
-          # For a save with no pending change `attributes_in_database` is empty -- an
-          # unchanged persisted record has no database-side delta -- which would make a
-          # first snapshot diff every attribute as an addition. Capture the full
-          # persisted state instead, so an unchanged save yields an empty item diff and
-          # the nil-guard keeps that baseline stable for later changes on the same
-          # instance. New records keep `attributes_in_database` (their all-nil
-          # originals): that non-empty baseline is exactly why a create's version 1
-          # records the full initial state. `unless` keeps the accessor nil, never
-          # `false`, for call-site nil-safety.
-          self.attributes_before_save = attributes_in_database.presence || (attributes unless new_record?)
+          self.attributes_before_save = attributes_in_database
           self.snapshot_changed = check_snapshot_changed
         end
       end
@@ -137,10 +125,8 @@ module Glib
               }
 
               # Don't create version if same as before. Reuses `calculated_diff` instead of
-              # recomputing it inside `same_as_before?`. `version == 1` (no previous
-              # version exists) tells `same_as_before?` this is a FIRST snapshot, where the
-              # association baseline is empty and the flag carries decision weight.
-              unless same_as_before?(calculated_diff, first_snapshot: version == 1)
+              # recomputing it inside `same_as_before?`.
+              unless same_as_before?(calculated_diff)
                 # No nil guard: active_snapshot's `create_snapshot!` raises on failure and
                 # always returns the snapshot.
                 snapshot = create_snapshot!(**snapshot_obj)
@@ -200,28 +186,13 @@ module Glib
     end
 
     # Decides whether `glib_create_snapshot!` should skip the version write. Derived from
-    # the computed diff (persisted state), not from the instance-local `snapshot_changed`
+    # the computed diff (persisted state), never from the instance-local `snapshot_changed`
     # flag: the flag dies with its instance, and an owner's snapshot replaces association
     # targets with fresh instances (`with_lock`'s reload clears the association cache, then
     # the diff walk re-reads the targets), so flag-gating used to silently drop a target's
-    # follow-up snapshot taken through the association (issue #481).
-    #
-    # The one place the flag still decides: a FIRST snapshot whose association diff is
-    # meaningless. With no previous version, `fetch_snapshot_items(nil)` returns `{}`
-    # for associations, so every pre-existing child row reads as `+` and the diff alone
-    # can never prove a no-op — any never-snapshotted record with children would mint a
-    # version on every snapshot call (observed as spurious "Edited" audit entries on
-    # routine re-saves). There the flag is consulted as a NEGATIVE signal:
-    #   - `false` — this instance crossed the before_save boundary and
-    #     `check_snapshot_changed` found no item or child change, and the item diff is
-    #     net-zero against `attributes_before_save` => skip;
-    #   - `nil` — no save boundary on this instance (an owner-triggered snapshot of a
-    #     replaced association target, issue #481) => mint;
-    #   - `true` — a change was detected at the save boundary; falls through to the
-    #     diff-based decision (a reverted change still no-ops).
-    def same_as_before?(computed_diff = diff, first_snapshot: false)
-      return true if first_snapshot && snapshot_changed == false && computed_diff['item'].blank?
-
+    # follow-up snapshot taken through the association (issue #481). For a first snapshot
+    # (no previous version) the diff runs current state against `attributes_before_save || {}`.
+    def same_as_before?(computed_diff = diff)
       item_unchanged = computed_diff['item'].blank?
       assoc_diff = computed_diff['associations']
       associations_unchanged = assoc_diff.nil? || assoc_diff.values.all?(&:blank?)

data/app/views/json_ui/garage/tables/bulk_edit.json.jbuilder

@@ -37,17 +37,17 @@ page.scroll childViews: ->(scroll) do
         displayed: false
     end
 
-    wres.panels_bulkEdit3 \
+    wres.panels_bulkEdit2 \
       id: 'import_table',
       width: 'matchParent',
       paramNameForRowId: 'row_id',
       import: { submitUrl: json_ui_garage_url(path: 'forms/bulk_edit_post2'), paramName: 'user' },
       viewHeaders: [
-        { text: 'Month', id: 'month', tooltip: { text: 'Month', placement: 'top' } },
-        { text: 'Electricity usage', id: 'electricity_usage' },
-        { text: 'Gas usage', id: 'gas_usage' },
-        { text: 'Sources', id: 'sources' },
-        { text: 'Compliant', id: 'compliant' }
+        { text: 'Month', id: 'month', importable: true, tooltip: { text: 'Month', placement: 'top' } },
+        { text: 'Electricity usage', id: 'electricity_usage', importable: true },
+        { text: 'Gas usage', id: 'gas_usage', importable: true },
+        { text: 'Sources', id: 'sources', importable: true },
+        { text: 'Compliant', id: 'compliant', importable: true }
       ],
       viewCells: [
         ->(view) do

data/app/views/json_ui/garage/test_page/_header.json.jbuilder

@@ -56,6 +56,7 @@ nav_groups = {
     'flow',
     'grid',
     'horizontal',
+    'panels_bulkEdit2',
     'pagination',
     'responsive',
     'scroll',

data/app/views/json_ui/garage/test_page/panels_bulkEdit2.json.jbuilder

@@ -0,0 +1,244 @@
+json.title 'Test Page (Panels Bulk Edit 2)'
+
+page = json_ui_page json
+
+options = [{ text: 'yes', value: 'yes' }, { text: 'no', value: 'no' }]
+source_options = ['Generators', 'Power Station', 'Solar Panels'].map do |source|
+  { text: source, value: source }
+end
+
+default_headers = [
+  { text: 'Month', id: 'month', importable: true, tooltip: { text: 'Month', placement: 'top' } },
+  { text: 'Electricity usage', id: 'electricity_usage', importable: true },
+  { text: 'Gas usage', id: 'gas_usage', importable: true },
+  { text: 'Sources', id: 'sources', importable: true },
+  { text: 'Compliant', id: 'compliant', importable: true }
+]
+
+default_cells = [
+  ->(view) do
+    view.fields_number width: 'matchParent'
+  end,
+  ->(view) do
+    view.fields_number width: 'matchParent'
+  end,
+  ->(view) do
+    view.fields_number width: 'matchParent'
+  end,
+  ->(view) do
+    view.fields_select width: 'matchParent', options: source_options, multiple: true
+  end,
+  ->(view) do
+    view.fields_select width: 'matchParent', options: options
+  end
+]
+
+[
+  { text: 'Month', id: 'month', importable: true },
+  { text: 'Usage (kWh)', id: 'electricity_usage', importable: true },
+  { text: 'Usage (Therms)', id: 'gas_usage', importable: true }
+]
+
+[
+  ->(view) do
+    view.fields_number width: 120
+  end,
+  ->(view) do
+    view.fields_number width: 140
+  end,
+  ->(view) do
+    view.fields_number width: 140
+  end
+]
+
+import_settings = { submitUrl: json_ui_garage_url(path: 'forms/bulk_edit_post2'), paramName: 'user' }
+
+render 'json_ui/garage/test_page/header', json: json, page: page
+
+page.body(
+  childViews: ->(body) do
+    body.panels_responsive(
+      padding: glib_json_padding_body,
+      childViews: ->(res) do
+        res.h2 text: 'Overview'
+        res.p text: 'panels_bulkEdit2 renders an editable grid with import-backed rows and bulk submission events.'
+        res.spacer height: 12
+        res.hr width: 'matchParent'
+        res.h2 text: 'Basic example'
+        res.spacer height: 8
+        res.panels_horizontal(
+          align: 'middle',
+          childViews: ->(horizontal) do
+            horizontal.label text: 'Loaded: '
+            horizontal.label id: 'bulk_edit_loaded_count', text: '0'
+            horizontal.spacer width: 12
+            horizontal.label text: 'Selected: '
+            horizontal.label id: 'bulk_edit_selected_count', text: '0'
+            horizontal.spacer width: 12
+            horizontal.chip id: 'bulk_edit_succeeded_count', styleClass: 'success', text: 'Uploaded: 0', displayed: false
+          end
+        )
+        res.spacer height: 8
+        res.panels_bulkEdit2(
+          id: 'bulk_edit_main',
+          width: 'matchParent',
+          paramNameForRowId: 'row_id',
+          # Files with more data rows than this are rejected wholesale with an
+          # error dialog (the header row doesn't count).
+          maxRows: 3,
+          import: import_settings,
+          viewHeaders: default_headers,
+          viewCells: default_cells,
+          onCellChange: ->(action) do
+            action.snackbars_alert debug: true, paramName: 'message', fieldName: 'row'
+          end,
+          onSubmitStart: ->(action) do
+            action.runMultiple(
+              childActions: ->(multiple) do
+                multiple.logics_set targetId: 'bulk_edit_submit_top', data: { disabled: true }
+                multiple.logics_set targetId: 'bulk_edit_submit_bottom', data: { disabled: true }
+                multiple.logics_set targetId: 'bulk_edit_cancel', data: { displayed: true }
+              end
+            )
+          end,
+          onSubmitEnd: ->(action) do
+            action.runMultiple(
+              childActions: ->(multiple) do
+                multiple.logics_set targetId: 'bulk_edit_submit_top', data: { disabled: false }
+                multiple.logics_set targetId: 'bulk_edit_submit_bottom', data: { disabled: false }
+                multiple.logics_set targetId: 'bulk_edit_cancel', data: { displayed: false }
+              end
+            )
+          end,
+          onRowsLoad: ->(action) do
+            action.runMultiple(
+              debug: true,
+              fieldName: 'data',
+              childActions: ->(multiple) do
+                multiple.logics_set(
+                  targetId: 'bulk_edit_loaded_count',
+                  debug: true,
+                  paramName: 'variables',
+                  fieldName: 'data',
+                  conditionalData: {
+                  text: { "count": [{ "var": 'data.loadedRows' }] }
+                  }
+                )
+                multiple.logics_set targetId: 'bulk_edit_submit_top', data: { displayed: true }
+                multiple.logics_set targetId: 'bulk_edit_submit_bottom', data: { displayed: true }
+              end
+            )
+          end,
+          onRowsSelect: ->(action) do
+            action.logics_set(
+              targetId: 'bulk_edit_selected_count',
+              debug: true,
+              paramName: 'variables',
+              fieldName: 'data',
+              conditionalData: {
+              text: { "count": [{ "var": 'data.selectedRows' }] }
+              }
+            )
+          end,
+          onRowsSucceed: ->(action) do
+            action.logics_set(
+              targetId: 'bulk_edit_succeeded_count',
+              debug: true,
+              paramName: 'variables',
+              fieldName: 'data',
+              conditionalData: {
+              text: {
+              "printf": [
+              'Uploaded: {0}',
+              { "count": [{ "var": 'data.succeededRows' }] }
+              ]
+              },
+              displayed: true
+              }
+            )
+          end
+        )
+        res.spacer height: 12
+        res.panels_flow(
+          innerPadding: { bottom: 0 },
+          childViews: ->(flow) do
+            flow.button(
+              id: 'bulk_edit_submit_top',
+              text: 'Submit (top)',
+              displayed: false,
+              onClick: ->(action) do
+                action.forms_submit targetId: 'bulk_edit_main'
+              end
+            )
+            flow.spacer width: 8
+            flow.button(
+              id: 'bulk_edit_submit_bottom',
+              text: 'Submit (bottom)',
+              displayed: false,
+              onClick: ->(action) do
+                action.forms_submit targetId: 'bulk_edit_main'
+              end
+            )
+            flow.spacer width: 8
+            flow.button(
+              id: 'bulk_edit_cancel',
+              text: 'Cancel import',
+              displayed: false,
+              onClick: ->(action) do
+                action.components_invoke targetId: 'bulk_edit_main', name: 'cancelImport'
+              end
+            )
+          end
+        )
+        res.spacer height: 12
+        res.hr width: 'matchParent'
+        # res.h2 text: 'Variants/props'
+        # res.spacer height: 8
+        # res.panels_flow innerPadding: { bottom: 0 }, childViews: ->(flow) do
+        #   flow.button text: 'Default columns', onClick: ->(action) do
+        #     action.logics_set targetId: 'bulk_edit_main', data: { viewHeaders: default_headers, viewCells: default_cells }
+        #   end
+        #   flow.spacer width: 8
+        #   flow.button text: 'Compact columns', onClick: ->(action) do
+        #     action.logics_set targetId: 'bulk_edit_main', data: { viewHeaders: compact_headers, viewCells: compact_cells }
+        #   end
+        #   flow.spacer width: 8
+        #   flow.button text: 'Reset counts', onClick: ->(action) do
+        #     action.runMultiple childActions: ->(multiple) do
+        #       multiple.logics_set targetId: 'bulk_edit_loaded_count', data: { text: '0' }
+        #       multiple.logics_set targetId: 'bulk_edit_selected_count', data: { text: '0' }
+        #       multiple.logics_set targetId: 'bulk_edit_succeeded_count', data: { text: 'Uploaded: 0', displayed: false }
+        #     end
+        #   end
+        # end
+        # res.spacer height: 12
+        # res.hr width: 'matchParent'
+        res.h2 text: 'Actions/events'
+        res.spacer height: 8
+        res.panels_flow(
+          innerPadding: { bottom: 0 },
+          childViews: ->(flow) do
+            flow.button(
+              text: 'Submit (forms_submit)',
+              onClick: ->(action) do
+                action.forms_submit targetId: 'bulk_edit_main'
+              end
+            )
+            flow.spacer width: 8
+            flow.button(
+              text: 'Cancel import (invoke)',
+              onClick: ->(action) do
+                action.components_invoke targetId: 'bulk_edit_main', name: 'cancelImport'
+              end
+            )
+          end
+        )
+        res.spacer height: 12
+        res.hr width: 'matchParent'
+        res.h2 text: 'Edge/advanced'
+        res.spacer height: 8
+        res.label text: 'Swap to compact headers then try editing rows before submitting.'
+      end
+    )
+  end
+)

data/app/views/json_ui/garage/test_page/panels_bulkEdit3.json.jbuilder

@@ -83,13 +83,13 @@ page.body(
       padding: glib_json_padding_body,
       childViews: ->(res) do
         res.h2 text: 'Overview'
-        res.p text: 'panels_bulkEdit3 is the generic bulk-edit table: cells, pagination and one-POST-per-row submission, plus config-driven file parsing, row gates, read-only columns, view chips and submission extras.'
+        res.p text: 'panels_bulkEdit3 is the generic bulk-edit table: bulkEdit2’s skeleton plus config-driven file parsing, row gates, read-only columns, view chips and submission extras.'
         res.spacer height: 12
         res.hr width: 'matchParent'
 
         res.h2 text: 'Basic example (plain CSV)'
         res.spacer height: 8
-        res.p text: 'Without parse config the table takes plain CSVs: header row first, header names as field ids. maxRows 3 rejects oversized files whole.'
+        res.p text: 'Without parse config the table takes plain CSVs: header row first, header names as field ids (the bulkEdit2 shape). maxRows 3 rejects oversized files whole.'
         res.spacer height: 8
         res.panels_horizontal(
           align: 'middle',

data/lib/glib/json_crawler/action_crawlers/forms_submit.rb

@@ -16,7 +16,7 @@ module Glib
         raise "Submit action needs to be inside a form: #{http.router.last_log}" unless form
 
         case Glib::JsonUi::Name.normalize(form['view'])
-        when 'panels/bulkEdit3'
+        when 'panels/bulkEdit2', 'panels/bulkEdit3'
           method = 'post'
           action = "forms/#{method}"
           url = form['import']['submitUrl']

data/lib/glib/test_helpers.rb

@@ -239,6 +239,7 @@ module Glib
         name = Glib::JsonUi::Name.normalize(view['view'])
         name&.start_with?('panels/form') ||
           name&.start_with?('panels/fullPageForm') ||
+          name == 'panels/bulkEdit2' ||
           name == 'panels/bulkEdit3' ||
           name == 'panels/splitForm'
       end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: glib-web
 version: !ruby/object:Gem::Version
-  version: 7.0.0
+  version: 7.0.1.beta1
 platform: ruby
 authors:
 - ''
@@ -497,6 +497,7 @@ files:
 - app/views/json_ui/garage/test_page/multimedia_video.json.jbuilder
 - app/views/json_ui/garage/test_page/pagination.json.jbuilder
 - app/views/json_ui/garage/test_page/panels.json.jbuilder
+- app/views/json_ui/garage/test_page/panels_bulkEdit2.json.jbuilder
 - app/views/json_ui/garage/test_page/panels_bulkEdit3.json.jbuilder
 - app/views/json_ui/garage/test_page/popovers.json.jbuilder
 - app/views/json_ui/garage/test_page/progressCircle.json.jbuilder