From 75bf192af579510a757cceff8c9b30a2f2f4d6e2 Mon Sep 17 00:00:00 2001 From: Daniel Imms <2193314+Tyriar@users.noreply.github.com> Date: Wed, 10 Jul 2024 07:03:51 -0700 Subject: [PATCH] Have scroll bar bg react to theme change --- src/browser/Viewport.ts | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/browser/Viewport.ts b/src/browser/Viewport.ts index 211d7504..62415a42 100644 --- a/src/browser/Viewport.ts +++ b/src/browser/Viewport.ts @@ -4,7 +4,7 @@ */ import { IRenderService, IThemeService } from 'browser/services/Services'; -import { EventEmitter } from 'common/EventEmitter'; +import { EventEmitter, runAndSubscribe } from 'common/EventEmitter'; import { Disposable } from 'common/Lifecycle'; import { IBufferService, IOptionsService } from 'common/services/Services'; import { DomScrollableElement } from 'vs/base/browser/ui/scrollbar/scrollableElement'; @@ -52,7 +52,9 @@ export class Viewport extends Disposable{ ], () => this._scrollableElement.updateOptions(this._getMutableOptions()))); this._scrollableElement.setScrollDimensions({ height: 0, scrollHeight: 0 }); - this._scrollableElement.getDomNode().style.backgroundColor = themeService.colors.background.css; + this.register(runAndSubscribe(themeService.onChangeColors, () => { + this._scrollableElement.getDomNode().style.backgroundColor = themeService.colors.background.css; + })); element.appendChild(this._scrollableElement.getDomNode()); this.register(this._bufferService.onResize(() => this._queueSync()));