@atlaskit/editor-plugin-block-controls 8.5.1 → 8.6.0

package/CHANGELOG.md

@@ -1,5 +1,16 @@
 # @atlaskit/editor-plugin-block-controls
 
+## 8.6.0
+
+### Minor Changes
+
+- [`0190ead71b6e6`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/0190ead71b6e6) -
+  [ux] Cleanup fg: platform_editor_selection_sync_fix
+
+### Patch Changes
+
+- Updated dependencies
+
 ## 8.5.1
 
 ### Patch Changes

package/dist/cjs/pm-plugins/selection-preservation/pm-plugin.js

@@ -10,7 +10,6 @@ var _bindEventListener = require("bind-event-listener");
 var _browserApis = require("@atlaskit/browser-apis");
 var _safePlugin = require("@atlaskit/editor-common/safe-plugin");
 var _styles = require("@atlaskit/editor-common/styles");
-var _platformFeatureFlags = require("@atlaskit/platform-feature-flags");
 var _main = require("../main");
 var _selection = require("../utils/selection");
 var _editorCommands = require("./editor-commands");
@@ -159,42 +158,23 @@ var createSelectionPreservationPlugin = exports.createSelectionPreservationPlugi
         });
         return {
           update: function update(updateView, prevState) {
+            var _selectionPreservatio, _selectionPreservatio2, _key$getState, _key$getState2;
             view = updateView;
+            var prevPreservedSelection = (_selectionPreservatio = _pluginKey.selectionPreservationPluginKey.getState(prevState)) === null || _selectionPreservatio === void 0 ? void 0 : _selectionPreservatio.preservedSelection;
+            var currPreservedSelection = (_selectionPreservatio2 = _pluginKey.selectionPreservationPluginKey.getState(view.state)) === null || _selectionPreservatio2 === void 0 ? void 0 : _selectionPreservatio2.preservedSelection;
+            var prevActiveNode = (_key$getState = _main.key.getState(prevState)) === null || _key$getState === void 0 ? void 0 : _key$getState.activeNode;
+            var currActiveNode = (_key$getState2 = _main.key.getState(view.state)) === null || _key$getState2 === void 0 ? void 0 : _key$getState2.activeNode;
 
-            // [FEATURE FLAG: platform_editor_selection_sync_fix]
-            // When enabled, syncs DOM selection even when editor doesn't have focus.
-            // This prevents ghost highlighting after moving nodes when block menu is open.
-            // To clean up: remove the if-else block and keep only the flag-on behavior.
-            if ((0, _platformFeatureFlags.fg)('platform_editor_selection_sync_fix')) {
-              var _selectionPreservatio, _selectionPreservatio2, _key$getState, _key$getState2;
-              var prevPreservedSelection = (_selectionPreservatio = _pluginKey.selectionPreservationPluginKey.getState(prevState)) === null || _selectionPreservatio === void 0 ? void 0 : _selectionPreservatio.preservedSelection;
-              var currPreservedSelection = (_selectionPreservatio2 = _pluginKey.selectionPreservationPluginKey.getState(view.state)) === null || _selectionPreservatio2 === void 0 ? void 0 : _selectionPreservatio2.preservedSelection;
-              var prevActiveNode = (_key$getState = _main.key.getState(prevState)) === null || _key$getState === void 0 ? void 0 : _key$getState.activeNode;
-              var currActiveNode = (_key$getState2 = _main.key.getState(view.state)) === null || _key$getState2 === void 0 ? void 0 : _key$getState2.activeNode;
-
-              // Sync DOM selection when the preserved selection or active node changes
-              // AND the document has changed (e.g., nodes moved)
-              // This prevents stealing focus during menu navigation while still fixing ghost highlighting
-              var hasPreservedSelection = !!currPreservedSelection;
-              var preservedSelectionChanged = !(0, _utils.compareSelections)(prevPreservedSelection, currPreservedSelection);
-              var activeNodeChanged = prevActiveNode !== currActiveNode;
-              var docChanged = prevState.doc !== view.state.doc;
-              var shouldSyncDOMSelection = hasPreservedSelection && (preservedSelectionChanged || activeNodeChanged) && docChanged;
-              if (shouldSyncDOMSelection) {
-                (0, _utils.syncDOMSelection)(view.state.selection, view);
-              }
-            } else {
-              var _selectionPreservatio3, _selectionPreservatio4, _key$getState3, _key$getState4;
-              // OLD BEHAVIOR (to be removed when flag is cleaned up)
-              // Only synced when editor had focus, causing ghost highlighting issues
-              var _prevPreservedSelection = (_selectionPreservatio3 = _pluginKey.selectionPreservationPluginKey.getState(prevState)) === null || _selectionPreservatio3 === void 0 ? void 0 : _selectionPreservatio3.preservedSelection;
-              var _currPreservedSelection = (_selectionPreservatio4 = _pluginKey.selectionPreservationPluginKey.getState(view.state)) === null || _selectionPreservatio4 === void 0 ? void 0 : _selectionPreservatio4.preservedSelection;
-              var _prevActiveNode = (_key$getState3 = _main.key.getState(prevState)) === null || _key$getState3 === void 0 ? void 0 : _key$getState3.activeNode;
-              var _currActiveNode = (_key$getState4 = _main.key.getState(view.state)) === null || _key$getState4 === void 0 ? void 0 : _key$getState4.activeNode;
-              if (_currPreservedSelection && view.hasFocus() && (!(0, _utils.compareSelections)(_prevPreservedSelection, _currPreservedSelection) || _prevActiveNode !== _currActiveNode)) {
-                // Old syncDOMSelection signature (to be removed)
-                (0, _utils.syncDOMSelection)(view.state.selection);
-              }
+            // Sync DOM selection when the preserved selection or active node changes
+            // AND the document has changed (e.g., nodes moved)
+            // This prevents stealing focus during menu navigation while still fixing ghost highlighting
+            var hasPreservedSelection = !!currPreservedSelection;
+            var preservedSelectionChanged = !(0, _utils.compareSelections)(prevPreservedSelection, currPreservedSelection);
+            var activeNodeChanged = prevActiveNode !== currActiveNode;
+            var docChanged = prevState.doc !== view.state.doc;
+            var shouldSyncDOMSelection = hasPreservedSelection && (preservedSelectionChanged || activeNodeChanged) && docChanged;
+            if (shouldSyncDOMSelection) {
+              (0, _utils.syncDOMSelection)(view.state.selection, view);
             }
           },
           destroy: function destroy() {

package/dist/cjs/pm-plugins/selection-preservation/utils.js

@@ -5,7 +5,6 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.syncDOMSelection = exports.hasUserSelectionChange = exports.getSelectionPreservationMeta = exports.compareSelections = void 0;
 var _browserApis = require("@atlaskit/browser-apis");
-var _platformFeatureFlags = require("@atlaskit/platform-feature-flags");
 var _pluginKey = require("./plugin-key");
 /**
  * Detects if any of the transactions include user-driven selection changes.
@@ -41,54 +40,36 @@ var compareSelections = exports.compareSelections = function compareSelections(a
  * the browser's native selection remains at the old position, causing ghost highlighting.
  *
  * @param selection The current ProseMirror selection state to sync to DOM.
- * @param view The EditorView instance used to convert ProseMirror positions to DOM positions (when feature flag is enabled).
+ * @param view The EditorView instance used to convert ProseMirror positions to DOM positions.
  */
 var syncDOMSelection = exports.syncDOMSelection = function syncDOMSelection(selection, view) {
-  // [FEATURE FLAG: platform_editor_selection_sync_fix]
-  // When enabled, uses improved DOM selection syncing with EditorView.
-  // To clean up: remove the if-else block, remove the optional view parameter,
-  // make view required, and keep only the flag-on behavior.
-  if (view && (0, _platformFeatureFlags.fg)('platform_editor_selection_sync_fix')) {
-    try {
-      var domSelection = window.getSelection();
-      if (!domSelection) {
-        return;
-      }
-      var doc = (0, _browserApis.getDocument)();
-      if (!doc) {
-        return;
-      }
+  try {
+    var domSelection = window.getSelection();
+    if (!domSelection) {
+      return;
+    }
+    var doc = (0, _browserApis.getDocument)();
+    if (!doc) {
+      return;
+    }
 
-      // Convert ProseMirror selection to DOM selection using view.domAtPos
-      var anchor = view.domAtPos(selection.anchor);
-      var head = view.domAtPos(selection.head);
-      if (!anchor || !head) {
-        return;
-      }
+    // Convert ProseMirror selection to DOM selection using view.domAtPos
+    var anchor = view.domAtPos(selection.anchor);
+    var head = view.domAtPos(selection.head);
+    if (!anchor || !head) {
+      return;
+    }
 
-      // Create a new DOM range from the ProseMirror selection
-      var range = doc.createRange();
-      range.setStart(anchor.node, anchor.offset);
-      range.setEnd(head.node, head.offset);
+    // Create a new DOM range from the ProseMirror selection
+    var range = doc.createRange();
+    range.setStart(anchor.node, anchor.offset);
+    range.setEnd(head.node, head.offset);
 
-      // Update the DOM selection to match ProseMirror's selection
-      domSelection.removeAllRanges();
-      domSelection.addRange(range);
-    } catch (_unused) {
-      // Silently fail if DOM selection sync fails
-      // This can happen if positions are invalid or DOM hasn't updated yet
-    }
-  } else {
-    // OLD BEHAVIOR (to be removed when flag is cleaned up)
-    // Only checked if selection was out of sync using incorrect offset comparison
-    var _domSelection = window.getSelection();
-    var domRange = _domSelection && _domSelection.rangeCount === 1 && _domSelection.getRangeAt(0).cloneRange();
-    var isOutOfSync = domRange && (selection.from !== domRange.startOffset || selection.to !== domRange.endOffset);
-    if (isOutOfSync && _domSelection && domRange) {
-      // Force the DOM selection to refresh, setting it to the same range
-      // This will trigger ProseMirror to re-apply its selection logic based on the current state
-      _domSelection.removeAllRanges();
-      _domSelection.addRange(domRange);
-    }
+    // Update the DOM selection to match ProseMirror's selection
+    domSelection.removeAllRanges();
+    domSelection.addRange(range);
+  } catch (_unused) {
+    // Silently fail if DOM selection sync fails
+    // This can happen if positions are invalid or DOM hasn't updated yet
   }
 };

package/dist/es2019/pm-plugins/selection-preservation/pm-plugin.js

@@ -2,7 +2,6 @@ import { bind } from 'bind-event-listener';
 import { getDocument } from '@atlaskit/browser-apis';
 import { SafePlugin } from '@atlaskit/editor-common/safe-plugin';
 import { DRAG_HANDLE_SELECTOR } from '@atlaskit/editor-common/styles';
-import { fg } from '@atlaskit/platform-feature-flags';
 import { key } from '../main';
 import { createPreservedSelection, mapPreservedSelection } from '../utils/selection';
 import { stopPreservingSelection } from './editor-commands';
@@ -152,42 +151,23 @@ export const createSelectionPreservationPlugin = api => () => {
       });
       return {
         update(updateView, prevState) {
+          var _selectionPreservatio, _selectionPreservatio2, _key$getState, _key$getState2;
           view = updateView;
+          const prevPreservedSelection = (_selectionPreservatio = selectionPreservationPluginKey.getState(prevState)) === null || _selectionPreservatio === void 0 ? void 0 : _selectionPreservatio.preservedSelection;
+          const currPreservedSelection = (_selectionPreservatio2 = selectionPreservationPluginKey.getState(view.state)) === null || _selectionPreservatio2 === void 0 ? void 0 : _selectionPreservatio2.preservedSelection;
+          const prevActiveNode = (_key$getState = key.getState(prevState)) === null || _key$getState === void 0 ? void 0 : _key$getState.activeNode;
+          const currActiveNode = (_key$getState2 = key.getState(view.state)) === null || _key$getState2 === void 0 ? void 0 : _key$getState2.activeNode;
 
-          // [FEATURE FLAG: platform_editor_selection_sync_fix]
-          // When enabled, syncs DOM selection even when editor doesn't have focus.
-          // This prevents ghost highlighting after moving nodes when block menu is open.
-          // To clean up: remove the if-else block and keep only the flag-on behavior.
-          if (fg('platform_editor_selection_sync_fix')) {
-            var _selectionPreservatio, _selectionPreservatio2, _key$getState, _key$getState2;
-            const prevPreservedSelection = (_selectionPreservatio = selectionPreservationPluginKey.getState(prevState)) === null || _selectionPreservatio === void 0 ? void 0 : _selectionPreservatio.preservedSelection;
-            const currPreservedSelection = (_selectionPreservatio2 = selectionPreservationPluginKey.getState(view.state)) === null || _selectionPreservatio2 === void 0 ? void 0 : _selectionPreservatio2.preservedSelection;
-            const prevActiveNode = (_key$getState = key.getState(prevState)) === null || _key$getState === void 0 ? void 0 : _key$getState.activeNode;
-            const currActiveNode = (_key$getState2 = key.getState(view.state)) === null || _key$getState2 === void 0 ? void 0 : _key$getState2.activeNode;
-
-            // Sync DOM selection when the preserved selection or active node changes
-            // AND the document has changed (e.g., nodes moved)
-            // This prevents stealing focus during menu navigation while still fixing ghost highlighting
-            const hasPreservedSelection = !!currPreservedSelection;
-            const preservedSelectionChanged = !compareSelections(prevPreservedSelection, currPreservedSelection);
-            const activeNodeChanged = prevActiveNode !== currActiveNode;
-            const docChanged = prevState.doc !== view.state.doc;
-            const shouldSyncDOMSelection = hasPreservedSelection && (preservedSelectionChanged || activeNodeChanged) && docChanged;
-            if (shouldSyncDOMSelection) {
-              syncDOMSelection(view.state.selection, view);
-            }
-          } else {
-            var _selectionPreservatio3, _selectionPreservatio4, _key$getState3, _key$getState4;
-            // OLD BEHAVIOR (to be removed when flag is cleaned up)
-            // Only synced when editor had focus, causing ghost highlighting issues
-            const prevPreservedSelection = (_selectionPreservatio3 = selectionPreservationPluginKey.getState(prevState)) === null || _selectionPreservatio3 === void 0 ? void 0 : _selectionPreservatio3.preservedSelection;
-            const currPreservedSelection = (_selectionPreservatio4 = selectionPreservationPluginKey.getState(view.state)) === null || _selectionPreservatio4 === void 0 ? void 0 : _selectionPreservatio4.preservedSelection;
-            const prevActiveNode = (_key$getState3 = key.getState(prevState)) === null || _key$getState3 === void 0 ? void 0 : _key$getState3.activeNode;
-            const currActiveNode = (_key$getState4 = key.getState(view.state)) === null || _key$getState4 === void 0 ? void 0 : _key$getState4.activeNode;
-            if (currPreservedSelection && view.hasFocus() && (!compareSelections(prevPreservedSelection, currPreservedSelection) || prevActiveNode !== currActiveNode)) {
-              // Old syncDOMSelection signature (to be removed)
-              syncDOMSelection(view.state.selection);
-            }
+          // Sync DOM selection when the preserved selection or active node changes
+          // AND the document has changed (e.g., nodes moved)
+          // This prevents stealing focus during menu navigation while still fixing ghost highlighting
+          const hasPreservedSelection = !!currPreservedSelection;
+          const preservedSelectionChanged = !compareSelections(prevPreservedSelection, currPreservedSelection);
+          const activeNodeChanged = prevActiveNode !== currActiveNode;
+          const docChanged = prevState.doc !== view.state.doc;
+          const shouldSyncDOMSelection = hasPreservedSelection && (preservedSelectionChanged || activeNodeChanged) && docChanged;
+          if (shouldSyncDOMSelection) {
+            syncDOMSelection(view.state.selection, view);
           }
         },
         destroy() {

package/dist/es2019/pm-plugins/selection-preservation/utils.js

@@ -1,5 +1,4 @@
 import { getDocument } from '@atlaskit/browser-apis';
-import { fg } from '@atlaskit/platform-feature-flags';
 import { selectionPreservationPluginKey } from './plugin-key';
 /**
  * Detects if any of the transactions include user-driven selection changes.
@@ -33,54 +32,36 @@ export const compareSelections = (a, b) => {
  * the browser's native selection remains at the old position, causing ghost highlighting.
  *
  * @param selection The current ProseMirror selection state to sync to DOM.
- * @param view The EditorView instance used to convert ProseMirror positions to DOM positions (when feature flag is enabled).
+ * @param view The EditorView instance used to convert ProseMirror positions to DOM positions.
  */
 export const syncDOMSelection = (selection, view) => {
-  // [FEATURE FLAG: platform_editor_selection_sync_fix]
-  // When enabled, uses improved DOM selection syncing with EditorView.
-  // To clean up: remove the if-else block, remove the optional view parameter,
-  // make view required, and keep only the flag-on behavior.
-  if (view && fg('platform_editor_selection_sync_fix')) {
-    try {
-      const domSelection = window.getSelection();
-      if (!domSelection) {
-        return;
-      }
-      const doc = getDocument();
-      if (!doc) {
-        return;
-      }
+  try {
+    const domSelection = window.getSelection();
+    if (!domSelection) {
+      return;
+    }
+    const doc = getDocument();
+    if (!doc) {
+      return;
+    }
 
-      // Convert ProseMirror selection to DOM selection using view.domAtPos
-      const anchor = view.domAtPos(selection.anchor);
-      const head = view.domAtPos(selection.head);
-      if (!anchor || !head) {
-        return;
-      }
+    // Convert ProseMirror selection to DOM selection using view.domAtPos
+    const anchor = view.domAtPos(selection.anchor);
+    const head = view.domAtPos(selection.head);
+    if (!anchor || !head) {
+      return;
+    }
 
-      // Create a new DOM range from the ProseMirror selection
-      const range = doc.createRange();
-      range.setStart(anchor.node, anchor.offset);
-      range.setEnd(head.node, head.offset);
+    // Create a new DOM range from the ProseMirror selection
+    const range = doc.createRange();
+    range.setStart(anchor.node, anchor.offset);
+    range.setEnd(head.node, head.offset);
 
-      // Update the DOM selection to match ProseMirror's selection
-      domSelection.removeAllRanges();
-      domSelection.addRange(range);
-    } catch {
-      // Silently fail if DOM selection sync fails
-      // This can happen if positions are invalid or DOM hasn't updated yet
-    }
-  } else {
-    // OLD BEHAVIOR (to be removed when flag is cleaned up)
-    // Only checked if selection was out of sync using incorrect offset comparison
-    const domSelection = window.getSelection();
-    const domRange = domSelection && domSelection.rangeCount === 1 && domSelection.getRangeAt(0).cloneRange();
-    const isOutOfSync = domRange && (selection.from !== domRange.startOffset || selection.to !== domRange.endOffset);
-    if (isOutOfSync && domSelection && domRange) {
-      // Force the DOM selection to refresh, setting it to the same range
-      // This will trigger ProseMirror to re-apply its selection logic based on the current state
-      domSelection.removeAllRanges();
-      domSelection.addRange(domRange);
-    }
+    // Update the DOM selection to match ProseMirror's selection
+    domSelection.removeAllRanges();
+    domSelection.addRange(range);
+  } catch {
+    // Silently fail if DOM selection sync fails
+    // This can happen if positions are invalid or DOM hasn't updated yet
   }
 };

package/dist/esm/pm-plugins/selection-preservation/pm-plugin.js

@@ -5,7 +5,6 @@ import { bind } from 'bind-event-listener';
 import { getDocument } from '@atlaskit/browser-apis';
 import { SafePlugin } from '@atlaskit/editor-common/safe-plugin';
 import { DRAG_HANDLE_SELECTOR } from '@atlaskit/editor-common/styles';
-import { fg } from '@atlaskit/platform-feature-flags';
 import { key } from '../main';
 import { createPreservedSelection, mapPreservedSelection } from '../utils/selection';
 import { stopPreservingSelection } from './editor-commands';
@@ -153,42 +152,23 @@ export var createSelectionPreservationPlugin = function createSelectionPreservat
         });
         return {
           update: function update(updateView, prevState) {
+            var _selectionPreservatio, _selectionPreservatio2, _key$getState, _key$getState2;
             view = updateView;
+            var prevPreservedSelection = (_selectionPreservatio = selectionPreservationPluginKey.getState(prevState)) === null || _selectionPreservatio === void 0 ? void 0 : _selectionPreservatio.preservedSelection;
+            var currPreservedSelection = (_selectionPreservatio2 = selectionPreservationPluginKey.getState(view.state)) === null || _selectionPreservatio2 === void 0 ? void 0 : _selectionPreservatio2.preservedSelection;
+            var prevActiveNode = (_key$getState = key.getState(prevState)) === null || _key$getState === void 0 ? void 0 : _key$getState.activeNode;
+            var currActiveNode = (_key$getState2 = key.getState(view.state)) === null || _key$getState2 === void 0 ? void 0 : _key$getState2.activeNode;
 
-            // [FEATURE FLAG: platform_editor_selection_sync_fix]
-            // When enabled, syncs DOM selection even when editor doesn't have focus.
-            // This prevents ghost highlighting after moving nodes when block menu is open.
-            // To clean up: remove the if-else block and keep only the flag-on behavior.
-            if (fg('platform_editor_selection_sync_fix')) {
-              var _selectionPreservatio, _selectionPreservatio2, _key$getState, _key$getState2;
-              var prevPreservedSelection = (_selectionPreservatio = selectionPreservationPluginKey.getState(prevState)) === null || _selectionPreservatio === void 0 ? void 0 : _selectionPreservatio.preservedSelection;
-              var currPreservedSelection = (_selectionPreservatio2 = selectionPreservationPluginKey.getState(view.state)) === null || _selectionPreservatio2 === void 0 ? void 0 : _selectionPreservatio2.preservedSelection;
-              var prevActiveNode = (_key$getState = key.getState(prevState)) === null || _key$getState === void 0 ? void 0 : _key$getState.activeNode;
-              var currActiveNode = (_key$getState2 = key.getState(view.state)) === null || _key$getState2 === void 0 ? void 0 : _key$getState2.activeNode;
-
-              // Sync DOM selection when the preserved selection or active node changes
-              // AND the document has changed (e.g., nodes moved)
-              // This prevents stealing focus during menu navigation while still fixing ghost highlighting
-              var hasPreservedSelection = !!currPreservedSelection;
-              var preservedSelectionChanged = !compareSelections(prevPreservedSelection, currPreservedSelection);
-              var activeNodeChanged = prevActiveNode !== currActiveNode;
-              var docChanged = prevState.doc !== view.state.doc;
-              var shouldSyncDOMSelection = hasPreservedSelection && (preservedSelectionChanged || activeNodeChanged) && docChanged;
-              if (shouldSyncDOMSelection) {
-                syncDOMSelection(view.state.selection, view);
-              }
-            } else {
-              var _selectionPreservatio3, _selectionPreservatio4, _key$getState3, _key$getState4;
-              // OLD BEHAVIOR (to be removed when flag is cleaned up)
-              // Only synced when editor had focus, causing ghost highlighting issues
-              var _prevPreservedSelection = (_selectionPreservatio3 = selectionPreservationPluginKey.getState(prevState)) === null || _selectionPreservatio3 === void 0 ? void 0 : _selectionPreservatio3.preservedSelection;
-              var _currPreservedSelection = (_selectionPreservatio4 = selectionPreservationPluginKey.getState(view.state)) === null || _selectionPreservatio4 === void 0 ? void 0 : _selectionPreservatio4.preservedSelection;
-              var _prevActiveNode = (_key$getState3 = key.getState(prevState)) === null || _key$getState3 === void 0 ? void 0 : _key$getState3.activeNode;
-              var _currActiveNode = (_key$getState4 = key.getState(view.state)) === null || _key$getState4 === void 0 ? void 0 : _key$getState4.activeNode;
-              if (_currPreservedSelection && view.hasFocus() && (!compareSelections(_prevPreservedSelection, _currPreservedSelection) || _prevActiveNode !== _currActiveNode)) {
-                // Old syncDOMSelection signature (to be removed)
-                syncDOMSelection(view.state.selection);
-              }
+            // Sync DOM selection when the preserved selection or active node changes
+            // AND the document has changed (e.g., nodes moved)
+            // This prevents stealing focus during menu navigation while still fixing ghost highlighting
+            var hasPreservedSelection = !!currPreservedSelection;
+            var preservedSelectionChanged = !compareSelections(prevPreservedSelection, currPreservedSelection);
+            var activeNodeChanged = prevActiveNode !== currActiveNode;
+            var docChanged = prevState.doc !== view.state.doc;
+            var shouldSyncDOMSelection = hasPreservedSelection && (preservedSelectionChanged || activeNodeChanged) && docChanged;
+            if (shouldSyncDOMSelection) {
+              syncDOMSelection(view.state.selection, view);
             }
           },
           destroy: function destroy() {

package/dist/esm/pm-plugins/selection-preservation/utils.js

@@ -1,5 +1,4 @@
 import { getDocument } from '@atlaskit/browser-apis';
-import { fg } from '@atlaskit/platform-feature-flags';
 import { selectionPreservationPluginKey } from './plugin-key';
 /**
  * Detects if any of the transactions include user-driven selection changes.
@@ -35,54 +34,36 @@ export var compareSelections = function compareSelections(a, b) {
  * the browser's native selection remains at the old position, causing ghost highlighting.
  *
  * @param selection The current ProseMirror selection state to sync to DOM.
- * @param view The EditorView instance used to convert ProseMirror positions to DOM positions (when feature flag is enabled).
+ * @param view The EditorView instance used to convert ProseMirror positions to DOM positions.
  */
 export var syncDOMSelection = function syncDOMSelection(selection, view) {
-  // [FEATURE FLAG: platform_editor_selection_sync_fix]
-  // When enabled, uses improved DOM selection syncing with EditorView.
-  // To clean up: remove the if-else block, remove the optional view parameter,
-  // make view required, and keep only the flag-on behavior.
-  if (view && fg('platform_editor_selection_sync_fix')) {
-    try {
-      var domSelection = window.getSelection();
-      if (!domSelection) {
-        return;
-      }
-      var doc = getDocument();
-      if (!doc) {
-        return;
-      }
+  try {
+    var domSelection = window.getSelection();
+    if (!domSelection) {
+      return;
+    }
+    var doc = getDocument();
+    if (!doc) {
+      return;
+    }
 
-      // Convert ProseMirror selection to DOM selection using view.domAtPos
-      var anchor = view.domAtPos(selection.anchor);
-      var head = view.domAtPos(selection.head);
-      if (!anchor || !head) {
-        return;
-      }
+    // Convert ProseMirror selection to DOM selection using view.domAtPos
+    var anchor = view.domAtPos(selection.anchor);
+    var head = view.domAtPos(selection.head);
+    if (!anchor || !head) {
+      return;
+    }
 
-      // Create a new DOM range from the ProseMirror selection
-      var range = doc.createRange();
-      range.setStart(anchor.node, anchor.offset);
-      range.setEnd(head.node, head.offset);
+    // Create a new DOM range from the ProseMirror selection
+    var range = doc.createRange();
+    range.setStart(anchor.node, anchor.offset);
+    range.setEnd(head.node, head.offset);
 
-      // Update the DOM selection to match ProseMirror's selection
-      domSelection.removeAllRanges();
-      domSelection.addRange(range);
-    } catch (_unused) {
-      // Silently fail if DOM selection sync fails
-      // This can happen if positions are invalid or DOM hasn't updated yet
-    }
-  } else {
-    // OLD BEHAVIOR (to be removed when flag is cleaned up)
-    // Only checked if selection was out of sync using incorrect offset comparison
-    var _domSelection = window.getSelection();
-    var domRange = _domSelection && _domSelection.rangeCount === 1 && _domSelection.getRangeAt(0).cloneRange();
-    var isOutOfSync = domRange && (selection.from !== domRange.startOffset || selection.to !== domRange.endOffset);
-    if (isOutOfSync && _domSelection && domRange) {
-      // Force the DOM selection to refresh, setting it to the same range
-      // This will trigger ProseMirror to re-apply its selection logic based on the current state
-      _domSelection.removeAllRanges();
-      _domSelection.addRange(domRange);
-    }
+    // Update the DOM selection to match ProseMirror's selection
+    domSelection.removeAllRanges();
+    domSelection.addRange(range);
+  } catch (_unused) {
+    // Silently fail if DOM selection sync fails
+    // This can happen if positions are invalid or DOM hasn't updated yet
   }
 };

package/dist/types/pm-plugins/selection-preservation/utils.d.ts

@@ -25,6 +25,6 @@ export declare const compareSelections: (a?: Selection, b?: Selection) => boolea
  * the browser's native selection remains at the old position, causing ghost highlighting.
  *
  * @param selection The current ProseMirror selection state to sync to DOM.
- * @param view The EditorView instance used to convert ProseMirror positions to DOM positions (when feature flag is enabled).
+ * @param view The EditorView instance used to convert ProseMirror positions to DOM positions.
  */
-export declare const syncDOMSelection: (selection: Selection, view?: EditorView) => void;
+export declare const syncDOMSelection: (selection: Selection, view: EditorView) => void;

package/dist/types-ts4.5/pm-plugins/selection-preservation/utils.d.ts

@@ -25,6 +25,6 @@ export declare const compareSelections: (a?: Selection, b?: Selection) => boolea
  * the browser's native selection remains at the old position, causing ghost highlighting.
  *
  * @param selection The current ProseMirror selection state to sync to DOM.
- * @param view The EditorView instance used to convert ProseMirror positions to DOM positions (when feature flag is enabled).
+ * @param view The EditorView instance used to convert ProseMirror positions to DOM positions.
  */
-export declare const syncDOMSelection: (selection: Selection, view?: EditorView) => void;
+export declare const syncDOMSelection: (selection: Selection, view: EditorView) => void;

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@atlaskit/editor-plugin-block-controls",
-	"version": "8.5.1",
+	"version": "8.6.0",
 	"description": "Block controls plugin for @atlaskit/editor-core",
 	"author": "Atlassian Pty Ltd",
 	"license": "Apache-2.0",
@@ -56,7 +56,7 @@
 		"@atlaskit/pragmatic-drag-and-drop-react-drop-indicator": "^3.2.0",
 		"@atlaskit/primitives": "^18.0.0",
 		"@atlaskit/theme": "^21.0.0",
-		"@atlaskit/tmp-editor-statsig": "^29.5.0",
+		"@atlaskit/tmp-editor-statsig": "^29.6.0",
 		"@atlaskit/tokens": "^11.0.0",
 		"@atlaskit/tooltip": "^20.14.0",
 		"@babel/runtime": "^7.0.0",
@@ -146,9 +146,6 @@
 		},
 		"platform_editor_native_anchor_patch_2": {
 			"type": "boolean"
-		},
-		"platform_editor_selection_sync_fix": {
-			"type": "boolean"
 		}
 	}
 }