Copy item's command-name and display-name from contextMenu

Author: t9md

lib/command-palette-view.js

@@ -94,12 +94,43 @@ export default class CommandPaletteView {
       }
     })
     this.selectListView.element.classList.add('command-palette')
+    this.registerCommands()
   }
 
   async destroy () {
     await this.selectListView.destroy()
   }
 
+  copyItemText (propertyName, event) {
+    const item = event
+      ? this.selectListView.getItemForElement(event.target.closest("li")) // by contextMenu
+      : this.selectListView.getSelectedItem() // by keyborad shortcut
+    if (item) atom.clipboard.write("`" + item[propertyName] + "`")
+  }
+
+  registerCommands () {
+    atom.commands.add(this.selectListView.element, {
+      "command-palette:copy-display-name": () => this.copyItemText("displayName"),
+      "command-palette:copy-display-name-from-context-menu": event => this.copyItemText("displayName", event),
+      "command-palette:copy-command-name": () => this.copyItemText("name"),
+      "command-palette:copy-command-name-from-context-menu": event => this.copyItemText("name", event)
+    })
+    const queryEditor = this.selectListView.refs.queryEditor
+    atom.commands.add(queryEditor.element, {
+      "core:copy": event => {
+        const propertyToCopy = atom.config.get('command-palette.copySelectedItemTextWhenEmptySelection')
+        if (propertyToCopy !== "none" && queryEditor.getSelectedBufferRange().isEmpty()) {
+          event.stopImmediatePropagation()
+          if (propertyToCopy === "display-name") {
+            this.copyItemText("displayName")
+          } else if (propertyToCopy === "command-name") {
+            this.copyItemText("name")
+          }
+        }
+      }
+    })
+  }
+
   toggle () {
     if (this.panel && this.panel.isVisible()) {
       this.hide()

menus/command-palette.cson

@@ -23,3 +23,9 @@
     ]
   }
 ]
+
+'context-menu':
+  '.command-palette .list-group': [
+    {label: 'Copy Display Name', command: 'command-palette:copy-display-name-from-context-menu'}
+    {label: 'Copy Command Name', command: 'command-palette:copy-command-name-from-context-menu'}
+  ]

package.json

@@ -15,7 +15,7 @@
   },
   "atomTestRunner": "atom-mocha-test-runner",
   "dependencies": {
-    "atom-select-list": "^0.7.0",
+    "atom-select-list": "atom/atom-select-list#24/head",
     "fuzzaldrin": "^2.1.0",
     "fuzzaldrin-plus": "^0.4.1",
     "underscore-plus": "^1.0.0"
@@ -29,14 +29,23 @@
   },
   "configSchema": {
     "useAlternateScoring": {
+      "order": 0,
       "type": "boolean",
       "default": true,
       "description": "Use an alternative scoring approach which prefers run of consecutive characters, acronyms and start of words."
     },
     "preserveLastSearch": {
+      "order": 1,
       "type": "boolean",
       "default": false,
       "description": "Preserve the last search when reopening the command palette."
+    },
+    "copySelectedItemTextWhenEmptySelection": {
+      "order": 2,
+      "type": "string",
+      "default": "none",
+      "enum": ["none", "command-name", "display-name"],
+      "description": "Copy selected item's command name or display name by `core:copy`."
     }
   }
 }

test/command-palette-view.test.js

@@ -351,4 +351,119 @@ describe('CommandPaletteView', () => {
       })
     })
   })
