use specific disposable for commands, fix disable/enable minimap-elem…

…ent attach bug

lib/main.coffee

@@ -89,17 +89,21 @@ class Main
   # Internal: Used only at export time.
   constructor: ->
     @emitter = new Emitter
+
+    # Commands Subscriptions
+    @subscriptionsOfCommands = new CompositeDisposable
+    @subscriptionsOfCommands.add atom.commands.add 'atom-workspace',
+      'minimap:toggle': => @toggle()
+      'minimap:generate-plugin': => @generatePlugin()
+
+    # Other Subscriptions
     @subscriptions = new CompositeDisposable
 
-  # Activates the minimap package.
-  activate: ->
     MinimapElement ?= require './minimap-element'
-
     MinimapElement.registerViewProvider()
 
-    @subscriptions.add atom.commands.add 'atom-workspace',
-      'minimap:toggle': => @toggle()
-      'minimap:generate-plugin': => @generatePlugin()
+  # Activates the minimap package.
+  activate: ->
 
     @toggle() if atom.config.get 'minimap.autoToggle'
 
@@ -132,7 +136,7 @@ class Main
       @editorsMinimaps?.forEach (value, key) =>
         value.destroy()
         @editorsMinimaps.delete(key)
-      @editorsMinimaps = undefined
+      @subscriptions.dispose()
     else
       @toggled = true
       @initSubscriptions()

lib/minimap-element.coffee

@@ -93,6 +93,7 @@ class MinimapElement extends HTMLElement
   attach: ->
     return if @attached
     @swapMinimapPosition()
+    @attached = true
 
   attachToLeft: ->
     root = @getTextEditorElementRoot()