this._mouseOverEntry can be null on resizing header/footer

seen sometimes when debugging moving rows and cols with comments in them Signed-off-by: Caolán McNamara <[email protected]> Change-Id: I20a01f27957b5fa507cd37a1020a93bbba05c630
CollaboraOnline · Oct 29, 2023 · b3ff11b · b3ff11b
1 parent 09c18a4
commit b3ff11b
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/browser/src/control/Control.ColumnHeader.ts b/browser/src/control/Control.ColumnHeader.ts
@@ -254,7 +254,7 @@ export class ColumnHeader extends Header {
 			const entry = this._mouseOverEntry;
 			let modifier = 0;
 
-			if (this._startSelectionEntry && this._startSelectionEntry.index !== entry.index) {
+			if (this._startSelectionEntry && entry && this._startSelectionEntry.index !== entry.index) {
 				this._selectColumn(this._startSelectionEntry.index, modifier);
 				modifier += UNOModifier.SHIFT;
 				this._selectColumn(entry.index, modifier);

diff --git a/browser/src/control/Control.RowHeader.ts b/browser/src/control/Control.RowHeader.ts
@@ -242,7 +242,7 @@ export class RowHeader extends cool.Header {
 			const entry = this._mouseOverEntry;
 			let modifier = 0;
 
-			if (this._startSelectionEntry && this._startSelectionEntry.index !== entry.index) {
+			if (this._startSelectionEntry && entry && this._startSelectionEntry.index !== entry.index) {
 				this._selectRow(this._startSelectionEntry.index, modifier);
 				modifier += UNOModifier.SHIFT;
 				this._selectRow(entry.index, modifier);