annotation: avoid marking comment autosave if its not saved

problem: After entering the comment modify mode and clicking somewhere else, to lose comment focus, comment cannot be closed by clicking cancel. Signed-off-by: Pranam Lashkari <[email protected]> Change-Id: I05ce2b4f6dcba47d40095bb16fdb56ce5b9c20cf
CollaboraOnline · Oct 27, 2023 · 2c720a6 · 2c720a6
1 parent c1db537
commit 2c720a6
Showing 1 changed file with 6 additions and 13 deletions.
diff --git a/browser/src/layer/tile/CommentSection.ts b/browser/src/layer/tile/CommentSection.ts
@@ -844,22 +844,15 @@ export class Comment extends CanvasSectionObject {
 	// eslint-disable-next-line @typescript-eslint/explicit-module-boundary-types
 	public onLostFocus (e: any): void {
 		if (!this.sectionProperties.isRemoved) {
-			if (!this.sectionProperties.nodeModifyText.value &&
-				!this.sectionProperties.contentText.origText)
-				return;
-			app.view.commentAutoSave = this;
 			if (this.sectionProperties.contentText.origText !== this.sectionProperties.nodeModifyText.value) {
+				app.view.commentAutoSave = this;
 				this.onSaveComment(e);
 			}
-			else {
-				if (!this.containerObject.testing) // eslint-disable-line no-lonely-if
-					this.onCancelClick(e);
-				else {
-					var insertButton = document.getElementById('menu-insertcomment');
-					if (insertButton) {
-						if (window.getComputedStyle(insertButton).display === 'none') {
-							this.onCancelClick(e);
-						}
+			else if (this.containerObject.testing) {
+				var insertButton = document.getElementById('menu-insertcomment');
+				if (insertButton) {
+					if (window.getComputedStyle(insertButton).display === 'none') {
+						this.onCancelClick(e);
 					}
 				}
 			}