@atlaskit/editor-core 153.1.4 → 156.0.0

package/dist/cjs/plugins/scroll-into-view/index.js

@@ -5,6 +5,8 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.scrollIntoViewPluginKey = exports.default = void 0;
 
+var _safePlugin = require("@atlaskit/editor-common/safe-plugin");
+
 var _prosemirrorState = require("prosemirror-state");
 
 var _prosemirrorInputRules = require("@atlaskit/prosemirror-input-rules");
@@ -22,7 +24,7 @@ var scrollIntoViewPluginKey = new _prosemirrorState.PluginKey('scrollIntoViewPlu
 exports.scrollIntoViewPluginKey = scrollIntoViewPluginKey;
 
 var createPlugin = function createPlugin() {
-  return new _prosemirrorState.Plugin({
+  return new _safePlugin.SafePlugin({
     key: scrollIntoViewPluginKey,
     appendTransaction: function appendTransaction(transactions, oldState, newState) {
       if (!transactions.length) {

package/dist/cjs/plugins/selection/gap-cursor/actions.js

@@ -13,13 +13,13 @@ var _prosemirrorState = require("prosemirror-state");
 
 var _prosemirrorUtils = require("prosemirror-utils");
 
-var _editorCommon = require("@atlaskit/editor-common");
+var _utils = require("@atlaskit/editor-common/utils");
 
 var _direction = require("./direction");
 
 var _selection = require("./selection");
 
-var _utils = require("./utils");
+var _utils2 = require("./utils");
 
 var _isValidTargetNode = require("./utils/is-valid-target-node");
 
@@ -40,7 +40,7 @@ function shouldHandleMediaGapCursor(dir, state) {
   if (selection instanceof _prosemirrorState.TextSelection) {
     // Should not use gap cursor if I am moving from a text selection into a media node
     if (dir === _direction.Direction.UP && !(0, _position.atTheBeginningOfDoc)(state) || dir === _direction.Direction.DOWN && !(0, _position.atTheEndOfDoc)(state)) {
-      var media = (0, _utils.getMediaNearPos)(doc, $pos, schema, mapDirection[dir]);
+      var media = (0, _utils2.getMediaNearPos)(doc, $pos, schema, mapDirection[dir]);
 
       if (media) {
         return false;
@@ -49,7 +49,7 @@ function shouldHandleMediaGapCursor(dir, state) {
 
 
     if (dir === _direction.Direction.LEFT || dir === _direction.Direction.RIGHT) {
-      var _media = (0, _utils.getMediaNearPos)(doc, $pos, schema, mapDirection[dir]);
+      var _media = (0, _utils2.getMediaNearPos)(doc, $pos, schema, mapDirection[dir]);
 
       var mediaSingle = schema.nodes.mediaSingle;
 
@@ -90,7 +90,7 @@ var arrow = function arrow(dir, endOfTextblock) {
       } // UP/DOWN jumps to the nearest texblock skipping gapcursor whenever possible
 
 
-      if (dir === _direction.Direction.UP && !(0, _position.atTheBeginningOfDoc)(state) && (0, _utils.isTextBlockNearPos)(doc, schema, $pos, -1) || dir === _direction.Direction.DOWN && ((0, _position.atTheEndOfDoc)(state) || (0, _utils.isTextBlockNearPos)(doc, schema, $pos, 1))) {
+      if (dir === _direction.Direction.UP && !(0, _position.atTheBeginningOfDoc)(state) && (0, _utils2.isTextBlockNearPos)(doc, schema, $pos, -1) || dir === _direction.Direction.DOWN && ((0, _position.atTheEndOfDoc)(state) || (0, _utils2.isTextBlockNearPos)(doc, schema, $pos, 1))) {
         return false;
       } // otherwise resolve previous/next position
 
@@ -130,7 +130,7 @@ var arrow = function arrow(dir, endOfTextblock) {
       var domAtPos = view.domAtPos.bind(view);
       var target = (0, _prosemirrorUtils.findDomRefAtPos)($pos.pos, domAtPos);
 
-      if (target && target.textContent === _editorCommon.ZERO_WIDTH_SPACE) {
+      if (target && target.textContent === _utils.ZERO_WIDTH_SPACE) {
         return false;
       }
     }

package/dist/cjs/plugins/selection/pm-plugins/gap-cursor-main.js

@@ -5,6 +5,8 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.default = void 0;
 
+var _safePlugin = require("@atlaskit/editor-common/safe-plugin");
+
 var _prosemirrorState = require("prosemirror-state");
 
 var _prosemirrorView = require("prosemirror-view");
@@ -27,7 +29,7 @@ var _direction = require("../gap-cursor/direction");
 
 var _gapCursorPluginKey = require("./gap-cursor-plugin-key");
 
-var plugin = new _prosemirrorState.Plugin({
+var plugin = new _safePlugin.SafePlugin({
   key: _gapCursorPluginKey.gapCursorPluginKey,
   state: {
     init: function init() {

package/dist/cjs/plugins/selection/pm-plugins/selection-main.js

@@ -5,21 +5,23 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.getInitialState = exports.createPlugin = void 0;
 
+var _safePlugin = require("@atlaskit/editor-common/safe-plugin");
+
 var _prosemirrorState = require("prosemirror-state");
 
-var _editorCommon = require("@atlaskit/editor-common");
+var _utils = require("@atlaskit/editor-common/utils");
 
 var _pluginFactory = require("../plugin-factory");
 
 var _types = require("../types");
 
-var _utils = require("../utils");
+var _utils2 = require("../utils");
 
 var _actions = require("../actions");
 
 var getInitialState = function getInitialState(state) {
   return {
-    decorationSet: (0, _utils.getDecorations)(state.tr),
+    decorationSet: (0, _utils2.getDecorations)(state.tr),
     selection: state.selection
   };
 };
@@ -50,7 +52,7 @@ var toggleContentEditable = function toggleContentEditable(node) {
 
 var createPlugin = function createPlugin(dispatch, dispatchAnalyticsEvent) {
   var options = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : {};
-  return new _prosemirrorState.Plugin({
+  return new _safePlugin.SafePlugin({
     key: _types.selectionPluginKey,
     state: (0, _pluginFactory.createPluginState)(dispatch, getInitialState),
     view: function view() {
@@ -58,16 +60,16 @@ var createPlugin = function createPlugin(dispatch, dispatchAnalyticsEvent) {
         update: function update(editorView, oldEditorState) {
           var state = editorView.state;
 
-          if (!(0, _utils.shouldRecalcDecorations)({
+          if (!(0, _utils2.shouldRecalcDecorations)({
             oldEditorState: oldEditorState,
             newEditorState: state
           })) {
             return;
           }
 
-          var analyticsPayload = (0, _utils.getNodeSelectionAnalyticsPayload)(state.selection) || (0, _utils.getAllSelectionAnalyticsPayload)(state.selection) || // We handle all range/cell selections except click and drag here, which is
+          var analyticsPayload = (0, _utils2.getNodeSelectionAnalyticsPayload)(state.selection) || (0, _utils2.getAllSelectionAnalyticsPayload)(state.selection) || // We handle all range/cell selections except click and drag here, which is
           // handled in mouseup handler below
-          !editorView.mouseDown && ((0, _utils.getRangeSelectionAnalyticsPayload)(state.selection, state.doc) || (0, _utils.getCellSelectionAnalyticsPayload)(state)); // We have to use dispatchAnalyticsEvent over any of the analytics plugin helpers
+          !editorView.mouseDown && ((0, _utils2.getRangeSelectionAnalyticsPayload)(state.selection, state.doc) || (0, _utils2.getCellSelectionAnalyticsPayload)(state)); // We have to use dispatchAnalyticsEvent over any of the analytics plugin helpers
           // as there were several issues caused by the fact that adding analytics through
           // the plugin adds a new step to the transaction
           // This causes prosemirror to run through some different code paths, eg. attempting
@@ -80,7 +82,7 @@ var createPlugin = function createPlugin(dispatch, dispatchAnalyticsEvent) {
       };
     },
     appendTransaction: function appendTransaction(_transactions, oldEditorState, newEditorState) {
-      if (!(0, _utils.shouldRecalcDecorations)({
+      if (!(0, _utils2.shouldRecalcDecorations)({
         oldEditorState: oldEditorState,
         newEditorState: newEditorState
       })) {
@@ -91,7 +93,7 @@ var createPlugin = function createPlugin(dispatch, dispatchAnalyticsEvent) {
       tr.setMeta(_types.selectionPluginKey, {
         type: _actions.SelectionActionTypes.SET_DECORATIONS,
         selection: tr.selection,
-        decorationSet: (0, _utils.getDecorations)(tr)
+        decorationSet: (0, _utils2.getDecorations)(tr)
       });
       return tr;
     },
@@ -122,7 +124,7 @@ var createPlugin = function createPlugin(dispatch, dispatchAnalyticsEvent) {
           var mouseEvent = event;
 
           if (!mouseEvent.shiftKey) {
-            var analyticsPayload = (0, _utils.getRangeSelectionAnalyticsPayload)(editorView.state.selection, editorView.state.doc) || (0, _utils.getCellSelectionAnalyticsPayload)(editorView.state);
+            var analyticsPayload = (0, _utils2.getRangeSelectionAnalyticsPayload)(editorView.state.selection, editorView.state.doc) || (0, _utils2.getCellSelectionAnalyticsPayload)(editorView.state);
 
             if (analyticsPayload) {
               dispatchAnalyticsEvent(analyticsPayload);
@@ -137,7 +139,7 @@ var createPlugin = function createPlugin(dispatch, dispatchAnalyticsEvent) {
           // On keypress, if the head of cursor selection touches a node with contenteditable="false",
           // we temporarily remove the attribute, wait one tick, then restore it with its original value.
 
-          if (_editorCommon.browser.gecko) {
+          if (_utils.browser.gecko) {
             var node = editorView.nodeDOM(editorView.state.selection.head);
 
             if (node instanceof HTMLElement && node.getAttribute('contenteditable') === 'false') {

package/dist/cjs/plugins/status/nodeviews/status.js

@@ -2,8 +2,6 @@
 
 var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault");
 
-var _typeof = require("@babel/runtime/helpers/typeof");
-
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
@@ -34,22 +32,16 @@ var _reactIntlNext = require("react-intl-next");
 
 var _styledComponents = _interopRequireDefault(require("styled-components"));
 
-var _editorCommon = require("@atlaskit/editor-common");
+var _utils = require("@atlaskit/editor-common/utils");
 
 var _element = require("@atlaskit/status/element");
 
 var _nodeviews = require("../../../nodeviews");
 
-var _InlineNodeWrapper = _interopRequireWildcard(require("../../../ui/InlineNodeWrapper"));
-
 var _messages = require("./messages");
 
 var _templateObject;
 
-function _getRequireWildcardCache(nodeInterop) { if (typeof WeakMap !== "function") return null; var cacheBabelInterop = new WeakMap(); var cacheNodeInterop = new WeakMap(); return (_getRequireWildcardCache = function _getRequireWildcardCache(nodeInterop) { return nodeInterop ? cacheNodeInterop : cacheBabelInterop; })(nodeInterop); }
-
-function _interopRequireWildcard(obj, nodeInterop) { if (!nodeInterop && obj && obj.__esModule) { return obj; } if (obj === null || _typeof(obj) !== "object" && typeof obj !== "function") { return { default: obj }; } var cache = _getRequireWildcardCache(nodeInterop); if (cache && cache.has(obj)) { return cache.get(obj); } var newObj = {}; var hasPropertyDescriptor = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var key in obj) { if (key !== "default" && Object.prototype.hasOwnProperty.call(obj, key)) { var desc = hasPropertyDescriptor ? Object.getOwnPropertyDescriptor(obj, key) : null; if (desc && (desc.get || desc.set)) { Object.defineProperty(newObj, key, desc); } else { newObj[key] = obj[key]; } } } newObj.default = obj; if (cache) { cache.set(obj, newObj); } return newObj; }
-
 function _createSuper(Derived) { var hasNativeReflectConstruct = _isNativeReflectConstruct(); return function _createSuperInternal() { var Super = (0, _getPrototypeOf2.default)(Derived), result; if (hasNativeReflectConstruct) { var NewTarget = (0, _getPrototypeOf2.default)(this).constructor; result = Reflect.construct(Super, arguments, NewTarget); } else { result = Super.apply(this, arguments); } return (0, _possibleConstructorReturn2.default)(this, result); }; }
 
 function _isNativeReflectConstruct() { if (typeof Reflect === "undefined" || !Reflect.construct) return false; if (Reflect.construct.sham) return false; if (typeof Proxy === "function") return true; try { Boolean.prototype.valueOf.call(Reflect.construct(Boolean, [], function () {})); return true; } catch (e) { return false; } }
@@ -120,10 +112,6 @@ var StatusNodeView = /*#__PURE__*/function (_ReactNodeView) {
   (0, _createClass2.default)(StatusNodeView, [{
     key: "createDomRef",
     value: function createDomRef() {
-      if (this.reactComponentProps.options && this.reactComponentProps.options.useInlineWrapper) {
-        return (0, _InlineNodeWrapper.createMobileInlineDomRef)();
-      }
-
       return (0, _get2.default)((0, _getPrototypeOf2.default)(StatusNodeView.prototype), "createDomRef", this).call(this);
     }
   }, {
@@ -146,15 +134,13 @@ var StatusNodeView = /*#__PURE__*/function (_ReactNodeView) {
           color = _this$node$attrs.color,
           localId = _this$node$attrs.localId,
           style = _this$node$attrs.style;
-      return /*#__PURE__*/_react.default.createElement(_InlineNodeWrapper.default, {
-        useInlineWrapper: options && options.useInlineWrapper
-      }, /*#__PURE__*/_react.default.createElement(IntlStatusContainerView, {
+      return /*#__PURE__*/_react.default.createElement(_react.default.Fragment, null, options && options.allowZeroWidthSpaceAfter && _utils.ZERO_WIDTH_SPACE, /*#__PURE__*/_react.default.createElement(IntlStatusContainerView, {
         view: this.view,
         text: text,
         color: color,
         style: style,
         localId: localId
-      }), options && options.allowZeroWidthSpaceAfter && _editorCommon.ZERO_WIDTH_SPACE);
+      }), options && options.allowZeroWidthSpaceAfter && _utils.ZERO_WIDTH_SPACE);
     }
   }]);
   return StatusNodeView;

package/dist/cjs/plugins/status/plugin.js

@@ -21,11 +21,9 @@ Object.defineProperty(exports, "pluginKeyName", {
 
 var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/defineProperty"));
 
-var _prosemirrorState = require("prosemirror-state");
-
-var _prosemirrorView = require("prosemirror-view");
+var _safePlugin = require("@atlaskit/editor-common/safe-plugin");
 
-var _editorCommon = require("@atlaskit/editor-common");
+var _prosemirrorState = require("prosemirror-state");
 
 var _status = _interopRequireDefault(require("./nodeviews/status"));
 
@@ -38,7 +36,7 @@ function ownKeys(object, enumerableOnly) { var keys = Object.keys(object); if (O
 function _objectSpread(target) { for (var i = 1; i < arguments.length; i++) { var source = null != arguments[i] ? arguments[i] : {}; i % 2 ? ownKeys(Object(source), !0).forEach(function (key) { (0, _defineProperty2.default)(target, key, source[key]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(target, Object.getOwnPropertyDescriptors(source)) : ownKeys(Object(source)).forEach(function (key) { Object.defineProperty(target, key, Object.getOwnPropertyDescriptor(source, key)); }); } return target; }
 
 var createPlugin = function createPlugin(dispatch, portalProviderAPI, eventDispatcher, options) {
-  return new _prosemirrorState.Plugin({
+  return new _safePlugin.SafePlugin({
     state: {
       init: function init() {
         return {
@@ -139,26 +137,6 @@ var createPlugin = function createPlugin(dispatch, portalProviderAPI, eventDispa
     props: {
       nodeViews: {
         status: (0, _status.default)(portalProviderAPI, eventDispatcher, options)
-      },
-      decorations: function decorations(state) {
-        var tr = state.tr;
-        var nodeAtSelection = tr.doc.nodeAt(tr.selection.from);
-
-        if (options && options.allowZeroWidthSpaceAfter && nodeAtSelection && nodeAtSelection.type === state.schema.nodes.status) {
-          var delayedNodeRendering = function delayedNodeRendering() {
-            return document.createTextNode(_editorCommon.ZERO_WIDTH_SPACE);
-          };
-
-          var decoration = _prosemirrorView.Decoration.widget(tr.selection.from, delayedNodeRendering, {
-            side: 1,
-            key: '#status-zero-width-char-decoration'
-          });
-
-          var doc = state.doc;
-          return _prosemirrorView.DecorationSet.create(doc, [decoration]);
-        }
-
-        return null;
       }
     }
   });

package/dist/cjs/plugins/status/styles.js

@@ -11,11 +11,11 @@ var _taggedTemplateLiteral2 = _interopRequireDefault(require("@babel/runtime/hel
 
 var _styledComponents = require("styled-components");
 
-var _editorCommon = require("@atlaskit/editor-common");
+var _styles = require("@atlaskit/editor-common/styles");
 
 var _editorSharedStyles = require("@atlaskit/editor-shared-styles");
 
 var _templateObject;
 
-var statusStyles = (0, _styledComponents.css)(_templateObject || (_templateObject = (0, _taggedTemplateLiteral2.default)(["\n  .", " {\n    // these two styles can be removed when displayInlineBlockForInlineNodes is enabled by default\n    // as it will make all inline nodes inline-block by default\n    display: inline-block;\n    user-select: all;\n\n    > span {\n      display: inline-block;\n      cursor: pointer;\n      line-height: 0; /* Prevent responsive layouts increasing height of container. */\n    }\n\n    &.", "\n      .", "\n      > span {\n      ", "\n    }\n  }\n\n  .danger {\n    .", " > span {\n      background-color: ", ";\n    }\n\n    .", ".", "\n      .", "\n      > span {\n      box-shadow: 0 0 0 ", "px ", ";\n    }\n  }\n"])), _editorCommon.StatusSharedCssClassName.STATUS_CONTAINER, _editorSharedStyles.akEditorSelectedNodeClassName, _editorCommon.StatusSharedCssClassName.STATUS_LOZENGE, (0, _editorSharedStyles.getSelectionStyles)([_editorSharedStyles.SelectionStyle.BoxShadow]), _editorCommon.StatusSharedCssClassName.STATUS_LOZENGE, _editorSharedStyles.akEditorDeleteBackgroundWithOpacity, _editorCommon.StatusSharedCssClassName.STATUS_CONTAINER, _editorSharedStyles.akEditorSelectedNodeClassName, _editorCommon.StatusSharedCssClassName.STATUS_LOZENGE, _editorSharedStyles.akEditorSelectedBorderSize, _editorSharedStyles.akEditorDeleteBorder);
+var statusStyles = (0, _styledComponents.css)(_templateObject || (_templateObject = (0, _taggedTemplateLiteral2.default)(["\n  .", ",\n    .", ",\n    [data-layout-section] {\n    .", " {\n      max-width: 100%;\n      line-height: 0;\n\n      > span {\n        width: 100%;\n      }\n    }\n  }\n  .", " {\n    // these two styles can be removed when displayInlineBlockForInlineNodes is enabled by default\n    // as it will make all inline nodes inline-block by default\n    display: inline-block;\n    user-select: all;\n\n    > span {\n      display: inline-block;\n      cursor: pointer;\n      line-height: 0; /* Prevent responsive layouts increasing height of container. */\n    }\n\n    &.", "\n      .", "\n      > span {\n      ", "\n    }\n  }\n\n  .danger {\n    .", " > span {\n      background-color: ", ";\n    }\n\n    .", ".", "\n      .", "\n      > span {\n      box-shadow: 0 0 0 ", "px ", ";\n    }\n  }\n"])), _styles.TableSharedCssClassName.TABLE_CELL_WRAPPER, _styles.TableSharedCssClassName.TABLE_HEADER_CELL_WRAPPER, _styles.StatusSharedCssClassName.STATUS_CONTAINER, _styles.StatusSharedCssClassName.STATUS_CONTAINER, _editorSharedStyles.akEditorSelectedNodeClassName, _styles.StatusSharedCssClassName.STATUS_LOZENGE, (0, _editorSharedStyles.getSelectionStyles)([_editorSharedStyles.SelectionStyle.BoxShadow]), _styles.StatusSharedCssClassName.STATUS_LOZENGE, _editorSharedStyles.akEditorDeleteBackgroundWithOpacity, _styles.StatusSharedCssClassName.STATUS_CONTAINER, _editorSharedStyles.akEditorSelectedNodeClassName, _styles.StatusSharedCssClassName.STATUS_LOZENGE, _editorSharedStyles.akEditorSelectedBorderSize, _editorSharedStyles.akEditorDeleteBorder);
 exports.statusStyles = statusStyles;

package/dist/cjs/plugins/status/ui/statusPicker.js

@@ -29,7 +29,7 @@ var _styledComponents = _interopRequireDefault(require("styled-components"));
 
 var _analyticsNext = require("@atlaskit/analytics-next");
 
-var _editorCommon = require("@atlaskit/editor-common");
+var _ui = require("@atlaskit/editor-common/ui");
 
 var _editorSharedStyles = require("@atlaskit/editor-shared-styles");
 
@@ -53,7 +53,7 @@ function _createSuper(Derived) { var hasNativeReflectConstruct = _isNativeReflec
 
 function _isNativeReflectConstruct() { if (typeof Reflect === "undefined" || !Reflect.construct) return false; if (Reflect.construct.sham) return false; if (typeof Proxy === "function") return true; try { Boolean.prototype.valueOf.call(Reflect.construct(Boolean, [], function () {})); return true; } catch (e) { return false; } }
 
-var PopupWithListeners = (0, _withOuterListeners.default)(_editorCommon.Popup);
+var PopupWithListeners = (0, _withOuterListeners.default)(_ui.Popup);
 var InputMethod;
 exports.InputMethod = InputMethod;
 
@@ -248,8 +248,7 @@ var StatusPickerWithoutAnalytcs = /*#__PURE__*/function (_React$Component) {
           target = _this$props.target;
       var _this$state3 = this.state,
           color = _this$state3.color,
-          text = _this$state3.text,
-          localId = _this$state3.localId;
+          text = _this$state3.text;
       return target && /*#__PURE__*/_react.default.createElement(PopupWithListeners, {
         target: target,
         offset: [0, 8],
@@ -266,8 +265,7 @@ var StatusPickerWithoutAnalytcs = /*#__PURE__*/function (_React$Component) {
         onColorClick: this.onColorClick,
         onColorHover: this.onColorHover,
         onTextChanged: this.onTextChanged,
-        onEnter: this.onEnter,
-        localId: localId
+        onEnter: this.onEnter
       })));
     }
   }]);

package/dist/cjs/plugins/table/commands/collapse.js

@@ -8,14 +8,14 @@ exports.wrapTableInExpand = void 0;
 var _collapse = require("../utils/collapse");
 
 var wrapTableInExpand = function wrapTableInExpand(state, dispatch) {
-  var canCollapseTr = (0, _collapse.collapseSelectedTable)(state.tr);
+  var collapseTr = (0, _collapse.collapseSelectedTable)(state.tr);
 
-  if (!canCollapseTr) {
+  if (!collapseTr) {
     return false;
   }
 
   if (dispatch) {
-    dispatch(canCollapseTr);
+    dispatch(collapseTr);
   }
 
   return true;

package/dist/cjs/plugins/table/commands/misc.js

@@ -314,8 +314,19 @@ var moveCursorBackward = function moveCursorBackward(state, dispatch) {
   // it doesn't join node before (last cell) with node after (content after the cursor)
   // due to ridiculous amount of PM code that would have been required to overwrite
 
+  tr.setSelection(new _prosemirrorState.TextSelection(state.doc.resolve(pos))); // if we are inside an empty paragraph not at the end of the doc we delete it
+
+  var cursorNode = $cursor.node();
+  var docEnd = state.doc.content.size;
+  var paragraphWrapStart = $cursor.pos - 1;
+  var paragraphWrapEnd = $cursor.pos + 1;
+
+  if (cursorNode.content.size === 0 && $cursor.pos + 1 !== docEnd) {
+    tr.delete(paragraphWrapStart, paragraphWrapEnd);
+  }
+
   if (dispatch) {
-    dispatch(tr.setSelection(new _prosemirrorState.TextSelection(state.doc.resolve(pos))));
+    dispatch(tr);
   }
 
   return true;

package/dist/cjs/plugins/table/commands/sort.js

@@ -11,7 +11,7 @@ var _tableMap = require("@atlaskit/editor-tables/table-map");
 
 var _utils = require("@atlaskit/editor-tables/utils");
 
-var _editorCommon = require("@atlaskit/editor-common");
+var _utils2 = require("@atlaskit/editor-common/utils");
 
 var _main = require("../../card/pm-plugins/main");
 
@@ -81,7 +81,7 @@ var sortByColumn = function sortByColumn(columnIndex) {
       headerRow = tableArray.shift();
     }
 
-    var compareNodesInOrder = (0, _editorCommon.createCompareNodes)({
+    var compareNodesInOrder = (0, _utils2.createCompareNodes)({
       getInlineCardTextFromStore: createGetInlineCardTextFromStore(state)
     }, order);
     var sortedTable = tableArray.sort(function (rowA, rowB) {

package/dist/cjs/plugins/table/event-handlers.js

@@ -19,11 +19,9 @@ var _tableMap = require("@atlaskit/editor-tables/table-map");
 
 var _utils = require("@atlaskit/editor-tables/utils");
 
-var _rafSchd = _interopRequireDefault(require("raf-schd"));
+var _utils2 = require("@atlaskit/editor-common/utils");
 
-var _editorCommon = require("@atlaskit/editor-common");
-
-var _utils2 = require("../../utils");
+var _utils3 = require("../../utils");
 
 var _dom = require("../../utils/dom");
 
@@ -39,7 +37,7 @@ var _transforms = require("./transforms");
 
 var _types = require("./types");
 
-var _utils3 = require("./utils");
+var _utils4 = require("./utils");
 
 var _getAllowAddColumnCustomStep = require("./utils/get-allow-add-column-custom-step");
 
@@ -58,7 +56,7 @@ var handleBlur = function handleBlur(view, event) {
       dispatch = view.dispatch; // IE version check for ED-4665
   // Calendar focus check for ED-10466
 
-  if (_editorCommon.browser.ie_version !== 11 && !isFocusingCalendar(event) && !isFocusingModal(event)) {
+  if (_utils2.browser.ie_version !== 11 && !isFocusingCalendar(event) && !isFocusingModal(event)) {
     (0, _commands.setEditorFocus)(false)(state, dispatch);
   }
 
@@ -86,8 +84,8 @@ var handleClick = function handleClick(view, event) {
   var element = event.target;
   var table = (0, _utils.findTable)(view.state.selection);
 
-  if (event instanceof MouseEvent && (0, _utils3.isColumnControlsDecorations)(element)) {
-    var _getColumnOrRowIndex = (0, _utils3.getColumnOrRowIndex)(element),
+  if (event instanceof MouseEvent && (0, _utils4.isColumnControlsDecorations)(element)) {
+    var _getColumnOrRowIndex = (0, _utils4.getColumnOrRowIndex)(element),
         _getColumnOrRowIndex2 = (0, _slicedToArray2.default)(_getColumnOrRowIndex, 1),
         startIndex = _getColumnOrRowIndex2[0];
 
@@ -98,7 +96,7 @@ var handleClick = function handleClick(view, event) {
 
   var matchfn = element.matches ? element.matches : element.msMatchesSelector; // check if the table cell with an image is clicked and its not the image itself
 
-  if (!table || !(0, _utils2.isElementInTableCell)(element) || !matchfn || matchfn.call(element, 'table .image, table p, table .image div')) {
+  if (!table || !(0, _utils3.isElementInTableCell)(element) || !matchfn || matchfn.call(element, 'table .image, table p, table .image div')) {
     return false;
   }
 
@@ -111,7 +109,6 @@ var handleClick = function handleClick(view, event) {
   var rowElement = (0, _dom.closestElement)(element, 'tr');
   var rowIndex = rowElement && rowElement.rowIndex;
   var cellIndex = map.width * rowIndex + colIndex;
-  var posInTable = map.map[cellIndex + 1];
   var dispatch = view.dispatch,
       _view$state = view.state,
       tr = _view$state.tr,
@@ -119,10 +116,11 @@ var handleClick = function handleClick(view, event) {
   var editorElement = table.node.nodeAt(map.map[cellIndex]);
   /** Only if the last item is media group, insert a paragraph */
 
-  if ((0, _utils2.isLastItemMediaGroup)(editorElement)) {
+  if ((0, _utils3.isLastItemMediaGroup)(editorElement)) {
+    var posInTable = map.map[cellIndex] + editorElement.nodeSize;
     tr.insert(posInTable + table.pos, paragraph.create());
     dispatch(tr);
-    (0, _utils2.setNodeSelection)(view, posInTable + table.pos);
+    (0, _utils3.setNodeSelection)(view, posInTable + table.pos);
   }
 
   return true;
@@ -143,18 +141,18 @@ var handleMouseOver = function handleMouseOver(view, mouseEvent) {
       insertColumnButtonIndex = _getPluginState.insertColumnButtonIndex,
       insertRowButtonIndex = _getPluginState.insertRowButtonIndex;
 
-  if ((0, _utils3.isInsertRowButton)(target)) {
-    var _getColumnOrRowIndex3 = (0, _utils3.getColumnOrRowIndex)(target),
+  if ((0, _utils4.isInsertRowButton)(target)) {
+    var _getColumnOrRowIndex3 = (0, _utils4.getColumnOrRowIndex)(target),
         _getColumnOrRowIndex4 = (0, _slicedToArray2.default)(_getColumnOrRowIndex3, 2),
         startIndex = _getColumnOrRowIndex4[0],
         endIndex = _getColumnOrRowIndex4[1];
 
-    var positionRow = (0, _utils3.getMousePositionVerticalRelativeByElement)(mouseEvent) === 'bottom' ? endIndex : startIndex;
+    var positionRow = (0, _utils4.getMousePositionVerticalRelativeByElement)(mouseEvent) === 'bottom' ? endIndex : startIndex;
     return (0, _commands.showInsertRowButton)(positionRow)(state, dispatch);
   }
 
-  if ((0, _utils3.isColumnControlsDecorations)(target)) {
-    var _getColumnOrRowIndex5 = (0, _utils3.getColumnOrRowIndex)(target),
+  if ((0, _utils4.isColumnControlsDecorations)(target)) {
+    var _getColumnOrRowIndex5 = (0, _utils4.getColumnOrRowIndex)(target),
         _getColumnOrRowIndex6 = (0, _slicedToArray2.default)(_getColumnOrRowIndex5, 1),
         _startIndex = _getColumnOrRowIndex6[0];
 
@@ -163,12 +161,12 @@ var handleMouseOver = function handleMouseOver(view, mouseEvent) {
     return (0, _commands.hoverColumns)([_startIndex], false)(_state, _dispatch2);
   }
 
-  if (((0, _utils3.isCell)(target) || (0, _utils3.isCornerButton)(target)) && (typeof insertColumnButtonIndex === 'number' || typeof insertRowButtonIndex === 'number')) {
+  if (((0, _utils4.isCell)(target) || (0, _utils4.isCornerButton)(target)) && (typeof insertColumnButtonIndex === 'number' || typeof insertRowButtonIndex === 'number')) {
     return (0, _commands.hideInsertColumnOrRowButton)()(state, dispatch);
   }
 
-  if ((0, _utils3.isResizeHandleDecoration)(target)) {
-    var _getColumnOrRowIndex7 = (0, _utils3.getColumnOrRowIndex)(target),
+  if ((0, _utils4.isResizeHandleDecoration)(target)) {
+    var _getColumnOrRowIndex7 = (0, _utils4.getColumnOrRowIndex)(target),
         _getColumnOrRowIndex8 = (0, _slicedToArray2.default)(_getColumnOrRowIndex7, 2),
         _startIndex2 = _getColumnOrRowIndex8[0],
         _endIndex = _getColumnOrRowIndex8[1];
@@ -187,7 +185,7 @@ var handleMouseOver = function handleMouseOver(view, mouseEvent) {
 exports.handleMouseOver = handleMouseOver;
 
 var handleMouseDown = function handleMouseDown(_, event) {
-  var isControl = !!(event.target && event.target instanceof HTMLElement && ((0, _utils3.isTableContainerOrWrapper)(event.target) || (0, _utils3.isColumnControlsDecorations)(event.target) || (0, _utils3.isRowControlsButton)(event.target)));
+  var isControl = !!(event.target && event.target instanceof HTMLElement && ((0, _utils4.isTableContainerOrWrapper)(event.target) || (0, _utils4.isColumnControlsDecorations)(event.target) || (0, _utils4.isRowControlsButton)(event.target)));
 
   if (isControl) {
     event.preventDefault();
@@ -205,7 +203,7 @@ var handleMouseOut = function handleMouseOut(view, mouseEvent) {
 
   var target = mouseEvent.target;
 
-  if ((0, _utils3.isColumnControlsDecorations)(target)) {
+  if ((0, _utils4.isColumnControlsDecorations)(target)) {
     var state = view.state,
         dispatch = view.dispatch;
     return (0, _commands.clearHoverSelection)()(state, dispatch);
@@ -214,7 +212,7 @@ var handleMouseOut = function handleMouseOut(view, mouseEvent) {
   var relatedTarget = mouseEvent.relatedTarget; // In case the user is moving between cell at the same column
   // we don't need to hide the resize handle decoration
 
-  if ((0, _utils3.isResizeHandleDecoration)(target) && !(0, _utils3.isResizeHandleDecoration)(relatedTarget)) {
+  if ((0, _utils4.isResizeHandleDecoration)(target) && !(0, _utils4.isResizeHandleDecoration)(relatedTarget)) {
     var _state2 = view.state,
         _dispatch3 = view.dispatch;
     return (0, _commands.hideResizeHandleLine)()(_state2, _dispatch3);
@@ -239,7 +237,7 @@ var handleMouseLeave = function handleMouseLeave(view, event) {
 
   var target = event.target;
 
-  if ((0, _utils3.isTableControlsButton)(target)) {
+  if ((0, _utils4.isTableControlsButton)(target)) {
     return true;
   }
 
@@ -259,38 +257,38 @@ var handleMouseMove = function handleMouseMove(view, event, tableCellOptimizatio
 
   var element = event.target;
 
-  if ((0, _utils3.isColumnControlsDecorations)(element)) {
+  if ((0, _utils4.isColumnControlsDecorations)(element)) {
     var state = view.state,
         dispatch = view.dispatch;
 
     var _getPluginState3 = (0, _pluginFactory.getPluginState)(state),
         insertColumnButtonIndex = _getPluginState3.insertColumnButtonIndex;
 
-    var _getColumnOrRowIndex9 = (0, _utils3.getColumnOrRowIndex)(element),
+    var _getColumnOrRowIndex9 = (0, _utils4.getColumnOrRowIndex)(element),
         _getColumnOrRowIndex10 = (0, _slicedToArray2.default)(_getColumnOrRowIndex9, 2),
         startIndex = _getColumnOrRowIndex10[0],
         endIndex = _getColumnOrRowIndex10[1];
 
-    var positionColumn = (0, _utils3.getMousePositionHorizontalRelativeByElement)(event, tableCellOptimization, elementContentRects) === 'right' ? endIndex : startIndex;
+    var positionColumn = (0, _utils4.getMousePositionHorizontalRelativeByElement)(event, tableCellOptimization, elementContentRects) === 'right' ? endIndex : startIndex;
 
     if (positionColumn !== insertColumnButtonIndex) {
       return (0, _commands.showInsertColumnButton)(positionColumn)(state, dispatch);
     }
   }
 
-  if ((0, _utils3.isRowControlsButton)(element)) {
+  if ((0, _utils4.isRowControlsButton)(element)) {
     var _state3 = view.state,
         _dispatch4 = view.dispatch;
 
     var _getPluginState4 = (0, _pluginFactory.getPluginState)(_state3),
         insertRowButtonIndex = _getPluginState4.insertRowButtonIndex;
 
-    var _getColumnOrRowIndex11 = (0, _utils3.getColumnOrRowIndex)(element),
+    var _getColumnOrRowIndex11 = (0, _utils4.getColumnOrRowIndex)(element),
         _getColumnOrRowIndex12 = (0, _slicedToArray2.default)(_getColumnOrRowIndex11, 2),
         _startIndex3 = _getColumnOrRowIndex12[0],
         _endIndex2 = _getColumnOrRowIndex12[1];
 
-    var positionRow = (0, _utils3.getMousePositionVerticalRelativeByElement)(event) === 'bottom' ? _endIndex2 : _startIndex3;
+    var positionRow = (0, _utils4.getMousePositionVerticalRelativeByElement)(event) === 'bottom' ? _endIndex2 : _startIndex3;
 
     if (positionRow !== insertRowButtonIndex) {
       return (0, _commands.showInsertRowButton)(positionRow)(_state3, _dispatch4);
@@ -304,8 +302,8 @@ var handleMouseMove = function handleMouseMove(view, event, tableCellOptimizatio
 
   var useMouseMoveOptimisation = tableCellOptimization && mouseMoveOptimization;
 
-  if (!(0, _utils3.isResizeHandleDecoration)(element) && (0, _utils3.isCell)(element)) {
-    var _positionColumn = (0, _utils3.getMousePositionHorizontalRelativeByElement)(event, useMouseMoveOptimisation, elementContentRects, _types.RESIZE_HANDLE_AREA_DECORATION_GAP);
+  if (!(0, _utils4.isResizeHandleDecoration)(element) && (0, _utils4.isCell)(element)) {
+    var _positionColumn = (0, _utils4.getMousePositionHorizontalRelativeByElement)(event, useMouseMoveOptimisation, elementContentRects, _types.RESIZE_HANDLE_AREA_DECORATION_GAP);
 
     if (_positionColumn !== null) {
       var _state4 = view.state,
@@ -321,7 +319,7 @@ var handleMouseMove = function handleMouseMove(view, event, tableCellOptimizatio
       if (rect) {
         var columnEndIndexTarget = _positionColumn === 'left' ? rect.left : rect.right;
 
-        if (columnEndIndexTarget !== resizeHandleColumnIndex || !(0, _utils3.hasResizeHandler)({
+        if (columnEndIndexTarget !== resizeHandleColumnIndex || !(0, _utils4.hasResizeHandler)({
           target: element,
           columnEndIndexTarget: columnEndIndexTarget
         })) {
@@ -375,7 +373,7 @@ var handleCut = function handleCut(oldTr, oldState, newState) {
       var rect = (0, _utils.getSelectionRect)(cellSelection);
 
       if (rect) {
-        var _getSelectedCellInfo = (0, _utils3.getSelectedCellInfo)(tr.selection),
+        var _getSelectedCellInfo = (0, _utils4.getSelectedCellInfo)(tr.selection),
             verticalCells = _getSelectedCellInfo.verticalCells,
             horizontalCells = _getSelectedCellInfo.horizontalCells,
             totalCells = _getSelectedCellInfo.totalCells,
@@ -432,10 +430,9 @@ var whenTableInFocus = function whenTableInFocus(eventHandler, elementContentRec
 
     if (!hasTableNode || isDragging) {
       return false;
-    } // debounce event handler
-
+    }
 
-    return (0, _rafSchd.default)(eventHandler(view, mouseEvent, tableCellOptimization, elementContentRects));
+    return eventHandler(view, mouseEvent, tableCellOptimization, elementContentRects);
   };
 };