summaryrefslogtreecommitdiff
path: root/opendc-web/opendc-web-ui/src/util/effect-ref.js
diff options
context:
space:
mode:
authorFabian Mastenbroek <mail.fabianm@gmail.com>2023-03-30 22:58:30 +0100
committerGitHub <noreply@github.com>2023-03-30 22:58:30 +0100
commit0db9d47d2b3062ca867e0a7aa33ba7205307d062 (patch)
tree3d6fc9128dd9ff82434c8ad112a01d023791cba5 /opendc-web/opendc-web-ui/src/util/effect-ref.js
parent526d6cd6b48b30cf7bbe40478d57bbc67e7027cc (diff)
parente7d5c086832a24f3c6b98258b0b8eb1fbbd3336a (diff)
merge: Address issues with web UI (#145)
This pull request addresses several issues that have been reported for the OpenDC web UI. ## Implementation Notes :hammer_and_pick: * Update dependencies for web UI * Inform user when deleted topology is still used * Do not offset hover layer after dragging * Fix access to machines on lower shelves * Do not allow selection of empty unit * Fix rack deletion Fixes #135, #136, #137, #138, #139
Diffstat (limited to 'opendc-web/opendc-web-ui/src/util/effect-ref.js')
-rw-r--r--opendc-web/opendc-web-ui/src/util/effect-ref.js6
1 files changed, 3 insertions, 3 deletions
diff --git a/opendc-web/opendc-web-ui/src/util/effect-ref.js b/opendc-web/opendc-web-ui/src/util/effect-ref.js
index cda0324b..78528585 100644
--- a/opendc-web/opendc-web-ui/src/util/effect-ref.js
+++ b/opendc-web/opendc-web-ui/src/util/effect-ref.js
@@ -26,9 +26,9 @@ const noop = () => {}
/**
* A hook that will invoke the specified callback when the reference returned by this function is initialized.
- * The callback can return an optional clean up function.
+ * The callback can return an optional clean-up function.
*/
-export function useEffectRef(callback) {
+export function useEffectRef(callback, deps = []) {
const disposeRef = useRef(noop)
return useCallback((element) => {
disposeRef.current()
@@ -37,5 +37,5 @@ export function useEffectRef(callback) {
if (element) {
disposeRef.current = callback(element) || noop
}
- }, []) // eslint-disable-line react-hooks/exhaustive-deps
+ }, deps) // eslint-disable-line react-hooks/exhaustive-deps
}