tweaks

components/Notes.tsx

@@ -32,7 +32,7 @@ export default function Notes({ gameData: { dmID, cards, settings }, show }: Not
 						cards[Object.keys(cardMap).find((key) => cardMap[key] === index) || 0],
 				)
 				.filter((truthy) => truthy),
-		[settings],
+		[cards, isDM, settings],
 	);
 
 	const showNotes = show && open && (isDM || settings.notes);
@@ -49,7 +49,7 @@ export default function Notes({ gameData: { dmID, cards, settings }, show }: Not
 			</button>
 
 			<Scrim
-				onClick={() => setOpen((prev) => !prev)}
+				clickAction={() => setOpen((prev) => !prev)}
 				className={`transition-all duration-250 ${showNotes ? 'pointer-events-auto opacity-100' : 'pointer-events-none opacity-0'}`}
 			>
 				<div

components/Scrim.tsx

@@ -1,16 +1,16 @@
 'use client';
 
 type ScrimProps = {
-	children?: React.ReactNode;
-	onClick?: (event: React.MouseEvent<HTMLDivElement>) => void;
+	children: React.ReactNode;
+	clickAction: (event: React.MouseEvent<HTMLDivElement>) => void;
 	show?: boolean;
 	className?: string;
 };
 
-export default function Scrim({ children, onClick, show = true, className = '' }: ScrimProps) {
+export default function Scrim({ children, clickAction, show = true, className = '' }: ScrimProps) {
 	const handleClick = (event: React.MouseEvent<HTMLDivElement>) => {
 		if (event.target === event.currentTarget) {
-			onClick && onClick(event);
+			clickAction(event);
 		}
 	};
 	if (!show) return null;

components/Settings.tsx

@@ -105,7 +105,7 @@ export default function Settings({ gameData, changeAction }: SettingsProps) {
 	return (
 		<div className={`fixed top-4 right-4 z-25 ${cinzel.className}`}>
 			<Scrim
-				onClick={() => setOpen((prev) => !prev)}
+				clickAction={() => setOpen((prev) => !prev)}
 				className={`transition-all duration-250 ${open ? 'pointer-events-auto opacity-100' : 'pointer-events-none opacity-0'}`}
 			>
 				<div