#6653 Inplace Editor - A line break character \n appears when removing the title or description (#6655)

Fixes #6653

Author: novikov82

packages/survey-creator-core/src/components/string-editor.ts

@@ -422,9 +422,8 @@ export class StringEditorViewModelBase extends Base {
   private getClearedText(target: HTMLElement): string {
     const html = target.innerHTML;
     const text = target.innerText;
-    if(!this.creator) return this.locString.hasHtml ? html : text;
     let mdText = null;
-    if (!this.editAsText) {
+    if (!this.editAsText && this.creator) {
       const options = {
         element: <Base><any>this.locString.owner,
         text: <any>null,
@@ -438,9 +437,11 @@ export class StringEditorViewModelBase extends Base {
     if (mdText) {
       clearedText = mdText;
     } else {
-      let txt = this.locString.hasHtml && !this.editAsText ? html : text;
-      if (!this.locString.allowLineBreaks) txt = clearNewLines(txt);
-      clearedText = txt;
+      clearedText = this.locString.hasHtml && !this.editAsText ? html : text;
+      const txt = clearNewLines(clearedText);
+      if (!this.locString.allowLineBreaks || !txt) {
+        clearedText = txt;
+      }
     }
     const owner = this.locString.owner as any;
 
@@ -451,7 +452,7 @@ export class StringEditorViewModelBase extends Base {
       newValue: clearedText,
       doValidation: false
     };
-    this.creator.onValueChangingCallback(changingOptions);
+    if (this.creator) this.creator.onValueChangingCallback(changingOptions);
     return changingOptions.newValue;
   }
   private getErrorTextOnChanged(clearedText: string): string {

packages/survey-creator-core/tests/string-editor.tests.ts

@@ -1051,6 +1051,40 @@ test("Test string editor inplaceEditForValues without Creator", (): any => {
   expect(q0.otherText).toEqual("Other changed");
 });
 
+test("Test string editor description clear (with EOL)", (): any => {
+  const survey = new SurveyModel({
+    "pages": [
+      {
+        "name": "page1",
+        "elements": [
+          {
+            "type": "radiogroup",
+            "title": "title",
+            "description": "desc",
+            "name": "q0",
+            "choices": ["i1", "i2"]
+          }
+        ]
+      }
+    ]
+  });
+
+  const q0 = survey.getQuestionByName("q0");
+  const itemValue = q0.choices[0];
+  var seChoice = new StringEditorViewModelBase(itemValue.locText);
+  var seDescription = new StringEditorViewModelBase(q0.locDescription);
+  expect(itemValue.text).toEqual("i1");
+  seChoice.onBlur({ target: { innerText: "new\nTitle", innerHTML: "new\nTitle", setAttribute: () => { }, removeAttribute: () => { } } });
+  expect(itemValue.text).toEqual("newTitle");
+
+  expect(q0.description).toEqual("desc");
+  seDescription.onBlur({ target: { innerText: "new\nDesc", innerHTML: "new\nDesc", setAttribute: () => { }, removeAttribute: () => { } } });
+  expect(q0.locDescription.text).toEqual("new\nDesc");
+
+  seDescription.onBlur({ target: { innerText: "\n", innerHTML: "\n", setAttribute: () => { }, removeAttribute: () => { } } });
+  expect(q0.locDescription.text).toEqual("");
+});
+
 test("StringEditor multiline paste for selectbase questions should respect creator.maximumChoicesCount", (): any => {
   const creator = new CreatorTester();
   creator.maximumChoicesCount = 4;