fix: duplicate computations and renderings

This commit is contained in:
etiennecollin
2025-08-13 23:08:41 -04:00
parent 1bfae81adb
commit 7c17a790d4
2 changed files with 6 additions and 4 deletions

View File

@@ -20,7 +20,7 @@ export default function Header() {
})(); })();
setWifiConfig(config); setWifiConfig(config);
}); }, [generateWifiConfig]);
return ( return (
<header className="bg-surface border-b border-default sticky top-0 z-7000"> <header className="bg-surface border-b border-default sticky top-0 z-7000">

View File

@@ -1,6 +1,6 @@
"use client"; "use client";
import { useRef, useState, useEffect } from "react"; import { useRef, useState, useEffect, useMemo } from "react";
import Modal from "@/components/modals/Modal"; import Modal from "@/components/modals/Modal";
import { QRCodeSVG } from "qrcode.react"; import { QRCodeSVG } from "qrcode.react";
import { generateWiFiQRString, WifiConfig } from "@/utils/wifi"; import { generateWiFiQRString, WifiConfig } from "@/utils/wifi";
@@ -13,8 +13,10 @@ type Props = {
export default function WifiQrModal({ wifiConfig, onClose }: Props) { export default function WifiQrModal({ wifiConfig, onClose }: Props) {
const modalRef = useRef<HTMLDivElement | null>(null); const modalRef = useRef<HTMLDivElement | null>(null);
const [qrSize, setQrSize] = useState(220); const [qrSize, setQrSize] = useState(220);
const wifiString = wifiConfig && generateWiFiQRString(wifiConfig); const wifiString = useMemo(
console.log(wifiString); () => wifiConfig && generateWiFiQRString(wifiConfig),
[wifiConfig],
);
useEffect(() => { useEffect(() => {
function updateSize() { function updateSize() {