From 038714e7155c74fe682c81ea06662b164ca72c7b Mon Sep 17 00:00:00 2001 From: Aakansha Doshi Date: Wed, 5 Apr 2023 20:41:41 +0530 Subject: [PATCH] Revert "floor line height and height when dom height increases than canvas height" This reverts commit 8de65168238b8fb9a638e0c75f596f371983c2c7. --- src/element/textWysiwyg.tsx | 17 ++++------------- 1 file changed, 4 insertions(+), 13 deletions(-) diff --git a/src/element/textWysiwyg.tsx b/src/element/textWysiwyg.tsx index 05c4ed7f51..109b557d67 100644 --- a/src/element/textWysiwyg.tsx +++ b/src/element/textWysiwyg.tsx @@ -36,7 +36,6 @@ import { getMaxContainerWidth, computeContainerDimensionForBoundText, measureDOMHeight, - getLineHeightInPx, splitIntoLines, } from "./textElement"; import { @@ -279,19 +278,11 @@ export const textWysiwyg = ({ getFontString(updatedTextElement), updatedTextElement.lineHeight, ); + let lineHeight = element.lineHeight; - if (isSafari) { - const lineCount = splitIntoLines(element.text).length; - if (domHeight > textElementHeight) { - lineHeight = (Math.floor(element.lineHeight * element.fontSize) / - element.fontSize) as ExcalidrawTextElement["lineHeight"]; - textElementHeight = - getLineHeightInPx(element.fontSize, lineHeight) * lineCount; - mutateElement(updatedTextElement, { - lineHeight, - height: textElementHeight, - }); - } + if (isSafari && domHeight > textElementHeight) { + lineHeight = (Math.floor(element.lineHeight * element.fontSize) / + element.fontSize) as ExcalidrawTextElement["lineHeight"]; } // Make sure text editor height doesn't go beyond viewport