From 02795f8803224e431a019ab058ad2855c2a43e1d Mon Sep 17 00:00:00 2001
From: Daniel Imms <2193314+Tyriar@users.noreply.github.com>
Date: Thu, 24 Mar 2022 08:02:52 -0700
Subject: [PATCH] Don't change overview ruler position based on decorations

The original fix for #3705 ended up revealing this other bug; the canvas
shouldn't be set based on the anchor of its decorations.

Fixes #3705
---
 src/browser/Decorations/OverviewRulerRenderer.ts | 7 -------
 1 file changed, 7 deletions(-)

diff --git a/src/browser/Decorations/OverviewRulerRenderer.ts b/src/browser/Decorations/OverviewRulerRenderer.ts
index 950e2ac8..1407f4bb 100644
--- a/src/browser/Decorations/OverviewRulerRenderer.ts
+++ b/src/browser/Decorations/OverviewRulerRenderer.ts
@@ -141,13 +141,6 @@ export class OverviewRulerRenderer extends Disposable {
   }
 
   private _refreshStyle(decoration: IInternalDecoration): void {
-    if (this._shouldUpdateAnchor) {
-      if (decoration.options.anchor === 'right') {
-        this._canvas.style.right = decoration.options.x ? `${decoration.options.x * this._renderService.dimensions.actualCellWidth}px` : '';
-      } else {
-        this._canvas.style.left = decoration.options.x ? `${decoration.options.x * this._renderService.dimensions.actualCellWidth}px` : '';
-      }
-    }
     if (!decoration.options.overviewRulerOptions) {
       this._decorationElements.delete(decoration);
       return;