style(js): Small code style cleanup in stacktracePreview (#36388)

Co-authored-by: Alberto Leal <[email protected]>

static/app/components/stacktracePreview.tsx

@@ -20,7 +20,7 @@ import findBestThread from './events/interfaces/threads/threadSelector/findBestT
 import getThreadStacktrace from './events/interfaces/threads/threadSelector/getThreadStacktrace';
 
 const REQUEST_DELAY = 100;
-const HOVERCARD_DELAY = 400;
+const HOVERCARD_CONTENT_DELAY = 400;
 
 export const STACKTRACE_PREVIEW_TOOLTIP_DELAY = 1000;
 
@@ -137,9 +137,10 @@ function StackTracePreview(props: StackTracePreviewProps): React.ReactElement {
       return;
     }
 
-    loaderTimeoutRef.current = window.setTimeout(() => {
-      setLoadingVisible(true);
-    }, HOVERCARD_DELAY);
+    loaderTimeoutRef.current = window.setTimeout(
+      () => setLoadingVisible(true),
+      HOVERCARD_CONTENT_DELAY
+    );
 
     try {
       const evt = await api.requestPromise(
@@ -176,26 +177,23 @@ function StackTracePreview(props: StackTracePreviewProps): React.ReactElement {
     delayTimeoutRef.current = undefined;
   }, []);
 
-  // Not sure why we need to stop propagation, maybe to to prevent the hovercard from closing?
-  // If we are doing this often, maybe it should be part of the hovercard component.
-  const handleStackTracePreviewClick = useCallback((e: React.MouseEvent) => {
-    e.stopPropagation();
-  }, []);
+  // Not sure why we need to stop propagation, maybe to prevent the
+  // hovercard from closing? If we are doing this often, maybe it should be
+  // part of the hovercard component.
+  const handleStackTracePreviewClick = useCallback(
+    (e: React.MouseEvent) => void e.stopPropagation(),
+    []
+  );
 
-  const stacktrace = useMemo(() => {
-    if (event) {
-      return getStacktrace(event);
-    }
-    return null;
-  }, [event]);
+  const stacktrace = useMemo(() => (event ? getStacktrace(event) : null), [event]);
 
   return (
     <span
       className={props.className}
       onMouseEnter={handleMouseEnter}
       onMouseLeave={handleMouseLeave}
     >
-      <StyledHovercard
+      <StacktraceHovercard
         body={
           status === 'loading' && !loadingVisible ? null : status === 'loading' ? (
             <NoStackTraceWrapper onClick={handleStackTracePreviewClick}>
@@ -233,14 +231,14 @@ function StackTracePreview(props: StackTracePreviewProps): React.ReactElement {
         tipColor="background"
       >
         {props.children}
-      </StyledHovercard>
+      </StacktraceHovercard>
     </span>
   );
 }
 
 export {StackTracePreview};
 
-const StyledHovercard = styled(Hovercard)<{state: 'loading' | 'empty' | 'done'}>`
+const StacktraceHovercard = styled(Hovercard)<{state: 'loading' | 'empty' | 'done'}>`
   /* Lower z-index to match the modals (10000 vs 10002) to allow stackTraceLinkModal be on top of stack trace preview. */
   z-index: ${p => p.theme.zIndex.modal};
   width: ${p => {