Skip to content

Commit 699b131

Browse files
author
Incharajayaram
committed
Fix: Resolve react-hooks/exhaustive-deps warnings
1 parent d2facc1 commit 699b131

File tree

1 file changed

+23
-23
lines changed

1 file changed

+23
-23
lines changed

src/pages/EditorComponent.js

Lines changed: 23 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -133,39 +133,39 @@ function EditorComponent() {
133133
DEFAULT_LANGUAGE: selectedLanguage.DEFAULT_LANGUAGE,
134134
NAME: selectedLanguage.NAME,
135135
});
136-
let savedCode = null;
137-
try {
138-
savedCode = localStorage.getItem(`code-${selectedLanguage.DEFAULT_LANGUAGE}`);
139-
} catch (e) {
140-
enqueueSnackbar("Failed to load saved code. Local storage might be unavailable.", { variant: "error" });
141-
console.error("Local storage load error:", e);
142-
}
136+
let savedCode = null;
137+
try {
138+
savedCode = localStorage.getItem(`code-${selectedLanguage.DEFAULT_LANGUAGE}`);
139+
} catch (e) {
140+
enqueueSnackbar("Failed to load saved code. Local storage might be unavailable.", { variant: "error" });
141+
console.error("Local storage load error:", e);
142+
}
143143

144-
if (savedCode !== null) {
145-
setCode(savedCode);
146-
} else {
147-
setCode(selectedLanguage.HELLO_WORLD);
148-
} }, [currentLanguage]);
144+
if (savedCode !== null) {
145+
setCode(savedCode);
146+
} else {
147+
setCode(selectedLanguage.HELLO_WORLD);
148+
} }, [currentLanguage, enqueueSnackbar]);
149149

150150
useEffect(() => {
151151
if (isImportingRef.current) return;
152152

153153
const handler = setTimeout(() => {
154-
try {
155-
if (code) {
156-
localStorage.setItem(`code-${currentLanguage}`, code);
157-
} else {
158-
localStorage.removeItem(`code-${currentLanguage}`);
159-
}
160-
} catch (e) {
161-
enqueueSnackbar("Failed to save code automatically. Local storage might be full or unavailable.", { variant: "error" });
162-
console.error("Local storage save error:", e);
163-
} }, 500); // 500ms debounce
154+
try {
155+
if (code) {
156+
localStorage.setItem(`code-${currentLanguage}`, code);
157+
} else {
158+
localStorage.removeItem(`code-${currentLanguage}`);
159+
}
160+
} catch (e) {
161+
enqueueSnackbar("Failed to save code automatically. Local storage might be full or unavailable.", { variant: "error" });
162+
console.error("Local storage save error:", e);
163+
} }, 500); // 500ms debounce
164164

165165
return () => {
166166
clearTimeout(handler);
167167
};
168-
}, [code]);
168+
}, [code, currentLanguage, enqueueSnackbar]);
169169

170170
const handleEditorThemeChange = async (_, theme) => {
171171
if (["light", "vs-dark"].includes(theme.ID)) {

0 commit comments

Comments
 (0)