fix: preserve Windows newline characters

Related to camunda/camunda-modeler#2280
bpmn-io · May 20, 2021 · ba088af · ba088af
1 parent 4c925d7
commit ba088af
Show file tree

Hide file tree

Showing 2 changed files with 56 additions and 0 deletions.
diff --git a/lib/TextBox.js b/lib/TextBox.js
@@ -1,6 +1,7 @@
 import {
   assign,
   bind,
+  isUndefined,
   pick
 } from 'min-dash';
 
@@ -212,6 +213,7 @@ TextBox.prototype.handlePaste = function(e) {
 };
 
 TextBox.prototype.insertText = function(text) {
+  text = normalizeEndOfLineSequences(text);
 
   // insertText command not supported by Internet Explorer
   var success = document.execCommand('insertText', false, text);
@@ -452,3 +454,13 @@ TextBox.prototype.setSelection = function(container, offset) {
   selection.removeAllRanges();
   selection.addRange(range);
 };
+
+// helpers //////////
+
+function normalizeEndOfLineSequences(string, endOfLineSequence) {
+  if (isUndefined(endOfLineSequence)) {
+    endOfLineSequence = '\n';
+  }
+
+  return string.replace(/\r\n|\r|\n/g, endOfLineSequence);
+}
diff --git a/test/DirectEditingSpec.js b/test/DirectEditingSpec.js
@@ -646,6 +646,50 @@ describe('diagram-js-direct-editing', function() {
         expect(directEditing._textbox.content.innerHTML).to.eql('FOO&lt;video src=1 onerror=alert(\'hueh\')&gt;');
       }));
 
+
+      it('should preserve line breaks (LF)', inject(function(canvas, directEditing) {
+
+        // given
+        var shape = {
+          id: 's1',
+          x: 20, y: 10, width: 60, height: 50,
+          label: 'FOO'
+        };
+        canvas.addShape(shape);
+
+        directEditing.activate(shape);
+
+        var textBox = directEditing._textbox;
+
+        // when
+        textBox.insertText('Foo\n\nBar');
+
+        // then
+        expect(directEditing._textbox.content.innerHTML).to.eql('FOOFoo<div><br></div><div>Bar</div>');
+      }));
+
+
+      it.only('should preserve line breaks (CRLF)', inject(function(canvas, directEditing) {
+
+        // given
+        var shape = {
+          id: 's1',
+          x: 20, y: 10, width: 60, height: 50,
+          label: 'FOO'
+        };
+        canvas.addShape(shape);
+
+        directEditing.activate(shape);
+
+        var textBox = directEditing._textbox;
+
+        // when
+        textBox.insertText('Foo\r\n\r\nBar');
+
+        // then
+        expect(directEditing._textbox.content.innerHTML).to.eql('FOOFoo<div><br></div><div>Bar</div>');
+      }));
+
     });
 
   });