Skip to content

Commit 02795f8

Browse files
committed
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
1 parent 8528e8e commit 02795f8

File tree

1 file changed

+0
-7
lines changed

1 file changed

+0
-7
lines changed

src/browser/Decorations/OverviewRulerRenderer.ts

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -141,13 +141,6 @@ export class OverviewRulerRenderer extends Disposable {
141141
}
142142

143143
private _refreshStyle(decoration: IInternalDecoration): void {
144-
if (this._shouldUpdateAnchor) {
145-
if (decoration.options.anchor === 'right') {
146-
this._canvas.style.right = decoration.options.x ? `${decoration.options.x * this._renderService.dimensions.actualCellWidth}px` : '';
147-
} else {
148-
this._canvas.style.left = decoration.options.x ? `${decoration.options.x * this._renderService.dimensions.actualCellWidth}px` : '';
149-
}
150-
}
151144
if (!decoration.options.overviewRulerOptions) {
152145
this._decorationElements.delete(decoration);
153146
return;

0 commit comments

Comments
 (0)