@ckeditor/ckeditor5-ui 43.1.1 → 43.2.0-alpha.0

package/dist/index.js

@@ -2,7 +2,7 @@
  * @license Copyright (c) 2003-2024, CKSource Holding sp. z o.o. All rights reserved.
  * For licensing, see LICENSE.md or https://ckeditor.com/legal/ckeditor-oss-license
  */
-import { Collection, CKEditorError, EmitterMixin, isNode, toArray, DomEmitterMixin, ObservableMixin, isIterable, uid, env, delay, getEnvKeystrokeText, isVisible, global, KeystrokeHandler, FocusTracker, Rect, toUnit, createElement, ResizeObserver, getBorderWidths, logWarning, getOptimalPosition, isText, isRange, priorities, first, verifyLicense } from '@ckeditor/ckeditor5-utils/dist/index.js';
+import { Collection, CKEditorError, EmitterMixin, isNode, toArray, DomEmitterMixin, ObservableMixin, isIterable, uid, env, delay, getEnvKeystrokeText, isVisible, global, KeystrokeHandler, FocusTracker, Rect, toUnit, createElement, ResizeObserver, getBorderWidths, logWarning, getOptimalPosition, isText, isRange, priorities, first, verifyLicense, getAncestors } from '@ckeditor/ckeditor5-utils/dist/index.js';
 import { cloneDeepWith, isObject, isElement, debounce, throttle, cloneDeep, extend, escapeRegExp, escape } from 'lodash-es';
 import { icons, Plugin, ContextPlugin } from '@ckeditor/ckeditor5-core/dist/index.js';
 import parse from 'color-parse';
@@ -16780,10 +16780,19 @@ const toPx = /* #__PURE__ */ toUnit('px');
         const { buttonView } = this;
         let pendingAnimationFrame = false;
         // Reposition the button on scroll, but do it only once per animation frame to avoid performance issues.
-        const repositionOnScroll = ()=>{
+        const repositionOnScroll = (evt, domEvt)=>{
             if (pendingAnimationFrame) {
                 return;
             }
+            // It makes no sense to reposition the button when the user scrolls the dropdown or any other
+            // nested scrollable element. The button should be repositioned only when the user scrolls the
+            // editable or any other scrollable parent of the editable. Leaving it as it is buggy on Chrome
+            // where scrolling nested scrollables is not properly handled.
+            // See more: https://github.com/ckeditor/ckeditor5/issues/17067
+            const editableElement = this._getSelectedEditableElement();
+            if (domEvt.target !== global.document && !getAncestors(editableElement).includes(domEvt.target)) {
+                return;
+            }
             pendingAnimationFrame = true;
             global.window.requestAnimationFrame(()=>{
                 this._updateButton();