@atlaskit/editor-plugin-tasks-and-decisions 6.3.2 → 6.4.0

package/CHANGELOG.md

@@ -1,5 +1,20 @@
 # @atlaskit/editor-plugin-tasks-and-decisions
 
+## 6.4.0
+
+### Minor Changes
+
+- [`8ce4178519d92`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/8ce4178519d92) -
+  Added ignore mutation for decision node view
+
+## 6.3.3
+
+### Patch Changes
+
+- [`57b19274b9fdd`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/57b19274b9fdd) -
+  EDITOR-1373 Bump adf-schema version
+- Updated dependencies
+
 ## 6.3.2
 
 ### Patch Changes

package/dist/cjs/nodeviews/DecisionItemNodeView.js

@@ -9,6 +9,7 @@ var _classCallCheck2 = _interopRequireDefault(require("@babel/runtime/helpers/cl
 var _createClass2 = _interopRequireDefault(require("@babel/runtime/helpers/createClass"));
 var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/defineProperty"));
 var _model = require("@atlaskit/editor-prosemirror/model");
+var _expVal = require("@atlaskit/tmp-editor-statsig/expVal");
 var _decisionItemNodeSpec = require("./decisionItemNodeSpec");
 /**
  * NodeView for the DecisionItem node.
@@ -66,5 +67,30 @@ var DecisionItemNodeView = exports.DecisionItemNodeView = /*#__PURE__*/function
       this.updateHasChildren(node);
       return true;
     }
+
+    /**
+     * Determines whether a mutation should be ignored by ProseMirror.
+     * This prevents unnecessary node view destruction during DOM changes that don't affect our content.
+     * @param mutation - The DOM mutation record or selection change
+     * @returns true if the mutation should be ignored, false if it should be processed
+     * @example
+     * // Mutation outside contentDOM will be ignored
+     * const shouldIgnore = decisionItemNodeView.ignoreMutation(mutationRecord);
+     */
+  }, {
+    key: "ignoreMutation",
+    value: function ignoreMutation(mutation) {
+      // This was discovered while implementing the platform_editor_debounce_portal_provider experiment
+      // And there will only be an issue if the experiment is enabled.
+      // As such, this fix is behind this experiment.
+      if (!(0, _expVal.expVal)('platform_editor_debounce_portal_provider', 'isEnabled', false)) {
+        return false; // Let ProseMirror handle all mutations when experiment is disabled
+      }
+      if (!this.contentDOM) {
+        return true;
+      }
+      // Ignore mutations that don't target our contentDOM and aren't selection changes
+      return !this.contentDOM.contains(mutation.target) && mutation.type !== 'selection';
+    }
   }]);
 }();

package/dist/es2019/nodeviews/DecisionItemNodeView.js

@@ -1,5 +1,6 @@
 import _defineProperty from "@babel/runtime/helpers/defineProperty";
 import { DOMSerializer } from '@atlaskit/editor-prosemirror/model';
+import { expVal } from '@atlaskit/tmp-editor-statsig/expVal';
 import { decisionItemToDOM } from './decisionItemNodeSpec';
 
 /**
@@ -54,4 +55,27 @@ export class DecisionItemNodeView {
     this.updateHasChildren(node);
     return true;
   }
+
+  /**
+   * Determines whether a mutation should be ignored by ProseMirror.
+   * This prevents unnecessary node view destruction during DOM changes that don't affect our content.
+   * @param mutation - The DOM mutation record or selection change
+   * @returns true if the mutation should be ignored, false if it should be processed
+   * @example
+   * // Mutation outside contentDOM will be ignored
+   * const shouldIgnore = decisionItemNodeView.ignoreMutation(mutationRecord);
+   */
+  ignoreMutation(mutation) {
+    // This was discovered while implementing the platform_editor_debounce_portal_provider experiment
+    // And there will only be an issue if the experiment is enabled.
+    // As such, this fix is behind this experiment.
+    if (!expVal('platform_editor_debounce_portal_provider', 'isEnabled', false)) {
+      return false; // Let ProseMirror handle all mutations when experiment is disabled
+    }
+    if (!this.contentDOM) {
+      return true;
+    }
+    // Ignore mutations that don't target our contentDOM and aren't selection changes
+    return !this.contentDOM.contains(mutation.target) && mutation.type !== 'selection';
+  }
 }

package/dist/esm/nodeviews/DecisionItemNodeView.js

@@ -2,6 +2,7 @@ import _classCallCheck from "@babel/runtime/helpers/classCallCheck";
 import _createClass from "@babel/runtime/helpers/createClass";
 import _defineProperty from "@babel/runtime/helpers/defineProperty";
 import { DOMSerializer } from '@atlaskit/editor-prosemirror/model';
