Merge pull request #370 from DisciplinedSoftware/fix/esc-exits-edit-and-selects-node

fix: pressing ESC while editing a node exits edit and selects the node

Author: SSShooter

src/utils/dom.ts

@@ -210,6 +210,11 @@ export const editTopic = function (this: MindElixirInstance, el: Topic) {
       e.preventDefault()
       div.blur()
       this.container.focus()
+    } else if (key === 'Escape') {
+      e.preventDefault()
+      div.textContent = originalContent // Discard edits
+      div.blur()
+      this.container.focus()
     }
   })
 

tests/interaction.spec.ts

@@ -46,6 +46,18 @@ test('Clear and reset', async ({ page, me }) => {
   await me.toHaveScreenshot()
 })
 
+test('ESC cancels edit and discards changes', async ({ page, me }) => {
+  await me.dblclick(topic)
+  await expect(page.locator('#input-box')).toBeVisible()
+  await page.keyboard.insertText('changes to discard')
+  await page.keyboard.press('Escape')
+  await expect(page.locator('#input-box')).toBeHidden()
+  // Original topic text is preserved
+  await expect(page.getByText(topic)).toBeVisible()
+  // Node should remain selected
+  await expect(page.locator('.selected')).toHaveText(topic)
+})
+
 test('Remove Node', async ({ page, me }) => {
   await me.click(childTopic)
   await page.keyboard.press('Delete')