Skip to content
Snippets Groups Projects
Commit 894086f7 authored by David Bühler's avatar David Bühler
Browse files

[ivette] ASTinfo: fixes the dependency of a useEffect call.

parent eabbc0b7
No related branches found
No related tags found
No related merge requests found
...@@ -310,14 +310,14 @@ export default function ASTinfo(): JSX.Element { ...@@ -310,14 +310,14 @@ export default function ASTinfo(): JSX.Element {
React.useEffect(() => { React.useEffect(() => {
markers.setLocations(selectedLoc, hoveredLoc); markers.setLocations(selectedLoc, hoveredLoc);
}, [markers, selectedLoc, hoveredLoc]); }, [markers, selectedLoc, hoveredLoc]);
const pinMarker = (location: States.Location) : void => { const pinMarker = React.useCallback((location: States.Location) => {
if (location?.marker) if (location?.marker)
markers.setPinned(location?.marker, true); markers.setPinned(location?.marker, true);
}; }, [markers]);
React.useEffect(() => { React.useEffect(() => {
States.MetaSelection.on(pinMarker); States.MetaSelection.on(pinMarker);
return () => States.MetaSelection.off(pinMarker); return () => States.MetaSelection.off(pinMarker);
}); }, [pinMarker]);
// Rendering // Rendering
const renderMark = (mark: Mark): JSX.Element | null => { const renderMark = (mark: Mark): JSX.Element | null => {
const { marker } = mark; const { marker } = mark;
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment