Tidy up

Author: hughns

crates/data-model/src/site_config.rs

@@ -90,7 +90,7 @@ pub struct SiteConfig {
 
     /// Whether users can log in with their email address.
     pub login_with_email_allowed: bool,
-    
+
     /// The iframe URL to show in the plan tab of the UI
     pub plan_management_iframe_uri: Option<Url>,
 }

frontend/src/routes/_account.plan.index.tsx

@@ -43,37 +43,37 @@ function Plan(): React.ReactElement {
   }
 
   const ref = useRef<HTMLIFrameElement>(null);
-  const [height, setHeight] = useState("0px");
+  const [iframeHeight, setIframeHeight] = useState("0px");
 
   // Poll the size of the iframe content and set the height
   // This will only work where the iframe is served from the same origin
-  const doHeight = useCallback(() => {
+  const calculateHeight = useCallback(() => {
     const height =
       ref.current?.contentWindow?.document.body.parentElement?.scrollHeight;
     if (height) {
-      setHeight(`${height}px`);
+      setIframeHeight(`${height}px`);
     } else {
-      setHeight("500px");
+      setIframeHeight("500px");
     }
   }, []);
   useEffect(() => {
-    doHeight();
+    calculateHeight();
 
     const interval = setInterval(() => {
-      doHeight();
+      calculateHeight();
     }, 1000);
 
     return () => clearInterval(interval);
-  }, [doHeight]);
+  }, [calculateHeight]);
 
   return (
     <iframe
-      title="asd"
+      title="iframe" // no proper title as this is experimental feature
       ref={ref}
-      onLoad={doHeight}
+      onLoad={calculateHeight}
       src={planManagementIframeUri}
       scrolling="no"
-      height={height}
+      height={iframeHeight}
     />
   );
 }