From 809d7ba9f5a3bc6a2f0d2a84bbb9fc58f18d60d7 Mon Sep 17 00:00:00 2001 From: Christopher Chedeau Date: Sat, 14 Mar 2020 14:00:20 -0700 Subject: [PATCH] Remove text trim (#947) This was added when we were computing the width without adding "pre" behavior. Now it is no longer an issue --- src/element/textWysiwyg.tsx | 11 +---------- 1 file changed, 1 insertion(+), 10 deletions(-) diff --git a/src/element/textWysiwyg.tsx b/src/element/textWysiwyg.tsx index fe44a16e54..f4c23a6325 100644 --- a/src/element/textWysiwyg.tsx +++ b/src/element/textWysiwyg.tsx @@ -13,12 +13,6 @@ type TextWysiwygParams = { onCancel: () => void; }; -// When WYSIWYG text ends with white spaces, the text gets vertically misaligned -// in order to fix this issue, we remove those white spaces -function trimText(text: string) { - return text.trim(); -} - export function textWysiwyg({ initText, x, @@ -30,9 +24,6 @@ export function textWysiwyg({ onSubmit, onCancel, }: TextWysiwygParams) { - // Using contenteditable here as it has dynamic width. - // But this solution has an issue — it allows to paste - // multiline text, which is not currently supported const editable = document.createElement("div"); try { editable.contentEditable = "plaintext-only"; @@ -110,7 +101,7 @@ export function textWysiwyg({ function handleSubmit() { if (editable.innerText) { - onSubmit(trimText(editable.innerText)); + onSubmit(editable.innerText); } else { onCancel(); }