fix: Optimizations (#53)

Author: aminya

lib/minimap-highlight-selected.js

@@ -49,9 +49,15 @@ module.exports = {
 		this.highlightSelected.onDidRemoveAllMarkers(() => this.markersDestroyed())
 	},
 
+	destryDecorations () {
+		for (let i = 0, len = this.decorations.length; i < len; i++) {
+			this.decorations[i].destroy()
+		}
+	},
+
 	dispose () {
 		if (this.decorations) {
-			this.decorations.forEach(decoration => decoration.destroy())
+			this.destryDecorations()
 		}
 		this.decorations = null
 	},
@@ -60,17 +66,18 @@ module.exports = {
 		if (!selected) { selected = false }
 		const minimap = this.minimap.minimapForEditor(options.editor)
 		if (!minimap) { return }
-		let className = 'highlight-selected'
-		if (selected) { className += ' selected' }
 
-		const decoration = minimap.decorateMarker(options.marker,
-			{ type: 'highlight', class: className })
-		this.decorations.push(decoration)
+		this.decorations.push(
+			minimap.decorateMarker(options.marker, {
+				type: 'highlight',
+				class: `highlight-selected${selected ? ' selected' : ''}`,
+			}),
+		)
 	},
 
 	markersDestroyed () {
 		if (this.decorations) {
-			this.decorations.forEach(decoration => decoration.destroy())
+			this.destryDecorations()
 		}
 		this.decorations = []
 	},

styles/minimap-highlight-selected.less

@@ -5,10 +5,14 @@
 @import "ui-variables";
 @import "syntax-variables";
 
+@contain_all: layout size paint style;
+
 .minimap .highlight-selected {
+  contain: @contain_all;
   background: @text-color-highlight;
 
   &.selected {
+    contain: @contain_all;
     background: fadeout(@text-color-highlight, 50%);
   }
 }