From d47ea400101dbff2b6e59fc900b4dfc399b42026 Mon Sep 17 00:00:00 2001 From: Daniel Imms <2193314+Tyriar@users.noreply.github.com> Date: Sun, 31 May 2026 21:37:53 -0700 Subject: [PATCH] docs: fix comment typos in Constants, DomRenderer, and addons - Constants: "indicating" in combined-content flag JSDoc - DomRenderer: viewportStartRow spelling in comment - SerializeAddon: "simplify" instead of "simply" - addon-ligatures: "terminal" instead of "therminal" Co-authored-by: Cursor --- addons/addon-ligatures/src/index.ts | 2 +- addons/addon-serialize/src/SerializeAddon.ts | 4 ++-- src/browser/renderer/dom/DomRenderer.ts | 2 +- src/common/buffer/Constants.ts | 2 +- 4 files changed, 5 insertions(+), 5 deletions(-) diff --git a/addons/addon-ligatures/src/index.ts b/addons/addon-ligatures/src/index.ts index fb8e85e73..2fc664f5b 100644 --- a/addons/addon-ligatures/src/index.ts +++ b/addons/addon-ligatures/src/index.ts @@ -21,7 +21,7 @@ const CACHE_SIZE = 100000; /** * Enable ligature support for the provided Terminal instance. To function - * properly, this must be called after `open()` is called on the therminal. If + * properly, this must be called after `open()` is called on the terminal. If * the font currently in use supports ligatures, the terminal will automatically * start to render them. * @param term Terminal instance from xterm.js diff --git a/addons/addon-serialize/src/SerializeAddon.ts b/addons/addon-serialize/src/SerializeAddon.ts index 26ede1c41..e1728feb2 100644 --- a/addons/addon-serialize/src/SerializeAddon.ts +++ b/addons/addon-serialize/src/SerializeAddon.ts @@ -221,7 +221,7 @@ class StringSerializeHandler extends BaseSerializeHandler { // you can't use control sequence to move cursor to (x === row) (thisRowLastChar.getChars() || thisRowLastChar.getWidth() === 0) && // change background of the first wrapped cell also affects BCE - // so we mark it as invalid to simply the process to determine line separator + // so we mark it as invalid to simplify the process to determine line separator equalBg(thisRowLastChar, nextRowFirstChar) ) { isValid = true; @@ -233,7 +233,7 @@ class StringSerializeHandler extends BaseSerializeHandler { isNextRowFirstCharDoubleWidth && (thisRowLastSecondChar.getChars() || thisRowLastSecondChar.getWidth() === 0) && // change background of the first wrapped cell also affects BCE - // so we mark it as invalid to simply the process to determine line separator + // so we mark it as invalid to simplify the process to determine line separator equalBg(thisRowLastChar, nextRowFirstChar) && equalBg(thisRowLastSecondChar, nextRowFirstChar) ) { diff --git a/src/browser/renderer/dom/DomRenderer.ts b/src/browser/renderer/dom/DomRenderer.ts index 7623ee356..2771c79d7 100644 --- a/src/browser/renderer/dom/DomRenderer.ts +++ b/src/browser/renderer/dom/DomRenderer.ts @@ -430,7 +430,7 @@ export class DomRenderer extends Disposable implements IRenderer { documentFragment.appendChild(this._createSelectionElement(viewportCappedStartRow + 1, 0, this._bufferService.cols, middleRowsCount)); // Draw final row if (viewportCappedStartRow !== viewportCappedEndRow) { - // Only draw viewportEndRow if it's not the same as viewporttartRow + // Only draw viewportEndRow if it's not the same as viewportStartRow const finalEndCol = viewportEndRow === viewportCappedEndRow ? end[0] : this._bufferService.cols; documentFragment.appendChild(this._createSelectionElement(viewportCappedEndRow, 0, finalEndCol)); } diff --git a/src/common/buffer/Constants.ts b/src/common/buffer/Constants.ts index 5ce075cf7..9f21dbc5a 100644 --- a/src/common/buffer/Constants.ts +++ b/src/common/buffer/Constants.ts @@ -44,7 +44,7 @@ export const enum Content { CODEPOINT_MASK = 0x1FFFFF, /** - * bit 22 flag indication whether a cell contains combined content + * bit 22 flag indicating whether a cell contains combined content * read: `isCombined = content & Content.isCombined;` * set: `content |= Content.isCombined;` * clear: `content &= ~Content.isCombined;`