Improve XPath copy to clipboard

fizzyduck · Sep 13, 2016 · 1935439 · 1935439
1 parent 47306fd
commit 1935439
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 15 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,4 +1,5 @@
 ## 0.8.4
+* Add keymap to copy the current XPath into the clipboard.
 * Fix #26 - Error trying to dispose the status bar.
 * Fix tag value completion for fields with dots.
 

diff --git a/keymaps/autocomplete-xml.cson b/keymaps/autocomplete-xml.cson
@@ -0,0 +1,2 @@
+'atom-workspace':
+  'ctrl-alt-c': 'autocomplete-xml:copy-XPath-to-clipboard'
diff --git a/lib/keymaps/autocomplete-xml.cson b/lib/keymaps/autocomplete-xml.cson
diff --git a/lib/main.coffee b/lib/main.coffee
@@ -21,15 +21,15 @@ module.exports =
   getProvider: -> provider
 
   activate: (state) ->
-    # Events subscribed to in atom's system can be easily cleaned up with a CompositeDisposable
     @subscriptions = new CompositeDisposable
-    # Register command that toggles this view
-    @subscriptions.add atom.commands.add 'atom-workspace', 'main:copyXpathToClipboard': => @copyXpathToClipboard()
-    @subscriptions.dispose()
+    @subscriptions.add atom.commands.add 'atom-workspace',
+      'autocomplete-xml:copy-XPath-to-clipboard': => @copyXpathToClipboard()
 
   deactivate: ->
     @xpathView?.destroy()
     @xpathView = null
+    @subscriptions?.dispose()
+    @subscription = null
 
   consumeStatusBar: (statusBar) ->
     XPathStatusBarView = require './xpath-statusbar-view'
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,2 @@
		'atom-workspace':
		'ctrl-alt-c': 'autocomplete-xml:copy-XPath-to-clipboard'