From 1e554992f6e66eef349f92ff11e03ec3be442366 Mon Sep 17 00:00:00 2001 From: Daniel Imms <2193314+Tyriar@users.noreply.github.com> Date: Sun, 31 May 2026 20:51:51 -0700 Subject: [PATCH] docs: fix additional comment typos across renderer and selection Correct scrambled FIXME wording in InputHandler, "however" in DomRenderer, duplicate "the" in selection code, "separate" and grammar in DomRendererRowFactory, and "may cause" in WebglRenderer. Co-authored-by: Cursor --- addons/addon-webgl/src/WebglRenderer.ts | 2 +- src/browser/renderer/dom/DomRenderer.ts | 2 +- src/browser/renderer/dom/DomRendererRowFactory.ts | 2 +- src/browser/selection/SelectionModel.ts | 2 +- src/browser/services/SelectionService.ts | 4 ++-- src/common/InputHandler.ts | 2 +- 6 files changed, 7 insertions(+), 7 deletions(-) diff --git a/addons/addon-webgl/src/WebglRenderer.ts b/addons/addon-webgl/src/WebglRenderer.ts index 264bfda6e..d26c6d25e 100644 --- a/addons/addon-webgl/src/WebglRenderer.ts +++ b/addons/addon-webgl/src/WebglRenderer.ts @@ -667,7 +667,7 @@ export class WebglRenderer extends Disposable implements IRenderer { // The the size of the canvas on the page. It's important that this rounds to nearest integer // and not ceils as browsers often have floating point precision issues where // `window.devicePixelRatio` ends up being something like `1.100000023841858` for example, when - // it's actually 1.1. Ceiling may causes blurriness as the backing canvas image is 1 pixel too + // it's actually 1.1. Ceiling may cause blurriness as the backing canvas image is 1 pixel too // large for the canvas element size. this.dimensions.css.canvas.height = Math.round(this.dimensions.device.canvas.height / this._devicePixelRatio); this.dimensions.css.canvas.width = Math.round(this.dimensions.device.canvas.width / this._devicePixelRatio); diff --git a/src/browser/renderer/dom/DomRenderer.ts b/src/browser/renderer/dom/DomRenderer.ts index de08ac0ac..7623ee356 100644 --- a/src/browser/renderer/dom/DomRenderer.ts +++ b/src/browser/renderer/dom/DomRenderer.ts @@ -33,7 +33,7 @@ let nextTerminalId = 1; /** * The standard renderer and fallback for when the webgl addon is slow. This is not meant to be - * particularly fast and will even lack some features such as custom glyphs, hoever this is more + * particularly fast and will even lack some features such as custom glyphs, however this is more * reliable as webgl may not work on some machines. */ export class DomRenderer extends Disposable implements IRenderer { diff --git a/src/browser/renderer/dom/DomRendererRowFactory.ts b/src/browser/renderer/dom/DomRendererRowFactory.ts index 6ae089a43..4477408a2 100644 --- a/src/browser/renderer/dom/DomRendererRowFactory.ts +++ b/src/browser/renderer/dom/DomRendererRowFactory.ts @@ -380,7 +380,7 @@ export class DomRendererRowFactory { if (!isTop && isInSelection) { // If in the selection, force the element to be above the selection to improve contrast and // support opaque selections. The applies background is not actually needed here as - // selection is drawn in a seperate container, the main purpose of this to ensuring minimum + // selection is drawn in a separate container; the main purpose of this is to ensure minimum // contrast ratio bgOverride = this._coreBrowserService.isFocused ? colors.selectionBackgroundOpaque : colors.selectionInactiveBackgroundOpaque; bg = bgOverride.rgba >> 8 & 0xFFFFFF; diff --git a/src/browser/selection/SelectionModel.ts b/src/browser/selection/SelectionModel.ts index 7b62b230b..ab043b8c5 100644 --- a/src/browser/selection/SelectionModel.ts +++ b/src/browser/selection/SelectionModel.ts @@ -88,7 +88,7 @@ export class SelectionModel { return [startPlusLength, this.selectionStart[1]]; } - // Ensure the the word/line is selected after a double/triple click + // Ensure the word/line is selected after a double/triple click if (this.selectionStartLength) { // Select the larger of the two when start and end are on the same line if (this.selectionEnd[1] === this.selectionStart[1]) { diff --git a/src/browser/services/SelectionService.ts b/src/browser/services/SelectionService.ts index 576bdc525..42366e712 100644 --- a/src/browser/services/SelectionService.ts +++ b/src/browser/services/SelectionService.ts @@ -927,7 +927,7 @@ export class SelectionService extends Disposable implements ISelectionService { // Incremenet the end index so it is at the start of the next character endIndex++; - // Calculate the start _column_, converting the the string indexes back to + // Calculate the start _column_, converting the string indexes back to // column coordinates. let start = startIndex // The index of the selection's start char in the line string @@ -935,7 +935,7 @@ export class SelectionService extends Disposable implements ISelectionService { - leftWideCharCount // The number of wide chars left of the initial char + leftLongCharOffset; // The number of additional chars left of the initial char added by columns with strings longer than 1 (emojis) - // Calculate the length in _columns_, converting the the string indexes back + // Calculate the length in _columns_, converting the string indexes back // to column coordinates. let length = Math.min(this._bufferService.cols, // Disallow lengths larger than the terminal cols endIndex // The index of the selection's end char in the line string diff --git a/src/common/InputHandler.ts b/src/common/InputHandler.ts index 0b12dd8e1..9a8b84888 100644 --- a/src/common/InputHandler.ts +++ b/src/common/InputHandler.ts @@ -273,7 +273,7 @@ export class InputHandler extends Disposable implements IInputHandler { this._parser.setExecuteHandler(C0.HT, () => this.tab()); this._parser.setExecuteHandler(C0.SO, () => this.shiftOut()); this._parser.setExecuteHandler(C0.SI, () => this.shiftIn()); - // FIXME: What do to with missing? Old code just added those to print. + // FIXME: What to do with missing? Old code just added those to print. this._parser.setExecuteHandler(C1.IND, () => this.index()); this._parser.setExecuteHandler(C1.NEL, () => this.nextLine());