Editor auto-position now respects iframe bounds (#111)

recogito · Sep 3, 2022 · 745f04b · 745f04b
1 parent b92bd8d
commit 745f04b
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/editor/setPosition.js b/src/editor/setPosition.js
@@ -16,17 +16,19 @@ const setPosition = (wrapperEl, editorEl, selectedEl, autoPosition) => {
   if (autoPosition) {
     const defaultOrientation = editorEl.children[1].getBoundingClientRect();
 
+    const { innerWidth, innerHeight } = wrapperEl.ownerDocument.defaultView;
+
     // Test 1: does right edge extend beyond the width of the page?
     // If so, flip horizontally
-    if (defaultOrientation.right > window.innerWidth) {
+    if (defaultOrientation.right > innerWidth) {
       editorEl.classList.remove('r6o-arrow-left');
       editorEl.classList.add('r6o-arrow-right');
       editorEl.style.left = `${right - defaultOrientation.width - containerBounds.left}px`;
     }
 
     // Test 2: does the bottom edge extend beyond the height of the page?
     // If so, flip vertically
-    if (defaultOrientation.bottom > window.innerHeight) {
+    if (defaultOrientation.bottom > innerHeight) {
       editorEl.classList.remove('r6o-arrow-top');
       editorEl.classList.add('r6o-arrow-bottom');