Remove doc top offset workaround for checkCursorTag and checkCursorMenu which cause cursors floating around

1 files changed, 2 insertions, 6 deletions

diff --git a/public/js/index.js b/public/js/index.js
index dad51922..791f5333 100644
--- a/public/js/index.js
+++ b/public/js/index.js
@@ -2279,8 +2279,6 @@ function checkCursorTag(coord, ele) {
     // get coord position
     var left = coord.left;
     var top = coord.top;
-    // get doc top offset (to workaround with viewport)
-    var docTopOffset = ui.area.codemirrorSizerInner.position().top;
     // set offset
     var offsetLeft = -3;
     var offsetTop = defaultTextHeight;
@@ -2292,7 +2290,7 @@ function checkCursorTag(coord, ele) {
         }
         // flip y when element bottom bound larger than doc height
         // and element top position is larger than element height
-        if (top + docTopOffset + height + offsetTop + tagBottomMargin > Math.max(editor.doc.height, editorHeight) && top + docTopOffset > height + tagBottomMargin) {
+        if (top + height + offsetTop + tagBottomMargin > Math.max(editor.doc.height, editorHeight) && top > height + tagBottomMargin) {
             offsetTop = -(height);
         }
     }
@@ -2877,8 +2875,6 @@ function checkCursorMenuInner() {
     }, 'windows');
     var left = coord.left;
     var top = coord.top;
-    // get doc top offset (to workaround with viewport)
-    var docTopOffset = ui.area.codemirrorSizerInner.position().top;
     // set offset
     var offsetLeft = 0;
     var offsetTop = defaultTextHeight;
@@ -2889,7 +2885,7 @@ function checkCursorMenuInner() {
             offsetLeft = -(left + width - docWidth + menuRightMargin);
         // flip y when element bottom bound larger than doc height
         // and element top position is larger than element height
-        if (top + docTopOffset + height + offsetTop + menuBottomMargin > Math.max(editor.doc.height, editorHeight) && top + docTopOffset > height + menuBottomMargin) {
+        if (top + height + offsetTop + menuBottomMargin > Math.max(editor.doc.height, editorHeight) && top > height + menuBottomMargin) {
             offsetTop = -(height + menuBottomMargin);
             // reverse sort menu because upSideDown
             dropdown.html(reverseSortCursorMenu(dropdown));