+import { expVal } from '@atlaskit/tmp-editor-statsig/expVal';
 import { decisionItemToDOM } from './decisionItemNodeSpec';
 
 /**
@@ -60,5 +61,30 @@ export var DecisionItemNodeView = /*#__PURE__*/function () {
       this.updateHasChildren(node);
       return true;
     }
+
+    /**
+     * Determines whether a mutation should be ignored by ProseMirror.
+     * This prevents unnecessary node view destruction during DOM changes that don't affect our content.
+     * @param mutation - The DOM mutation record or selection change
+     * @returns true if the mutation should be ignored, false if it should be processed
+     * @example
+     * // Mutation outside contentDOM will be ignored
+     * const shouldIgnore = decisionItemNodeView.ignoreMutation(mutationRecord);
+     */
+  }, {
+    key: "ignoreMutation",
+    value: function ignoreMutation(mutation) {
+      // This was discovered while implementing the platform_editor_debounce_portal_provider experiment
+      // And there will only be an issue if the experiment is enabled.
+      // As such, this fix is behind this experiment.
+      if (!expVal('platform_editor_debounce_portal_provider', 'isEnabled', false)) {
+        return false; // Let ProseMirror handle all mutations when experiment is disabled
+      }
+      if (!this.contentDOM) {
+        return true;
+      }
+      // Ignore mutations that don't target our contentDOM and aren't selection changes
+      return !this.contentDOM.contains(mutation.target) && mutation.type !== 'selection';
+    }
   }]);
 }();

package/dist/types/nodeviews/DecisionItemNodeView.d.ts

@@ -34,4 +34,17 @@ export declare class DecisionItemNodeView implements NodeView {
      * @returns {boolean} - Returns true if the view was updated successfully.
      */
     update(node: PMNode): boolean;
+    /**
+     * Determines whether a mutation should be ignored by ProseMirror.
+     * This prevents unnecessary node view destruction during DOM changes that don't affect our content.
+     * @param mutation - The DOM mutation record or selection change
+     * @returns true if the mutation should be ignored, false if it should be processed
+     * @example
+     * // Mutation outside contentDOM will be ignored
+     * const shouldIgnore = decisionItemNodeView.ignoreMutation(mutationRecord);
+     */
+    ignoreMutation(mutation: MutationRecord | {
+        type: 'selection';
+        target: Node;
+    }): boolean;
 }

package/dist/types-ts4.5/nodeviews/DecisionItemNodeView.d.ts

@@ -34,4 +34,17 @@ export declare class DecisionItemNodeView implements NodeView {
      * @returns {boolean} - Returns true if the view was updated successfully.
      */
     update(node: PMNode): boolean;
+    /**
+     * Determines whether a mutation should be ignored by ProseMirror.
+     * This prevents unnecessary node view destruction during DOM changes that don't affect our content.
+     * @param mutation - The DOM mutation record or selection change
+     * @returns true if the mutation should be ignored, false if it should be processed
+     * @example
+     * // Mutation outside contentDOM will be ignored
+     * const shouldIgnore = decisionItemNodeView.ignoreMutation(mutationRecord);
+     */
+    ignoreMutation(mutation: MutationRecord | {
+        type: 'selection';
+        target: Node;
+    }): boolean;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@atlaskit/editor-plugin-tasks-and-decisions",
-	"version": "6.3.2",
+	"version": "6.4.0",
 	"description": "Tasks and decisions plugin for @atlaskit/editor-core",
 	"author": "Atlassian Pty Ltd",
 	"license": "Apache-2.0",
@@ -32,7 +32,7 @@
 		".": "./src/index.ts"
 	},
 	"dependencies": {
-		"@atlaskit/adf-schema": "^50.2.0",
+		"@atlaskit/adf-schema": "^50.2.1",
 		"@atlaskit/analytics-namespaced-context": "^7.0.0",
 		"@atlaskit/analytics-next": "^11.1.0",
 		"@atlaskit/css": "^0.12.0",
@@ -49,14 +49,14 @@
 		"@atlaskit/primitives": "^14.11.0",
 		"@atlaskit/prosemirror-input-rules": "^3.4.0",
 		"@atlaskit/task-decision": "^19.2.0",
-		"@atlaskit/tmp-editor-statsig": "^11.2.0",
+		"@atlaskit/tmp-editor-statsig": "^11.3.0",
 		"@atlaskit/tokens": "^6.0.0",
 		"@babel/runtime": "^7.0.0",
 		"@compiled/react": "^0.18.3",
 		"bind-event-listener": "^3.0.0"
 	},
 	"peerDependencies": {
-		"@atlaskit/editor-common": "^107.25.0",
+		"@atlaskit/editor-common": "^107.26.0",
 		"react": "^18.2.0",
 		"react-dom": "^18.2.0",
 		"react-intl-next": "npm:react-intl@^5.18.1"