Merge pull request #4181 from Tyriar/clear_race

Fix race condition that could when clearing atlas on 2+ terminals
xtermjs · Oct 7, 2022 · d948b31 · d948b31
2 parents 9a4d5db + c4eef2f
commit d948b31
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 1 deletion.
diff --git a/addons/xterm-addon-webgl/src/WebglRenderer.ts b/addons/xterm-addon-webgl/src/WebglRenderer.ts
@@ -298,7 +298,6 @@ export class WebglRenderer extends Disposable implements IRenderer {
   public clearTextureAtlas(): void {
     this._charAtlas?.clearTexture();
     this._clearModel(true);
-    this._updateModel(0, this._terminal.rows - 1);
     this._requestRedrawViewport();
   }
 

diff --git a/src/browser/renderer/shared/TextureAtlas.ts b/src/browser/renderer/shared/TextureAtlas.ts
@@ -153,6 +153,7 @@ export class TextureAtlas implements ITextureAtlas {
     this._currentRow.height = 0;
     this._fixedRows.length = 0;
     this._didWarmUp = false;
+    this.hasCanvasChanged = true;
   }
 
   public getRasterizedGlyphCombinedChar(chars: string, bg: number, fg: number, ext: number): IRasterizedGlyph {