Skip to content

Commit 48e6c69

Browse files
chore: add TODOs for overlay and toolbar position logic to consolidate with existing methods
1 parent 74e6e5b commit 48e6c69

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

src/visualBuilder/utils/updateFocussedState.ts

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -300,6 +300,9 @@ export function updateFocussedStateOnMutation(
300300
}
301301
}
302302

303+
//TODO: This logic for overlay position is already present in generateOverlay as `addFocusOverlay`.
304+
// We should refactor this to use the same logic. Refer "VB-593" branch for more details.
305+
303306
/**
304307
* Update the focus overlays if they exists.
305308
*/
@@ -386,6 +389,8 @@ export function updateFocussedStateOnMutation(
386389
* Update the focus toolbar if it exists.
387390
*/
388391

392+
//TODO: This logic for toolbar position is already present in same file as `positionToolbar`.
393+
// We should refactor this to use the same logic. Refer "VB-593" branch for more details.
389394
if (focusedToolbar) {
390395
const targetElementRightEdgeOffset =
391396
window.scrollX + window.innerWidth - selectedElementDimension.left;

0 commit comments

Comments
 (0)