Release 2.54.1: the first click on Create lands while a tag is still typed

apps/desktop/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@zennotes/desktop",
   "productName": "ZenNotes",
-  "version": "2.54.0",
+  "version": "2.54.1",
   "description": "ZenNotes desktop shell",
   "private": true,
   "main": "./out/main/index.js",

apps/share-viewer/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@zennotes/share-viewer",
   "private": true,
-  "version": "2.54.0",
+  "version": "2.54.1",
   "type": "module",
   "description": "Read-only renderer for publicly shared ZenNotes, embedded by the zennotes.org website",
   "homepage": "https://zennotes.org",

apps/web/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@zennotes/web",
   "private": true,
-  "version": "2.54.0",
+  "version": "2.54.1",
   "type": "module",
   "description": "ZenNotes web client for self-hosted and hosted deployments",
   "homepage": "https://zennotes.org",

package.json

@@ -1,7 +1,7 @@
 {
   "name": "zennotes-monorepo",
   "private": true,
-  "version": "2.54.0",
+  "version": "2.54.1",
   "description": "ZenNotes monorepo for desktop, web, and self-hosted server builds",
   "packageManager": "npm@10.9.2",
   "engines": {

packages/app-core/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@zennotes/app-core",
   "private": true,
-  "version": "2.54.0",
+  "version": "2.54.1",
   "type": "module",
   "exports": {
     "./main": "./src/main.tsx",

packages/app-core/src/components/SearchCreateForm.tsx

@@ -397,6 +397,9 @@ export function SearchCreateForm({
                 variant="ghost"
                 size="sm"
                 className="text-xs"
+                // Same reason as the footer: a blur here commits the typed tag
+                // as a chip, and a chips row that wraps moves this button.
+                onMouseDown={(e) => e.preventDefault()}
                 onClick={() => onOpenExisting(collision.note)}
               >
                 Open it <kbd className="rounded bg-paper-200 px-1 text-ink-500">Shift+↵</kbd>
@@ -484,7 +487,15 @@ export function SearchCreateForm({
             <kbd className="rounded bg-paper-200 px-1">esc</kbd> back
           </span>
         </div>
-        <div className="flex items-center gap-2">
+        <div
+          className="flex items-center gap-2"
+          // The buttons act on click and never need focus. Taking it on
+          // mousedown would blur the field, unmount the folder or tag list
+          // under the fields and move this footer before mouseup, so the
+          // click never fired: the first Create with a tag still typed was
+          // lost. `create()` counts that text, the way Enter does.
+          onMouseDown={(e) => e.preventDefault()}
+        >
           <Button variant="secondary" size="sm" onClick={onBack}>
             Back
           </Button>

packages/app-core/src/components/SearchPalette.test.ts

@@ -456,6 +456,62 @@ describe('SearchPalette: the create row opens a New note form', () => {
     expect(createAndOpen).toHaveBeenCalledWith('inbox', '', { title: 'Runbook', tags: ['oncall'] })
   })
 
+  // A browser moves focus to a pressed button as mousedown's default action,
+  // unless the event is cancelled; jsdom leaves that step to the test. The
+  // blur used to unmount the list under the fields and move the footer
+  // before mouseup, so the click never fired and the first Create was lost.
+  const press = async (button: HTMLElement, focusedField: HTMLInputElement): Promise<void> => {
+    await act(async () => {
+      const uncancelled = button.dispatchEvent(
+        new MouseEvent('mousedown', { bubbles: true, cancelable: true })
+      )
+      if (uncancelled) focusedField.blur()
+      await Promise.resolve()
+    })
+  }
+
+  it('pressing the mouse on Create keeps the Tags field focused, so the click lands and the typed tag counts', async () => {
+    type(search(), 'Runbook')
+    await key(search(), 'Enter', { shiftKey: true })
+    focus(field('tags'))
+    type(field('tags'), 'oncall')
+    expect(formRows('tag')).toEqual(['oncall'])
+
+    await press(createButton(), field('tags'))
+    expect(document.activeElement).toBe(field('tags'))
+    expect(formRows('tag')).toEqual(['oncall'])
+    expect(chipTags()).toEqual([])
+
+    await act(async () => {
+      createButton().click()
+      await Promise.resolve()
+    })
+    expect(createAndOpen).toHaveBeenCalledWith('inbox', '', { title: 'Runbook', tags: ['oncall'] })
+  })
+
+  it('pressing the mouse on Back keeps the folder list until the click', async () => {
+    type(search(), 'Runbook')
+    await key(search(), 'Enter', { shiftKey: true })
+    focus(field('folder'))
+    expect(formRows('folder')).toEqual(['', 'projects', 'projects/ideas', 'quick', 'archive', 'archive/old'])
+
+    const back = [...document.querySelectorAll<HTMLButtonElement>('button')].find(
+      (b) => b.textContent === 'Back'
+    )
+    if (!back) throw new Error('Back button not rendered')
+    await press(back, field('folder'))
+    expect(document.activeElement).toBe(field('folder'))
+    expect(formRows('folder')).toHaveLength(6)
+
+    await act(async () => {
+      back.click()
+      await Promise.resolve()
+    })
+    expect(form()).toBeNull()
+    expect(search().value).toBe('Runbook')
+    expect(createAndOpen).not.toHaveBeenCalled()
+  })
+
   it('Ctrl+Enter or Cmd+Enter creates from any field', async () => {
     type(search(), 'Runbook')
     await key(search(), 'Enter', { shiftKey: true })

packages/bridge-contract/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@zennotes/bridge-contract",
   "private": true,
-  "version": "2.54.0",
+  "version": "2.54.1",
   "type": "module",
   "exports": {
     "./bridge": "./src/bridge.ts",

packages/shared-domain/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@zennotes/shared-domain",
   "private": true,
-  "version": "2.54.0",
+  "version": "2.54.1",
   "type": "module",
   "exports": {
     "./*": "./src/*.ts"

packages/shared-ui/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@zennotes/shared-ui",
   "private": true,
-  "version": "2.54.0",
+  "version": "2.54.1",
   "type": "module",
   "exports": {
     ".": "./src/index.ts"