+
+  describe('`command-palette:copy-display-name` and `command-palette:copy-command-name`', () => {
+    beforeEach(() => {
+      atom.commands.add('*', {
+        'test-xxxxx-command:command-one': () => {},
+        'test-xxxxx-command:command-two': () => {},
+        'test-xxxxx-command:command-three': () => {},
+        'test-xxxxx-command:command-four': () => {},
+        'test-xxxxx-command:command-five': () => {},
+      })
+    })
+
+    it('[by keyboard shortcut] copy name for selected item', async () => {
+      const spy = sandbox.spy(atom.clipboard, 'write')
+
+      const commandPalette = new CommandPaletteView()
+      const selectListView = commandPalette.selectListView
+      await commandPalette.toggle()
+
+      selectListView.selectItem(selectListView.items.find(item => item.name === "test-xxxxx-command:command-one"))
+      atom.commands.dispatch(selectListView.element, 'command-palette:copy-display-name')
+      assert(spy.calledWith("`Test Xxxxx Command: Command One`"))
+      atom.commands.dispatch(selectListView.element, 'command-palette:copy-command-name')
+      assert(spy.calledWith("`test-xxxxx-command:command-one`"))
+
+      spy.reset()
+      selectListView.selectItem(selectListView.items.find(item => item.name === "test-xxxxx-command:command-two"))
+      atom.commands.dispatch(selectListView.element, 'command-palette:copy-display-name')
+      assert(spy.calledWith("`Test Xxxxx Command: Command Two`"))
+      atom.commands.dispatch(selectListView.element, 'command-palette:copy-command-name')
+      assert(spy.calledWith("`test-xxxxx-command:command-two`"))
+    })
+
+    it('[by context menu] copy name for selected item', async () => {
+      const spy = sandbox.spy(atom.clipboard, 'write')
+
+      const commandPalette = new CommandPaletteView()
+      const selectListView = commandPalette.selectListView
+      await commandPalette.toggle()
+
+      selectListView.refs.queryEditor.setText('xxxxx')
+      await selectListView.update()
+      const elements = Array.from(selectListView.refs.items.children)
+      assert.equal(elements.length, 5)
+
+      const elementOne = selectListView.element.querySelector("[data-event-name='test-xxxxx-command:command-one']")
+      atom.commands.dispatch(elementOne, 'command-palette:copy-display-name-from-context-menu')
+      assert(spy.calledWith("`Test Xxxxx Command: Command One`"))
+      atom.commands.dispatch(elementOne, 'command-palette:copy-command-name-from-context-menu')
+      assert(spy.calledWith("`test-xxxxx-command:command-one`"))
+
+      spy.reset()
+      const elementTwo = selectListView.element.querySelector("[data-event-name='test-xxxxx-command:command-two']")
+      atom.commands.dispatch(elementTwo, 'command-palette:copy-display-name-from-context-menu')
+      assert(spy.calledWith("`Test Xxxxx Command: Command Two`"))
+      atom.commands.dispatch(elementTwo, 'command-palette:copy-command-name-from-context-menu')
+      assert(spy.calledWith("`test-xxxxx-command:command-two`"))
+
+      spy.reset()
+      const elementThree = selectListView.element.querySelector("[data-event-name='test-xxxxx-command:command-three']")
+      atom.commands.dispatch(elementThree, 'command-palette:copy-display-name-from-context-menu')
+      assert(spy.calledWith("`Test Xxxxx Command: Command Three`"))
+      atom.commands.dispatch(elementThree, 'command-palette:copy-command-name-from-context-menu')
+      assert(spy.calledWith("`test-xxxxx-command:command-three`"))
+
+      spy.reset()
+      const elementFour = selectListView.element.querySelector("[data-event-name='test-xxxxx-command:command-four']")
+      atom.commands.dispatch(elementFour, 'command-palette:copy-display-name-from-context-menu')
+      assert(spy.calledWith("`Test Xxxxx Command: Command Four`"))
+      atom.commands.dispatch(elementFour, 'command-palette:copy-command-name-from-context-menu')
+      assert(spy.calledWith("`test-xxxxx-command:command-four`"))
+
+      spy.reset()
+      const elementFive = selectListView.element.querySelector("[data-event-name='test-xxxxx-command:command-five']")
+      atom.commands.dispatch(elementFive, 'command-palette:copy-display-name-from-context-menu')
+      assert(spy.calledWith("`Test Xxxxx Command: Command Five`"))
+      atom.commands.dispatch(elementFive, 'command-palette:copy-command-name-from-context-menu')
+      assert(spy.calledWith("`test-xxxxx-command:command-five`"))
+    })
+
+    describe('core:copy behavior with command-palette.copySelectedItemTextWhenEmptySelection setting', () => {
+      let spy, selectListView
+      beforeEach(async () => {
+        spy = sandbox.spy(atom.clipboard, 'write')
+
+        const commandPalette = new CommandPaletteView()
+        selectListView = commandPalette.selectListView
+        await commandPalette.toggle()
+
+        selectListView.refs.queryEditor.setText('xxxxx')
+        await selectListView.update()
+        assert.equal(selectListView.getSelectedItem().name, 'test-xxxxx-command:command-one')
+        const elements = Array.from(selectListView.refs.items.children)
+        assert.equal(elements.length, 5)
+      })
+
+      it('[value = "none"]: copy line text', async () => {
+        atom.config.set("command-palette.copySelectedItemTextWhenEmptySelection", "none")
+        atom.commands.dispatch(selectListView.refs.queryEditor.element, 'core:copy')
+        assert(spy.calledWith("xxxxx")) // default behavoir of atom-text-editor
+      })
+
+      it('[value = "display-name"]: copy displayName of selected item', async () => {
+        atom.config.set("command-palette.copySelectedItemTextWhenEmptySelection", "display-name")
+        atom.commands.dispatch(selectListView.refs.queryEditor.element, 'core:copy')
+        assert(spy.calledWith("`Test Xxxxx Command: Command One`"))
+      })
+
+      it('[value = "command-name]": copy command name of selected item', async () => {
+        atom.config.set("command-palette.copySelectedItemTextWhenEmptySelection", "command-name")
+        atom.commands.dispatch(selectListView.refs.queryEditor.element, 'core:copy')
+        assert(spy.calledWith("`test-xxxxx-command:command-one`"))
+      })
+    })
+  })
 })