@atlaskit/pragmatic-drag-and-drop 1.2.3 → 1.3.1

package/CHANGELOG.md

@@ -1,5 +1,23 @@
 # @atlaskit/pragmatic-drag-and-drop
 
+## 1.3.1
+
+### Patch Changes
+
+- [#141279](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/pull-requests/141279)
+  [`a38f3af4bfc79`](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/commits/a38f3af4bfc79) -
+  Minor refactor of internal helper.
+
+## 1.3.0
+
+### Minor Changes
+
+- [#128458](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/pull-requests/128458)
+  [`71c5224450c8a`](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/commits/71c5224450c8a) -
+  Adding workaround for a [bug in Firefox](https://bugzilla.mozilla.org/show_bug.cgi?id=1912164).
+  The external adpater optional URL utilities `containsURLs` and `getURLs` will now correctly
+  recognize URLs dragged from the Firefox address bar or bookmarks in to a Firefox `window`.
+
 ## 1.2.3
 
 ### Patch Changes

package/dist/cjs/adapter/element-adapter.js

@@ -82,6 +82,40 @@ var adapter = (0, _makeAdapter.makeAdapter)({
         if (!entry) {
           return null;
         }
+
+        /**
+         * A text selection drag _can_ have the `draggable` element be
+         * the `event.target` if the user is dragging the text selection
+         * from the `draggable`.
+         *
+         * To know if the `draggable` is being dragged, we look at whether any
+         * `"text/plain"` data is being dragged. If it is, then a text selection
+         * drag is occurring.
+         *
+         * This behaviour has been validated on:
+         *
+         * - Chrome@128 on Android@14
+         * - Chrome@128 on iOS@17.6.1
+         * - Chrome@128 on Windows@11
+         * - Chrome@128 on MacOS@14.6.1
+         * - Firefox@129 on Windows@11 (not possible for user to select text in a draggable)
+         * - Firefox@129 on MacOS@14.6.1 (not possible for user to select text in a draggable)
+         *
+         * Note: Could usually just use: `event.dataTransfer.types.includes(textMediaType)`
+         * but unfortunately ProseMirror is always setting `""` as the dragged text
+         *
+         * Note: Unfortunately editor is (heavily) leaning on the current functionality today
+         * and unwinding it will be a decent amount of effort. So for now, a text selection
+         * where the `event.target` is a `draggable` element will still trigger the
+         * element adapter.
+         *
+         * // Future state:
+         * if(event.dataTransfer.getData(textMediaType)) {
+         * 	return;
+         * }
+         *
+         */
+
         var input = (0, _getInput.getInput)(event);
         var feedback = {
           element: entry.element,
@@ -153,8 +187,7 @@ var adapter = (0, _makeAdapter.makeAdapter)({
          * Android version: 14 (November 5, 2023)
          * Chrome version: 120.0
          */
-        var types = event.dataTransfer.types;
-        if ((0, _android.isAndroid)() && !types.includes(_textMediaType.textMediaType) && !types.includes(_urlMediaType.urlMediaType)) {
+        if ((0, _android.isAndroid)() && !event.dataTransfer.types.includes(_textMediaType.textMediaType) && !event.dataTransfer.types.includes(_urlMediaType.URLMediaType)) {
           event.dataTransfer.setData(_textMediaType.textMediaType, _android.androidFallbackText);
         }
 

package/dist/cjs/adapter/text-selection-adapter.js

@@ -41,7 +41,7 @@ function findTextNode(event) {
     return null;
   }
 
-  // unlikely that this particular drag is a text selection drag
+  // Unlikely that this particular drag is a text selection drag
   if (event.target.draggable) {
     return null;
   }
@@ -52,7 +52,10 @@ function findTextNode(event) {
     return null;
   }
 
-  // Grab the first Text node and use that
+  /**
+   * Grab the first Text node and use that.
+   * Only doing a single level search as that is all we need for this bug.
+   */
   var text = Array.from(event.target.childNodes).find(function (node) {
     return node.nodeType === Node.TEXT_NODE;
   });
@@ -97,7 +100,14 @@ var adapter = (0, _makeAdapter.makeAdapter)({
         if (!api.canStart(event)) {
           return;
         }
+
+        // no text being dragged
+        if (!event.dataTransfer.types.includes(_textMediaType.textMediaType)) {
+          return;
+        }
         var target = findTextNode(event);
+
+        // could not find `Text` node that is being dragged from
         if (!target) {
           return;
         }
@@ -106,7 +116,6 @@ var adapter = (0, _makeAdapter.makeAdapter)({
           // that the user started the drag from.
           // The full text being dragged can be looked up from the `dataTransfer`.
           target: target,
-          // This is safe to do in "dragstart" as the `dataTransfer` is in read/write mode.
           plain: event.dataTransfer.getData(_textMediaType.textMediaType),
           HTML: event.dataTransfer.getData(_htmlMediaType.HTMLMediaType)
         };

package/dist/cjs/make-adapter/make-drop-target.js

@@ -11,9 +11,9 @@ var _combine = require("../public-utils/combine");
 var _addAttribute = require("../util/add-attribute");
 function ownKeys(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbols) { var o = Object.getOwnPropertySymbols(e); r && (o = o.filter(function (r) { return Object.getOwnPropertyDescriptor(e, r).enumerable; })), t.push.apply(t, o); } return t; }
 function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var t = null != arguments[r] ? arguments[r] : {}; r % 2 ? ownKeys(Object(t), !0).forEach(function (r) { (0, _defineProperty2.default)(e, r, t[r]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(e, Object.getOwnPropertyDescriptors(t)) : ownKeys(Object(t)).forEach(function (r) { Object.defineProperty(e, r, Object.getOwnPropertyDescriptor(t, r)); }); } return e; }
-function _createForOfIteratorHelper(o, allowArrayLike) { var it = typeof Symbol !== "undefined" && o[Symbol.iterator] || o["@@iterator"]; if (!it) { if (Array.isArray(o) || (it = _unsupportedIterableToArray(o)) || allowArrayLike && o && typeof o.length === "number") { if (it) o = it; var i = 0; var F = function F() {}; return { s: F, n: function n() { if (i >= o.length) return { done: true }; return { done: false, value: o[i++] }; }, e: function e(_e) { throw _e; }, f: F }; } throw new TypeError("Invalid attempt to iterate non-iterable instance.\nIn order to be iterable, non-array objects must have a [Symbol.iterator]() method."); } var normalCompletion = true, didErr = false, err; return { s: function s() { it = it.call(o); }, n: function n() { var step = it.next(); normalCompletion = step.done; return step; }, e: function e(_e2) { didErr = true; err = _e2; }, f: function f() { try { if (!normalCompletion && it.return != null) it.return(); } finally { if (didErr) throw err; } } }; }
-function _unsupportedIterableToArray(o, minLen) { if (!o) return; if (typeof o === "string") return _arrayLikeToArray(o, minLen); var n = Object.prototype.toString.call(o).slice(8, -1); if (n === "Object" && o.constructor) n = o.constructor.name; if (n === "Map" || n === "Set") return Array.from(o); if (n === "Arguments" || /^(?:Ui|I)nt(?:8|16|32)(?:Clamped)?Array$/.test(n)) return _arrayLikeToArray(o, minLen); }
-function _arrayLikeToArray(arr, len) { if (len == null || len > arr.length) len = arr.length; for (var i = 0, arr2 = new Array(len); i < len; i++) arr2[i] = arr[i]; return arr2; }
+function _createForOfIteratorHelper(r, e) { var t = "undefined" != typeof Symbol && r[Symbol.iterator] || r["@@iterator"]; if (!t) { if (Array.isArray(r) || (t = _unsupportedIterableToArray(r)) || e && r && "number" == typeof r.length) { t && (r = t); var _n = 0, F = function F() {}; return { s: F, n: function n() { return _n >= r.length ? { done: !0 } : { done: !1, value: r[_n++] }; }, e: function e(r) { throw r; }, f: F }; } throw new TypeError("Invalid attempt to iterate non-iterable instance.\nIn order to be iterable, non-array objects must have a [Symbol.iterator]() method."); } var o, a = !0, u = !1; return { s: function s() { t = t.call(r); }, n: function n() { var r = t.next(); return a = r.done, r; }, e: function e(r) { u = !0, o = r; }, f: function f() { try { a || null == t.return || t.return(); } finally { if (u) throw o; } } }; }
+function _unsupportedIterableToArray(r, a) { if (r) { if ("string" == typeof r) return _arrayLikeToArray(r, a); var t = {}.toString.call(r).slice(8, -1); return "Object" === t && r.constructor && (t = r.constructor.name), "Map" === t || "Set" === t ? Array.from(r) : "Arguments" === t || /^(?:Ui|I)nt(?:8|16|32)(?:Clamped)?Array$/.test(t) ? _arrayLikeToArray(r, a) : void 0; } }
+function _arrayLikeToArray(r, a) { (null == a || a > r.length) && (a = r.length); for (var e = 0, n = Array(a); e < a; e++) n[e] = r[e]; return n; }
 function copyReverse(array) {
   return array.slice(0).reverse();
 }
@@ -137,14 +137,14 @@ function makeDropTarget(_ref) {
         var _entry$eventName;
         var record = _step.value;
         var entry = registry.get(record.element);
-        var _args = _objectSpread(_objectSpread({}, payload), {}, {
+        var args = _objectSpread(_objectSpread({}, payload), {}, {
           self: record
         });
         entry === null || entry === void 0 || (_entry$eventName = entry[eventName]) === null || _entry$eventName === void 0 || _entry$eventName.call(entry,
         // I cannot seem to get the types right here.
         // TS doesn't seem to like that one event can need `nativeSetDragImage`
         // @ts-expect-error
-        _args);
+        args);
       }
     } catch (err) {
       _iterator.e(err);
@@ -172,15 +172,15 @@ function makeDropTarget(_ref) {
           visited.add(record.element);
           var entry = registry.get(record.element);
           var isOver = isCurrent.has(record.element);
-          var _args2 = _objectSpread(_objectSpread({}, payload), {}, {
+          var args = _objectSpread(_objectSpread({}, payload), {}, {
             self: record
           });
-          entry === null || entry === void 0 || (_entry$onDropTargetCh = entry.onDropTargetChange) === null || _entry$onDropTargetCh === void 0 || _entry$onDropTargetCh.call(entry, _args2);
+          entry === null || entry === void 0 || (_entry$onDropTargetCh = entry.onDropTargetChange) === null || _entry$onDropTargetCh === void 0 || _entry$onDropTargetCh.call(entry, args);
 
           // if we cannot find the drop target in the current array, then it has been left
           if (!isOver) {
             var _entry$onDragLeave;
-            entry === null || entry === void 0 || (_entry$onDragLeave = entry.onDragLeave) === null || _entry$onDragLeave === void 0 || _entry$onDragLeave.call(entry, _args2);
+            entry === null || entry === void 0 || (_entry$onDragLeave = entry.onDragLeave) === null || _entry$onDragLeave === void 0 || _entry$onDragLeave.call(entry, args);
           }
         }
       } catch (err) {
@@ -199,12 +199,12 @@ function makeDropTarget(_ref) {
             continue;
           }
           // at this point we have a new drop target that is being entered into
-          var _args3 = _objectSpread(_objectSpread({}, payload), {}, {
+          var _args = _objectSpread(_objectSpread({}, payload), {}, {
             self: _record
           });
           var _entry = registry.get(_record.element);
-          _entry === null || _entry === void 0 || (_entry$onDropTargetCh2 = _entry.onDropTargetChange) === null || _entry$onDropTargetCh2 === void 0 || _entry$onDropTargetCh2.call(_entry, _args3);
-          _entry === null || _entry === void 0 || (_entry$onDragEnter = _entry.onDragEnter) === null || _entry$onDragEnter === void 0 || _entry$onDragEnter.call(_entry, _args3);
+          _entry === null || _entry === void 0 || (_entry$onDropTargetCh2 = _entry.onDropTargetChange) === null || _entry$onDropTargetCh2 === void 0 || _entry$onDropTargetCh2.call(_entry, _args);
+          _entry === null || _entry === void 0 || (_entry$onDragEnter = _entry.onDragEnter) === null || _entry$onDragEnter === void 0 || _entry$onDragEnter.call(_entry, _args);
         }
       } catch (err) {
         _iterator3.e(err);

package/dist/cjs/make-adapter/make-monitor.js

@@ -6,9 +6,9 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.makeMonitor = makeMonitor;
 var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/defineProperty"));
-function _createForOfIteratorHelper(o, allowArrayLike) { var it = typeof Symbol !== "undefined" && o[Symbol.iterator] || o["@@iterator"]; if (!it) { if (Array.isArray(o) || (it = _unsupportedIterableToArray(o)) || allowArrayLike && o && typeof o.length === "number") { if (it) o = it; var i = 0; var F = function F() {}; return { s: F, n: function n() { if (i >= o.length) return { done: true }; return { done: false, value: o[i++] }; }, e: function e(_e) { throw _e; }, f: F }; } throw new TypeError("Invalid attempt to iterate non-iterable instance.\nIn order to be iterable, non-array objects must have a [Symbol.iterator]() method."); } var normalCompletion = true, didErr = false, err; return { s: function s() { it = it.call(o); }, n: function n() { var step = it.next(); normalCompletion = step.done; return step; }, e: function e(_e2) { didErr = true; err = _e2; }, f: function f() { try { if (!normalCompletion && it.return != null) it.return(); } finally { if (didErr) throw err; } } }; }
-function _unsupportedIterableToArray(o, minLen) { if (!o) return; if (typeof o === "string") return _arrayLikeToArray(o, minLen); var n = Object.prototype.toString.call(o).slice(8, -1); if (n === "Object" && o.constructor) n = o.constructor.name; if (n === "Map" || n === "Set") return Array.from(o); if (n === "Arguments" || /^(?:Ui|I)nt(?:8|16|32)(?:Clamped)?Array$/.test(n)) return _arrayLikeToArray(o, minLen); }
-function _arrayLikeToArray(arr, len) { if (len == null || len > arr.length) len = arr.length; for (var i = 0, arr2 = new Array(len); i < len; i++) arr2[i] = arr[i]; return arr2; }
+function _createForOfIteratorHelper(r, e) { var t = "undefined" != typeof Symbol && r[Symbol.iterator] || r["@@iterator"]; if (!t) { if (Array.isArray(r) || (t = _unsupportedIterableToArray(r)) || e && r && "number" == typeof r.length) { t && (r = t); var _n = 0, F = function F() {}; return { s: F, n: function n() { return _n >= r.length ? { done: !0 } : { done: !1, value: r[_n++] }; }, e: function e(r) { throw r; }, f: F }; } throw new TypeError("Invalid attempt to iterate non-iterable instance.\nIn order to be iterable, non-array objects must have a [Symbol.iterator]() method."); } var o, a = !0, u = !1; return { s: function s() { t = t.call(r); }, n: function n() { var r = t.next(); return a = r.done, r; }, e: function e(r) { u = !0, o = r; }, f: function f() { try { a || null == t.return || t.return(); } finally { if (u) throw o; } } }; }
+function _unsupportedIterableToArray(r, a) { if (r) { if ("string" == typeof r) return _arrayLikeToArray(r, a); var t = {}.toString.call(r).slice(8, -1); return "Object" === t && r.constructor && (t = r.constructor.name), "Map" === t || "Set" === t ? Array.from(r) : "Arguments" === t || /^(?:Ui|I)nt(?:8|16|32)(?:Clamped)?Array$/.test(t) ? _arrayLikeToArray(r, a) : void 0; } }
+function _arrayLikeToArray(r, a) { (null == a || a > r.length) && (a = r.length); for (var e = 0, n = Array(a); e < a; e++) n[e] = r[e]; return n; }
 function ownKeys(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbols) { var o = Object.getOwnPropertySymbols(e); r && (o = o.filter(function (r) { return Object.getOwnPropertyDescriptor(e, r).enumerable; })), t.push.apply(t, o); } return t; }
 function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var t = null != arguments[r] ? arguments[r] : {}; r % 2 ? ownKeys(Object(t), !0).forEach(function (r) { (0, _defineProperty2.default)(e, r, t[r]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(e, Object.getOwnPropertyDescriptors(t)) : ownKeys(Object(t)).forEach(function (r) { Object.defineProperty(e, r, Object.getOwnPropertyDescriptor(t, r)); }); } return e; }
 function makeMonitor() {

package/dist/cjs/public-utils/external/url.js

@@ -6,28 +6,50 @@ Object.defineProperty(exports, "__esModule", {
 exports.containsURLs = containsURLs;
 exports.getURLs = getURLs;
 var _urlMediaType = require("../../util/media-types/url-media-type");
+/**
+ * 🦊🐞
+ * When dragging a URL from the Firefox address bar or bookmarks
+ * they are currently not adding an entry for "text/uri-list".
+ * They add "text/x-moz-url" data which contains the same information
+ * in a different format.
+ *
+ * [Bug report](https://bugzilla.mozilla.org/show_bug.cgi?id=1912164)
+ */
+var firefoxURLType = 'text/x-moz-url';
 function containsURLs(_ref) {
   var source = _ref.source;
-  return source.types.includes(_urlMediaType.urlMediaType);
+  return source.types.includes(_urlMediaType.URLMediaType) || source.types.includes(firefoxURLType);
 }
 function getURLs(_ref2) {
   var source = _ref2.source;
-  var value = source.getStringData(_urlMediaType.urlMediaType);
-
-  // no values found
-  if (value == null) {
-    return [];
+  var standard = source.getStringData(_urlMediaType.URLMediaType);
+  if (standard != null) {
+    return standard
+    // You can have multiple urls split by CR+LF (EOL)
+    // - CR: Carriage Return '\r'
+    // - LF: Line Feed '\n'
+    // - EOL: End of Line '\r\n'
+    .split('\r\n')
+    // a uri-list can have comment lines starting with '#'
+    // so we need to remove those
+    .filter(function (piece) {
+      return !piece.startsWith('#');
+    });
+  }
+  var fallback = source.getStringData(firefoxURLType);
+  if (fallback != null) {
+    return fallback
+    // Values are split by a single LF: Line Feed (`\n`) character.
+    // It's not clear from the "text/x-moz-url" documentation that
+    // it's use `\n` and not `\r\n`, but based on testing and some
+    // Github code searches it seems like `\n` is correct.
+    // https://developer.mozilla.org/en-US/docs/Web/API/HTML_Drag_and_Drop_API/Recommended_drag_types#dragging_links
+    .split('\n')
+    // Every second line is the title of the url previous url.
+    // We are ignoring the page titles in this helper
+    .filter(function (_, index) {
+      return index % 2 === 0;
+    });
   }
-  var urls = value
-  // You can have multiple urls split by CR+LF (EOL)
-  // - CR: Carriage Return '\r'
-  // - LF: Line Feed '\n'
-  // - EOL: End of Line '\r\n'
-  .split('\r\n')
-  // a uri-list can have comment lines starting with '#'
-  // so we need to remove those
-  .filter(function (piece) {
-    return !piece.startsWith('#');
-  });
-  return urls;
+  return [];
 }

package/dist/cjs/util/media-types/url-media-type.js

@@ -3,10 +3,10 @@
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
-exports.urlMediaType = void 0;
+exports.URLMediaType = void 0;
 // Why we put the media types in their own files:
 //
 // - we are not putting them all in one file as not all adapters need all types
 // - we are not putting them in the external helpers as some things just need the
 //   types and not the external functions code
-var urlMediaType = exports.urlMediaType = 'text/uri-list';
+var URLMediaType = exports.URLMediaType = 'text/uri-list';

package/dist/es2019/adapter/element-adapter.js

@@ -7,7 +7,7 @@ import { addAttribute } from '../util/add-attribute';
 import { androidFallbackText, isAndroid } from '../util/android';
 import { getInput } from '../util/get-input';
 import { textMediaType } from '../util/media-types/text-media-type';
-import { urlMediaType } from '../util/media-types/url-media-type';
+import { URLMediaType } from '../util/media-types/url-media-type';
 import { elementAdapterNativeDataKey } from './element-adapter-native-data-key';
 const draggableRegistry = new WeakMap();
 function addToRegistry(args) {
@@ -81,6 +81,40 @@ const adapter = makeAdapter({
         if (!entry) {
           return null;
         }
+
+        /**
+         * A text selection drag _can_ have the `draggable` element be
+         * the `event.target` if the user is dragging the text selection
+         * from the `draggable`.
+         *
+         * To know if the `draggable` is being dragged, we look at whether any
+         * `"text/plain"` data is being dragged. If it is, then a text selection
+         * drag is occurring.
+         *
+         * This behaviour has been validated on:
+         *
+         * - Chrome@128 on Android@14
+         * - Chrome@128 on iOS@17.6.1
+         * - Chrome@128 on Windows@11
+         * - Chrome@128 on MacOS@14.6.1
+         * - Firefox@129 on Windows@11 (not possible for user to select text in a draggable)
+         * - Firefox@129 on MacOS@14.6.1 (not possible for user to select text in a draggable)
+         *
+         * Note: Could usually just use: `event.dataTransfer.types.includes(textMediaType)`
+         * but unfortunately ProseMirror is always setting `""` as the dragged text
+         *
+         * Note: Unfortunately editor is (heavily) leaning on the current functionality today
+         * and unwinding it will be a decent amount of effort. So for now, a text selection
+         * where the `event.target` is a `draggable` element will still trigger the
+         * element adapter.
+         *
+         * // Future state:
+         * if(event.dataTransfer.getData(textMediaType)) {
+         * 	return;
+         * }
+         *
+         */
+
         const input = getInput(event);
         const feedback = {
           element: entry.element,
@@ -149,10 +183,7 @@ const adapter = makeAdapter({
          * Android version: 14 (November 5, 2023)
          * Chrome version: 120.0
          */
-        const {
-          types
-        } = event.dataTransfer;
-        if (isAndroid() && !types.includes(textMediaType) && !types.includes(urlMediaType)) {
+        if (isAndroid() && !event.dataTransfer.types.includes(textMediaType) && !event.dataTransfer.types.includes(URLMediaType)) {
           event.dataTransfer.setData(textMediaType, androidFallbackText);
         }
 

package/dist/es2019/adapter/text-selection-adapter.js

@@ -34,7 +34,7 @@ function findTextNode(event) {
     return null;
   }
 
-  // unlikely that this particular drag is a text selection drag
+  // Unlikely that this particular drag is a text selection drag
   if (event.target.draggable) {
     return null;
   }
@@ -45,7 +45,10 @@ function findTextNode(event) {
     return null;
   }
 
-  // Grab the first Text node and use that
+  /**
+   * Grab the first Text node and use that.
+   * Only doing a single level search as that is all we need for this bug.
+   */
   const text = Array.from(event.target.childNodes).find(node => node.nodeType === Node.TEXT_NODE);
   return text !== null && text !== void 0 ? text : null;
 }
@@ -96,7 +99,14 @@ const adapter = makeAdapter({
         if (!api.canStart(event)) {
           return;
         }
+
+        // no text being dragged
+        if (!event.dataTransfer.types.includes(textMediaType)) {
+          return;
+        }
         const target = findTextNode(event);
+
+        // could not find `Text` node that is being dragged from
         if (!target) {
           return;
         }
@@ -105,7 +115,6 @@ const adapter = makeAdapter({
           // that the user started the drag from.
           // The full text being dragged can be looked up from the `dataTransfer`.
           target,
-          // This is safe to do in "dragstart" as the `dataTransfer` is in read/write mode.
           plain: event.dataTransfer.getData(textMediaType),
           HTML: event.dataTransfer.getData(HTMLMediaType)
         };

package/dist/es2019/public-utils/external/url.js

@@ -1,26 +1,46 @@
-import { urlMediaType } from '../../util/media-types/url-media-type';
+import { URLMediaType } from '../../util/media-types/url-media-type';
+/**
+ * 🦊🐞
+ * When dragging a URL from the Firefox address bar or bookmarks
+ * they are currently not adding an entry for "text/uri-list".
+ * They add "text/x-moz-url" data which contains the same information
+ * in a different format.
+ *
+ * [Bug report](https://bugzilla.mozilla.org/show_bug.cgi?id=1912164)
+ */
+const firefoxURLType = 'text/x-moz-url';
 export function containsURLs({
   source
 }) {
-  return source.types.includes(urlMediaType);
+  return source.types.includes(URLMediaType) || source.types.includes(firefoxURLType);
 }
 export function getURLs({
   source
 }) {
-  const value = source.getStringData(urlMediaType);
-
-  // no values found
-  if (value == null) {
-    return [];
+  const standard = source.getStringData(URLMediaType);
+  if (standard != null) {
+    return standard
+    // You can have multiple urls split by CR+LF (EOL)
+    // - CR: Carriage Return '\r'
+    // - LF: Line Feed '\n'
+    // - EOL: End of Line '\r\n'
+    .split('\r\n')
+    // a uri-list can have comment lines starting with '#'
+    // so we need to remove those
+    .filter(piece => !piece.startsWith('#'));
   }
-  const urls = value
-  // You can have multiple urls split by CR+LF (EOL)
-  // - CR: Carriage Return '\r'
-  // - LF: Line Feed '\n'
-  // - EOL: End of Line '\r\n'
-  .split('\r\n')
-  // a uri-list can have comment lines starting with '#'
-  // so we need to remove those
-  .filter(piece => !piece.startsWith('#'));
-  return urls;
+  const fallback = source.getStringData(firefoxURLType);
+  if (fallback != null) {
+    return fallback
+    // Values are split by a single LF: Line Feed (`\n`) character.
+    // It's not clear from the "text/x-moz-url" documentation that
+    // it's use `\n` and not `\r\n`, but based on testing and some
+    // Github code searches it seems like `\n` is correct.
+    // https://developer.mozilla.org/en-US/docs/Web/API/HTML_Drag_and_Drop_API/Recommended_drag_types#dragging_links
+    .split('\n')
+    // Every second line is the title of the url previous url.
+    // We are ignoring the page titles in this helper
+    .filter((_, index) => index % 2 === 0);
+  }
+  return [];
 }

package/dist/es2019/util/media-types/url-media-type.js

@@ -3,4 +3,4 @@
 // - we are not putting them all in one file as not all adapters need all types
 // - we are not putting them in the external helpers as some things just need the
 //   types and not the external functions code
-export const urlMediaType = 'text/uri-list';
+export const URLMediaType = 'text/uri-list';

package/dist/esm/adapter/element-adapter.js

@@ -8,7 +8,7 @@ import { addAttribute } from '../util/add-attribute';
 import { androidFallbackText, isAndroid } from '../util/android';
 import { getInput } from '../util/get-input';
 import { textMediaType } from '../util/media-types/text-media-type';
-import { urlMediaType } from '../util/media-types/url-media-type';
+import { URLMediaType } from '../util/media-types/url-media-type';
 import { elementAdapterNativeDataKey } from './element-adapter-native-data-key';
 var draggableRegistry = new WeakMap();
 function addToRegistry(args) {
@@ -74,6 +74,40 @@ var adapter = makeAdapter({
         if (!entry) {
           return null;
         }
+
+        /**
+         * A text selection drag _can_ have the `draggable` element be
+         * the `event.target` if the user is dragging the text selection
+         * from the `draggable`.
+         *
+         * To know if the `draggable` is being dragged, we look at whether any
+         * `"text/plain"` data is being dragged. If it is, then a text selection
+         * drag is occurring.
+         *
+         * This behaviour has been validated on:
+         *
+         * - Chrome@128 on Android@14
+         * - Chrome@128 on iOS@17.6.1
+         * - Chrome@128 on Windows@11
+         * - Chrome@128 on MacOS@14.6.1
+         * - Firefox@129 on Windows@11 (not possible for user to select text in a draggable)
+         * - Firefox@129 on MacOS@14.6.1 (not possible for user to select text in a draggable)
+         *
+         * Note: Could usually just use: `event.dataTransfer.types.includes(textMediaType)`
+         * but unfortunately ProseMirror is always setting `""` as the dragged text
+         *
+         * Note: Unfortunately editor is (heavily) leaning on the current functionality today
+         * and unwinding it will be a decent amount of effort. So for now, a text selection
+         * where the `event.target` is a `draggable` element will still trigger the
+         * element adapter.
+         *
+         * // Future state:
+         * if(event.dataTransfer.getData(textMediaType)) {
+         * 	return;
+         * }
+         *
+         */
+
         var input = getInput(event);
         var feedback = {
           element: entry.element,
@@ -145,8 +179,7 @@ var adapter = makeAdapter({
          * Android version: 14 (November 5, 2023)
          * Chrome version: 120.0
          */
-        var types = event.dataTransfer.types;
-        if (isAndroid() && !types.includes(textMediaType) && !types.includes(urlMediaType)) {
+        if (isAndroid() && !event.dataTransfer.types.includes(textMediaType) && !event.dataTransfer.types.includes(URLMediaType)) {
           event.dataTransfer.setData(textMediaType, androidFallbackText);
         }
 

package/dist/esm/adapter/text-selection-adapter.js

@@ -34,7 +34,7 @@ function findTextNode(event) {
     return null;
   }
 
-  // unlikely that this particular drag is a text selection drag
+  // Unlikely that this particular drag is a text selection drag
   if (event.target.draggable) {
     return null;
   }
@@ -45,7 +45,10 @@ function findTextNode(event) {
     return null;
   }
 
-  // Grab the first Text node and use that
+  /**
+   * Grab the first Text node and use that.
+   * Only doing a single level search as that is all we need for this bug.
+   */
   var text = Array.from(event.target.childNodes).find(function (node) {
     return node.nodeType === Node.TEXT_NODE;
   });
@@ -90,7 +93,14 @@ var adapter = makeAdapter({
         if (!api.canStart(event)) {
           return;
         }
+
+        // no text being dragged
+        if (!event.dataTransfer.types.includes(textMediaType)) {
+          return;
+        }
         var target = findTextNode(event);
+
+        // could not find `Text` node that is being dragged from
         if (!target) {
           return;
         }
@@ -99,7 +109,6 @@ var adapter = makeAdapter({
           // that the user started the drag from.
           // The full text being dragged can be looked up from the `dataTransfer`.
           target: target,
-          // This is safe to do in "dragstart" as the `dataTransfer` is in read/write mode.
           plain: event.dataTransfer.getData(textMediaType),
           HTML: event.dataTransfer.getData(HTMLMediaType)
         };

package/dist/esm/make-adapter/make-drop-target.js

@@ -2,9 +2,9 @@ import _defineProperty from "@babel/runtime/helpers/defineProperty";
 import _toConsumableArray from "@babel/runtime/helpers/toConsumableArray";
 function ownKeys(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbols) { var o = Object.getOwnPropertySymbols(e); r && (o = o.filter(function (r) { return Object.getOwnPropertyDescriptor(e, r).enumerable; })), t.push.apply(t, o); } return t; }
 function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var t = null != arguments[r] ? arguments[r] : {}; r % 2 ? ownKeys(Object(t), !0).forEach(function (r) { _defineProperty(e, r, t[r]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(e, Object.getOwnPropertyDescriptors(t)) : ownKeys(Object(t)).forEach(function (r) { Object.defineProperty(e, r, Object.getOwnPropertyDescriptor(t, r)); }); } return e; }
-function _createForOfIteratorHelper(o, allowArrayLike) { var it = typeof Symbol !== "undefined" && o[Symbol.iterator] || o["@@iterator"]; if (!it) { if (Array.isArray(o) || (it = _unsupportedIterableToArray(o)) || allowArrayLike && o && typeof o.length === "number") { if (it) o = it; var i = 0; var F = function F() {}; return { s: F, n: function n() { if (i >= o.length) return { done: true }; return { done: false, value: o[i++] }; }, e: function e(_e) { throw _e; }, f: F }; } throw new TypeError("Invalid attempt to iterate non-iterable instance.\nIn order to be iterable, non-array objects must have a [Symbol.iterator]() method."); } var normalCompletion = true, didErr = false, err; return { s: function s() { it = it.call(o); }, n: function n() { var step = it.next(); normalCompletion = step.done; return step; }, e: function e(_e2) { didErr = true; err = _e2; }, f: function f() { try { if (!normalCompletion && it.return != null) it.return(); } finally { if (didErr) throw err; } } }; }
-function _unsupportedIterableToArray(o, minLen) { if (!o) return; if (typeof o === "string") return _arrayLikeToArray(o, minLen); var n = Object.prototype.toString.call(o).slice(8, -1); if (n === "Object" && o.constructor) n = o.constructor.name; if (n === "Map" || n === "Set") return Array.from(o); if (n === "Arguments" || /^(?:Ui|I)nt(?:8|16|32)(?:Clamped)?Array$/.test(n)) return _arrayLikeToArray(o, minLen); }
-function _arrayLikeToArray(arr, len) { if (len == null || len > arr.length) len = arr.length; for (var i = 0, arr2 = new Array(len); i < len; i++) arr2[i] = arr[i]; return arr2; }
+function _createForOfIteratorHelper(r, e) { var t = "undefined" != typeof Symbol && r[Symbol.iterator] || r["@@iterator"]; if (!t) { if (Array.isArray(r) || (t = _unsupportedIterableToArray(r)) || e && r && "number" == typeof r.length) { t && (r = t); var _n = 0, F = function F() {}; return { s: F, n: function n() { return _n >= r.length ? { done: !0 } : { done: !1, value: r[_n++] }; }, e: function e(r) { throw r; }, f: F }; } throw new TypeError("Invalid attempt to iterate non-iterable instance.\nIn order to be iterable, non-array objects must have a [Symbol.iterator]() method."); } var o, a = !0, u = !1; return { s: function s() { t = t.call(r); }, n: function n() { var r = t.next(); return a = r.done, r; }, e: function e(r) { u = !0, o = r; }, f: function f() { try { a || null == t.return || t.return(); } finally { if (u) throw o; } } }; }
+function _unsupportedIterableToArray(r, a) { if (r) { if ("string" == typeof r) return _arrayLikeToArray(r, a); var t = {}.toString.call(r).slice(8, -1); return "Object" === t && r.constructor && (t = r.constructor.name), "Map" === t || "Set" === t ? Array.from(r) : "Arguments" === t || /^(?:Ui|I)nt(?:8|16|32)(?:Clamped)?Array$/.test(t) ? _arrayLikeToArray(r, a) : void 0; } }
+function _arrayLikeToArray(r, a) { (null == a || a > r.length) && (a = r.length); for (var e = 0, n = Array(a); e < a; e++) n[e] = r[e]; return n; }
 import { combine } from '../public-utils/combine';
 import { addAttribute } from '../util/add-attribute';
 function copyReverse(array) {
@@ -130,14 +130,14 @@ export function makeDropTarget(_ref) {
         var _entry$eventName;
         var record = _step.value;
         var entry = registry.get(record.element);
-        var _args = _objectSpread(_objectSpread({}, payload), {}, {
+        var args = _objectSpread(_objectSpread({}, payload), {}, {
           self: record
         });
         entry === null || entry === void 0 || (_entry$eventName = entry[eventName]) === null || _entry$eventName === void 0 || _entry$eventName.call(entry,
         // I cannot seem to get the types right here.
         // TS doesn't seem to like that one event can need `nativeSetDragImage`
         // @ts-expect-error
-        _args);
+        args);
       }
     } catch (err) {
       _iterator.e(err);
@@ -165,15 +165,15 @@ export function makeDropTarget(_ref) {
           visited.add(record.element);
           var entry = registry.get(record.element);
           var isOver = isCurrent.has(record.element);
-          var _args2 = _objectSpread(_objectSpread({}, payload), {}, {
+          var args = _objectSpread(_objectSpread({}, payload), {}, {
             self: record
           });
-          entry === null || entry === void 0 || (_entry$onDropTargetCh = entry.onDropTargetChange) === null || _entry$onDropTargetCh === void 0 || _entry$onDropTargetCh.call(entry, _args2);
+          entry === null || entry === void 0 || (_entry$onDropTargetCh = entry.onDropTargetChange) === null || _entry$onDropTargetCh === void 0 || _entry$onDropTargetCh.call(entry, args);
 
           // if we cannot find the drop target in the current array, then it has been left
           if (!isOver) {
             var _entry$onDragLeave;
-            entry === null || entry === void 0 || (_entry$onDragLeave = entry.onDragLeave) === null || _entry$onDragLeave === void 0 || _entry$onDragLeave.call(entry, _args2);
+            entry === null || entry === void 0 || (_entry$onDragLeave = entry.onDragLeave) === null || _entry$onDragLeave === void 0 || _entry$onDragLeave.call(entry, args);
           }
         }
       } catch (err) {
@@ -192,12 +192,12 @@ export function makeDropTarget(_ref) {
             continue;
           }
           // at this point we have a new drop target that is being entered into
-          var _args3 = _objectSpread(_objectSpread({}, payload), {}, {
+          var _args = _objectSpread(_objectSpread({}, payload), {}, {
             self: _record
           });
           var _entry = registry.get(_record.element);
-          _entry === null || _entry === void 0 || (_entry$onDropTargetCh2 = _entry.onDropTargetChange) === null || _entry$onDropTargetCh2 === void 0 || _entry$onDropTargetCh2.call(_entry, _args3);
-          _entry === null || _entry === void 0 || (_entry$onDragEnter = _entry.onDragEnter) === null || _entry$onDragEnter === void 0 || _entry$onDragEnter.call(_entry, _args3);
+          _entry === null || _entry === void 0 || (_entry$onDropTargetCh2 = _entry.onDropTargetChange) === null || _entry$onDropTargetCh2 === void 0 || _entry$onDropTargetCh2.call(_entry, _args);
+          _entry === null || _entry === void 0 || (_entry$onDragEnter = _entry.onDragEnter) === null || _entry$onDragEnter === void 0 || _entry$onDragEnter.call(_entry, _args);
         }
       } catch (err) {
         _iterator3.e(err);

package/dist/esm/make-adapter/make-monitor.js

@@ -1,7 +1,7 @@
 import _defineProperty from "@babel/runtime/helpers/defineProperty";
-function _createForOfIteratorHelper(o, allowArrayLike) { var it = typeof Symbol !== "undefined" && o[Symbol.iterator] || o["@@iterator"]; if (!it) { if (Array.isArray(o) || (it = _unsupportedIterableToArray(o)) || allowArrayLike && o && typeof o.length === "number") { if (it) o = it; var i = 0; var F = function F() {}; return { s: F, n: function n() { if (i >= o.length) return { done: true }; return { done: false, value: o[i++] }; }, e: function e(_e) { throw _e; }, f: F }; } throw new TypeError("Invalid attempt to iterate non-iterable instance.\nIn order to be iterable, non-array objects must have a [Symbol.iterator]() method."); } var normalCompletion = true, didErr = false, err; return { s: function s() { it = it.call(o); }, n: function n() { var step = it.next(); normalCompletion = step.done; return step; }, e: function e(_e2) { didErr = true; err = _e2; }, f: function f() { try { if (!normalCompletion && it.return != null) it.return(); } finally { if (didErr) throw err; } } }; }
-function _unsupportedIterableToArray(o, minLen) { if (!o) return; if (typeof o === "string") return _arrayLikeToArray(o, minLen); var n = Object.prototype.toString.call(o).slice(8, -1); if (n === "Object" && o.constructor) n = o.constructor.name; if (n === "Map" || n === "Set") return Array.from(o); if (n === "Arguments" || /^(?:Ui|I)nt(?:8|16|32)(?:Clamped)?Array$/.test(n)) return _arrayLikeToArray(o, minLen); }
-function _arrayLikeToArray(arr, len) { if (len == null || len > arr.length) len = arr.length; for (var i = 0, arr2 = new Array(len); i < len; i++) arr2[i] = arr[i]; return arr2; }
+function _createForOfIteratorHelper(r, e) { var t = "undefined" != typeof Symbol && r[Symbol.iterator] || r["@@iterator"]; if (!t) { if (Array.isArray(r) || (t = _unsupportedIterableToArray(r)) || e && r && "number" == typeof r.length) { t && (r = t); var _n = 0, F = function F() {}; return { s: F, n: function n() { return _n >= r.length ? { done: !0 } : { done: !1, value: r[_n++] }; }, e: function e(r) { throw r; }, f: F }; } throw new TypeError("Invalid attempt to iterate non-iterable instance.\nIn order to be iterable, non-array objects must have a [Symbol.iterator]() method."); } var o, a = !0, u = !1; return { s: function s() { t = t.call(r); }, n: function n() { var r = t.next(); return a = r.done, r; }, e: function e(r) { u = !0, o = r; }, f: function f() { try { a || null == t.return || t.return(); } finally { if (u) throw o; } } }; }
+function _unsupportedIterableToArray(r, a) { if (r) { if ("string" == typeof r) return _arrayLikeToArray(r, a); var t = {}.toString.call(r).slice(8, -1); return "Object" === t && r.constructor && (t = r.constructor.name), "Map" === t || "Set" === t ? Array.from(r) : "Arguments" === t || /^(?:Ui|I)nt(?:8|16|32)(?:Clamped)?Array$/.test(t) ? _arrayLikeToArray(r, a) : void 0; } }
+function _arrayLikeToArray(r, a) { (null == a || a > r.length) && (a = r.length); for (var e = 0, n = Array(a); e < a; e++) n[e] = r[e]; return n; }
 function ownKeys(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbols) { var o = Object.getOwnPropertySymbols(e); r && (o = o.filter(function (r) { return Object.getOwnPropertyDescriptor(e, r).enumerable; })), t.push.apply(t, o); } return t; }
 function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var t = null != arguments[r] ? arguments[r] : {}; r % 2 ? ownKeys(Object(t), !0).forEach(function (r) { _defineProperty(e, r, t[r]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(e, Object.getOwnPropertyDescriptors(t)) : ownKeys(Object(t)).forEach(function (r) { Object.defineProperty(e, r, Object.getOwnPropertyDescriptor(t, r)); }); } return e; }
 export function makeMonitor() {

package/dist/esm/public-utils/external/url.js

@@ -1,26 +1,48 @@
-import { urlMediaType } from '../../util/media-types/url-media-type';
+import { URLMediaType } from '../../util/media-types/url-media-type';
+/**
+ * 🦊🐞
+ * When dragging a URL from the Firefox address bar or bookmarks
+ * they are currently not adding an entry for "text/uri-list".
+ * They add "text/x-moz-url" data which contains the same information
+ * in a different format.
+ *
+ * [Bug report](https://bugzilla.mozilla.org/show_bug.cgi?id=1912164)
+ */
+var firefoxURLType = 'text/x-moz-url';
 export function containsURLs(_ref) {
   var source = _ref.source;
-  return source.types.includes(urlMediaType);
+  return source.types.includes(URLMediaType) || source.types.includes(firefoxURLType);
 }
 export function getURLs(_ref2) {
   var source = _ref2.source;
-  var value = source.getStringData(urlMediaType);
-
-  // no values found
-  if (value == null) {
-    return [];
+  var standard = source.getStringData(URLMediaType);
+  if (standard != null) {
+    return standard
+    // You can have multiple urls split by CR+LF (EOL)
+    // - CR: Carriage Return '\r'
+    // - LF: Line Feed '\n'
+    // - EOL: End of Line '\r\n'
+    .split('\r\n')
+    // a uri-list can have comment lines starting with '#'
+    // so we need to remove those
+    .filter(function (piece) {
+      return !piece.startsWith('#');
+    });
   }
-  var urls = value
-  // You can have multiple urls split by CR+LF (EOL)
-  // - CR: Carriage Return '\r'
-  // - LF: Line Feed '\n'
-  // - EOL: End of Line '\r\n'
-  .split('\r\n')
-  // a uri-list can have comment lines starting with '#'
-  // so we need to remove those
-  .filter(function (piece) {
-    return !piece.startsWith('#');
-  });
-  return urls;
+  var fallback = source.getStringData(firefoxURLType);
+  if (fallback != null) {
+    return fallback
+    // Values are split by a single LF: Line Feed (`\n`) character.
+    // It's not clear from the "text/x-moz-url" documentation that
+    // it's use `\n` and not `\r\n`, but based on testing and some
+    // Github code searches it seems like `\n` is correct.
+    // https://developer.mozilla.org/en-US/docs/Web/API/HTML_Drag_and_Drop_API/Recommended_drag_types#dragging_links
+    .split('\n')
+    // Every second line is the title of the url previous url.
+    // We are ignoring the page titles in this helper
+    .filter(function (_, index) {
+      return index % 2 === 0;
+    });
+  }
+  return [];
 }

package/dist/esm/util/media-types/url-media-type.js

@@ -3,4 +3,4 @@
 // - we are not putting them all in one file as not all adapters need all types
 // - we are not putting them in the external helpers as some things just need the
 //   types and not the external functions code
-export var urlMediaType = 'text/uri-list';
+export var URLMediaType = 'text/uri-list';

package/dist/types/util/media-types/url-media-type.d.ts

@@ -1 +1 @@
-export declare const urlMediaType = "text/uri-list";
+export declare const URLMediaType = "text/uri-list";

package/dist/types-ts4.5/util/media-types/url-media-type.d.ts

@@ -1 +1 @@
-export declare const urlMediaType = "text/uri-list";
+export declare const URLMediaType = "text/uri-list";

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@atlaskit/pragmatic-drag-and-drop",
-	"version": "1.2.3",
+	"version": "1.3.1",
 	"description": "The core package for Pragmatic drag and drop - enabling fast drag and drop for any experience on any tech stack",
 	"repository": "https://github.com/atlassian/pragmatic-drag-and-drop",
 	"author": "Atlassian Pty Ltd",
@@ -24,8 +24,6 @@
 	"atlaskit:src": "src/index.ts",
 	"atlassian": {
 		"team": "Design System Team",
-		"inPublicMirror": true,
-		"releaseModel": "continuous",
 		"website": {
 			"name": "Core",
 			"category": "Libraries"
@@ -72,6 +70,7 @@
 		"@atlaskit/visual-regression": "*",
 		"@emotion/react": "^11.7.1",
 		"@testing-library/dom": "^10.1.0",
+		"@testing-library/react": "^12.1.5",
 		"@types/raf-schd": "^4.0.1",
 		"globby": "^6.1.0",
 		"react": "^16.8.0",