@operato/data-grist 10.29.17 → 10.29.20

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@operato/data-grist",
-  "version": "10.29.17",
+  "version": "10.29.20",
   "description": "User interface for grid (desktop) and list (mobile)",
   "author": "heartyoh",
   "type": "module",
@@ -70,14 +70,14 @@
     "@material/web": "^2.0.0",
     "@operato/headroom": "^10.29.1",
     "@operato/i18n": "^10.29.1",
-    "@operato/input": "^10.29.17",
-    "@operato/list-state": "^10.29.1",
-    "@operato/p13n": "^10.29.17",
-    "@operato/popup": "^10.29.17",
+    "@operato/input": "^10.29.20",
+    "@operato/list-state": "^10.29.20",
+    "@operato/p13n": "^10.29.20",
+    "@operato/popup": "^10.29.20",
     "@operato/pull-to-refresh": "^10.29.1",
     "@operato/styles": "^10.29.17",
     "@operato/time-calculator": "^10.29.1",
-    "@operato/utils": "^10.29.1",
+    "@operato/utils": "^10.29.20",
     "i18next": "^24.2.1",
     "json5": "^2.2.0",
     "lit": "^3.1.2",
@@ -116,5 +116,5 @@
       "prettier --write"
     ]
   },
-  "gitHead": "ace66019af9afb5ba5dc50da8ade3a962f2c8cbb"
+  "gitHead": "d1f454a95c5aead1bbe2169d107c7b6f5f98c525"
 }

package/test/record-view-popup-editor.test.ts

@@ -0,0 +1,68 @@
+/* Elements are compared by presence, not by value: a failing element comparison makes chai render the DOM and hang the runner. */
+import { expect, fixture } from '@open-wc/testing'
+import { html } from 'lit'
+
+import '../src/record-view/record-view.js'
+import { buildColumn } from '../src/configure/column-builder.js'
+import { OxGristEditor } from '../src/editors/ox-grist-editor.js'
+import { registerEditor } from '../src/editors/registry.js'
+
+/*
+ * In a form (ox-record-view - the 「만들기」 dialog of a grid) a field is pressed to be edited. A field whose editor
+ * opens a popup used to become 「editing」 on the first press but open nothing until its editor was pressed again,
+ * so what was typed after the first press went nowhere (pdf template 「만들기」, body HTML, 2026-10-10).
+ */
+
+let opened = 0
+
+class ProbePopupEditor extends OxGristEditor {
+  static opensPopup = true
+
+  _onclick(e: Event): void {
+    e.stopPropagation()
+    opened++
+  }
+}
+customElements.define('probe-popup-editor', ProbePopupEditor)
+registerEditor('probe-popup', ProbePopupEditor)
+
+const settle = (ms = 100) => new Promise(resolve => setTimeout(resolve, ms))
+
+describe('a popup field in a form', () => {
+  it('opens its popup on the first press', async () => {
+    opened = 0
+    const columns = [
+      buildColumn({ type: 'string', name: 'name', header: 'name', record: { editable: true } } as any),
+      buildColumn({ type: 'probe-popup', name: 'body', header: 'body', record: { editable: true } } as any)
+    ]
+    const view = (await fixture(html`<ox-record-view .columns=${columns} .record=${{}}></ox-record-view>`)) as any
+    await view.updateComplete
+    await settle()
+
+    const body = view.renderRoot.querySelector('ox-record-view-body') as any
+    await body.updateComplete
+    const field = Array.from(body.renderRoot.querySelectorAll('ox-grid-field') as NodeListOf<any>).find(one => one.column?.name === 'body')
+    expect(!!field, 'the body field is drawn').to.equal(true)
+
+    field.click()
+    await settle(200)
+
+    expect(field.getAttribute('editing'), 'the field is being edited').to.equal('true')
+    expect(opened, 'the popup editor was opened once').to.equal(1)
+  })
+
+  it('leaves a field edited in place as it was - nothing extra is pressed', async () => {
+    opened = 0
+    const columns = [buildColumn({ type: 'string', name: 'name', header: 'name', record: { editable: true } } as any)]
+    const view = (await fixture(html`<ox-record-view .columns=${columns} .record=${{}}></ox-record-view>`)) as any
+    await view.updateComplete
+    await settle()
+    const body = view.renderRoot.querySelector('ox-record-view-body') as any
+    await body.updateComplete
+    const field = body.renderRoot.querySelector('ox-grid-field') as any
+    field.click()
+    await settle(200)
+    expect(field.getAttribute('editing')).to.equal('true')
+    expect(opened).to.equal(0)
+  })
+})