filter decorations

src/vs/editor/browser/viewParts/overviewRuler/decorationsOverviewRuler.ts

@@ -66,6 +66,14 @@ class Settings {
 
 		this.themeType = theme.type;
 
+		const overviewRulerEnabled = options.get(EditorOption.overviewRulerEnabled);
+		if (!overviewRulerEnabled) {
+			this.renderBorder = false;
+			this.borderColor = null;
+			this.hideCursor = true;
+
+		}
+
 		const minimapOpts = options.get(EditorOption.minimap);
 		const minimapEnabled = minimapOpts.enabled;
 		const minimapSide = minimapOpts.side;
@@ -416,6 +424,7 @@ export class DecorationsOverviewRuler extends ViewPart {
 		for (const decorationGroup of decorations) {
 			const color = decorationGroup.color;
 			const decorationGroupData = decorationGroup.data;
+			decorationGroup.data
 
 			canvasCtx.fillStyle = color;
 

src/vs/editor/common/viewModel/viewModelImpl.ts

@@ -769,7 +769,16 @@ export class ViewModel extends Disposable implements IViewModel {
 	public getAllOverviewRulerDecorations(theme: EditorTheme): OverviewRulerDecorationsGroup[] {
 		const decorations = this.model.getOverviewRulerDecorations(this._editorId, filterValidationDecorations(this._configuration.options));
 		const result = new OverviewRulerDecorations();
-		for (const decoration of decorations) {
+		const overviewRulerDecorationsEnabled = this._configuration.options.get(EditorOption.fontInfo);
+		const minimalAllowed: string[] = [];
+		const filteredDecorations = decorations.filter(d => {
+			if (overviewRulerDecorationsEnabled) {
+				return true;
+			}
+			return minimalAllowed.includes(d.id);
+		});
+
+		for (const decoration of filteredDecorations) {
 			const decorationOptions = <ModelDecorationOptions>decoration.options;
 			const opts = decorationOptions.overviewRuler;
 			if (!opts) {