fix: script error occurs in the wysiwyg table (fix #1110) (#1111)

* fix: element is empty when selection is finished * fix: column is not added when cursor is at specific position
nhn · Jul 20, 2020 · 6be680c · 6be680c
1 parent b6b3215
commit 6be680c
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/apps/editor/src/js/wwTableSelectionManager.js b/apps/editor/src/js/wwTableSelectionManager.js
@@ -89,8 +89,11 @@ class WwTableSelectionManager {
             window.getSelection().removeAllRanges();
           }, 10);
         }
-        this.highlightTableCellsBy(selectionStart, selectionEnd);
-        validSelectionEnd = selectionEnd;
+
+        if (selectionStart && selectionEnd) {
+          this.highlightTableCellsBy(selectionStart, selectionEnd);
+          validSelectionEnd = selectionEnd;
+        }
       }
     };
 

diff --git a/apps/editor/src/js/wysiwygCommands/tableAddCol.js b/apps/editor/src/js/wysiwygCommands/tableAddCol.js
@@ -74,7 +74,7 @@ function getCellByRange(range) {
   let cell = range.startContainer;
 
   if (domUtils.getNodeName(cell) !== 'TD' && domUtils.getNodeName(cell) !== 'TH') {
-    cell = domUtils.parentsUntil(cell, 'tr');
+    cell = domUtils.closest(cell, 'td, th');
   }
 
   return cell;