update comment and change offsetWidth to measure function

Expensify · May 3, 2022 · e098176 · e098176
1 parent 6ade224
commit e098176
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/src/components/Tooltip/TooltipRenderedOnPageBody.js b/src/components/Tooltip/TooltipRenderedOnPageBody.js
@@ -64,8 +64,10 @@ class TooltipRenderedOnPageBody extends React.PureComponent {
     }
 
     componentDidMount() {
-        this.setState({
-            tooltipTextWidth: this.textRef.offsetWidth,
+        this.textRef.measure((x, y, width) => {
+            this.setState({
+                tooltipTextWidth: width,
+            });
         });
     }
 

diff --git a/src/styles/getTooltipStyles.js b/src/styles/getTooltipStyles.js
@@ -97,8 +97,7 @@ export default function getTooltipStyles(
     const tooltipVerticalPadding = spacing.pv1;
     const tooltipFontSize = variables.fontSizeSmall;
 
-    // We get wrapper width based on tooltip's inner text width so the wrapper
-    // is just big enough to fit text and prevent white space
+    // We get wrapper width based on tooltip's inner text width so the wrapper is just big enough to fit text and prevent white space
     const wrapperWidth = tooltipTextWidth && tooltipTextWidth < maxWidth
 
         // Add horizontal padding to the text width to get the wrapper width.