diff --git a/browser/components/tabbrowser/content/drag-and-drop.js b/browser/components/tabbrowser/content/drag-and-drop.js index 97b931c3c7385a52d20204369fcf6d6999053687..bc49f4f5a90638d725eca016d00f30d9548dce83 100644 --- a/browser/components/tabbrowser/content/drag-and-drop.js +++ b/browser/components/tabbrowser/content/drag-and-drop.js @@ -32,6 +32,9 @@ * @returns {MozTabbrowserTab|vbox} */ const elementToMove = element => { + if (element.group?.hasAttribute("split-view-group")) { + return element.group; + } if (isTab(element)) { return element; } @@ -112,6 +115,10 @@ } let draggedTab = event.dataTransfer.mozGetDataAt(TAB_DROP_TYPE, 0); + if (draggedTab && dropEffect === "move") { + gZenPinnedTabManager.applyDragoverClass(event, draggedTab); + gZenViewSplitter.onBrowserDragEndToSplit(event); + } if ( (dropEffect == "move" || dropEffect == "copy") && document == draggedTab.ownerDocument && @@ -266,6 +273,18 @@ this._tabDropIndicator.hidden = true; event.stopPropagation(); + if (draggedTab?.hasAttribute("zen-has-splitted")) { + draggedTab.removeAttribute("zen-has-splitted"); + draggedTab._visuallySelected = false; + } + if (draggedTab && dropEffect == "move") { + let moved = gZenPinnedTabManager.moveToAnotherTabContainerIfNecessary(event, movingTabs); + + if (moved) { + this.finishMoveTogetherSelectedTabs(draggedTab); + return; + } + } if (draggedTab && dropEffect == "copy") { let duplicatedDraggedTab; let duplicatedTabs = []; @@ -291,8 +310,9 @@ let translateOffsetY = oldTranslateY % tabHeight; let newTranslateX = oldTranslateX - translateOffsetX; let newTranslateY = oldTranslateY - translateOffsetY; - let isPinned = draggedTab.pinned; - let numPinned = gBrowser.pinnedTabCount; + let isPinned = draggedTab?.group ? draggedTab.group.pinned : draggedTab.pinned; + let numPinned = gBrowser._numVisiblePinTabsWithoutCollapsed; + let essential = draggedTab.hasAttribute("zen-essential"); if (this._isContainerVerticalPinnedGrid(draggedTab)) { // Update both translate axis for pinned vertical expanded tabs @@ -308,8 +328,8 @@ } } else { let tabs = this._tabbrowserTabs.ariaFocusableItems.slice( - isPinned ? 0 : numPinned, - isPinned ? numPinned : undefined + isPinned ? (essential ? 0 : gBrowser._numZenEssentials) : numPinned, + isPinned ? (essential ? gBrowser._numZenEssentials : numPinned) : undefined ); let size = this._tabbrowserTabs.verticalMode ? "height" : "width"; let screenAxis = this._tabbrowserTabs.verticalMode @@ -362,11 +382,13 @@ this._dragToPinPromoCard, ]; let shouldPin = + false && isTab(draggedTab) && !draggedTab.pinned && (overPinnedDropIndicator || dragToPinTargets.some(el => el.contains(event.target))); let shouldUnpin = + false && isTab(draggedTab) && draggedTab.pinned && this._tabbrowserTabs.arrowScrollbox.contains(event.target); @@ -384,6 +406,7 @@ (oldTranslateY && oldTranslateY != newTranslateY); } else if (this._tabbrowserTabs.verticalMode) { shouldTranslate &&= oldTranslateY && oldTranslateY != newTranslateY; + shouldTranslate = false; } else { shouldTranslate &&= oldTranslateX && oldTranslateX != newTranslateX; } @@ -440,7 +463,7 @@ item.removeAttribute("tabdrop-samewindow"); resolve(); }; - if (gReduceMotion) { + if (true || gReduceMotion) { postTransitionCleanup(); } else { let onTransitionEnd = transitionendEvent => { @@ -581,6 +604,7 @@ let nextItem = this._tabbrowserTabs.ariaFocusableItems[newIndex]; let tabGroup = isTab(nextItem) && nextItem.group; + if (gZenViewSplitter.handleTabDrop(event, urls, replace, inBackground)) return; gBrowser.loadTabs(urls, { inBackground, replace, @@ -618,7 +642,16 @@ this._expandGroupOnDrop(draggedTab); } this._resetTabsAfterDrop(draggedTab.ownerDocument); - + if (!dt.mozUserCancelled && dt.dropEffect == "none" && !this._isCustomizing) { + const moved = gZenViewSplitter.moveTabToSplitView(event, draggedTab); + if (moved) { + delete draggedTab._dragData; + return; + } + } else if (dt.mozUserCancelled) { + gZenViewSplitter.onBrowserDragEndToSplit(event, true); + if (gZenViewSplitter._lastOpenedTab) gZenViewSplitter._lastOpenedTab._visuallySelected = false; + } if ( dt.mozUserCancelled || dt.dropEffect != "none" || @@ -822,7 +855,10 @@ _getDragTarget(event, { ignoreSides = false } = {}) { let { target } = event; while (target) { - if (isTab(target) || isTabGroupLabel(target)) { + if (isTab(target) || isTabGroupLabel(target) || target?.classList?.contains("tab-group-label-container")) { + if (target.classList?.contains("tab-group-label-container")) { + target = target.querySelector(".tab-group-label"); + } break; } target = target.parentNode; @@ -839,14 +875,17 @@ return null; } } + if (target?.group?.hasAttribute("split-view-group")) { + target = target.group.labelElement; + } return target; } _isContainerVerticalPinnedGrid(tab) { return ( this._tabbrowserTabs.verticalMode && - tab.pinned && - this._tabbrowserTabs.hasAttribute("expanded") && + tab.hasAttribute("zen-essential") && + (this._tabbrowserTabs.hasAttribute("expanded") || document.documentElement.hasAttribute("zen-sidebar-expanded")) && !this._tabbrowserTabs.expandOnHover ); } @@ -877,7 +916,8 @@ isTabGroupLabel(draggedTab) && draggedTab._dragData?.expandGroupOnDrop ) { - draggedTab.group.collapsed = false; + draggedTab.group.collapsed = draggedTab.group.hasAttribute("has-active"); + gZenFolders.animateGroupMove(draggedTab.group, true); } } @@ -942,10 +982,7 @@ if (this._isContainerVerticalPinnedGrid(tab)) { // In expanded vertical mode, the max number of pinned tabs per row is dynamic // Set this before adjusting dragged tab's position - let pinnedTabs = this._tabbrowserTabs.visibleTabs.slice( - 0, - gBrowser.pinnedTabCount - ); + let pinnedTabs = this._tabbrowserTabs.ariaFocusableItems.slice(0, gBrowser._numZenEssentials); let tabsPerRow = 0; let position = RTL_UI ? window.windowUtils.getBoundsWithoutFlushing( @@ -1112,7 +1149,7 @@ let dropEffect = this.getDropEffectForTabDrag(event); let isMovingInTabStrip = !fromTabList && dropEffect == "move"; let collapseTabGroupDuringDrag = - isMovingInTabStrip && isTabGroupLabel(tab) && !tab.group.collapsed; + isMovingInTabStrip && isTabGroupLabel(tab) && (!tab.group.collapsed || tab.group.hasAttribute("has-active")); tab._dragData = { offsetX: this._tabbrowserTabs.verticalMode @@ -1122,7 +1159,7 @@ ? event.screenY - window.screenY - tabOffset : event.screenY - window.screenY, scrollPos: - this._tabbrowserTabs.verticalMode && tab.pinned + this._tabbrowserTabs.verticalMode && tab.pinned && false ? this._tabbrowserTabs.pinnedTabsContainer.scrollPosition : this._tabbrowserTabs.arrowScrollbox.scrollPosition, screenX: event.screenX, @@ -1149,6 +1186,7 @@ if (collapseTabGroupDuringDrag) { tab.group.collapsed = true; + gZenFolders.animateGroupMove(tab.group); } } } @@ -1173,6 +1211,16 @@ if (tabStripItemElement.hasAttribute("dragtarget")) { return; } + let { movingTabs: zenMovingTabs } = tab._dragData; + for (let movingTab of zenMovingTabs.slice(zenMovingTabs.findIndex(t => t._tPos == tab._tPos))) { + if (isTabGroupLabel(tab)) { + movingTab = movingTab.parentElement; + } + // "dragtarget" contains the following rules which must only be set AFTER the above + // elements have been adjusted. {z-index: 3 !important, position: absolute !important} + movingTab.setAttribute("zen-dragtarget", ""); + } + return; let isPinned = tab.pinned; let numPinned = gBrowser.pinnedTabCount; let allTabs = this._tabbrowserTabs.ariaFocusableItems; @@ -1624,10 +1672,7 @@ return; } - let tabs = this._tabbrowserTabs.visibleTabs.slice( - 0, - gBrowser.pinnedTabCount - ); + let tabs = this._tabbrowserTabs.ariaFocusableItems.slice(0, gBrowser._numZenEssentials); let directionX = screenX > dragData.animLastScreenX; let directionY = screenY > dragData.animLastScreenY; @@ -1636,6 +1681,8 @@ let { width: tabWidth, height: tabHeight } = draggedTab.getBoundingClientRect(); + tabWidth += 4; // Add 4px to account for the gap + tabHeight += 4; let shiftSizeX = tabWidth * movingTabs.length; let shiftSizeY = tabHeight; dragData.tabWidth = tabWidth; @@ -1672,8 +1719,8 @@ let lastBoundX = lastTabInRow.screenX + lastTabInRow.getBoundingClientRect().width - - (lastMovingTabScreenX + tabWidth); - let lastBoundY = periphery.screenY - (lastMovingTabScreenY + tabHeight); + (lastMovingTabScreenX + tabWidth) + 4; + let lastBoundY = lastTab.screenY - lastMovingTabScreenY; translateX = Math.min(Math.max(translateX, firstBoundX), lastBoundX); translateY = Math.min(Math.max(translateY, firstBoundY), lastBoundY); @@ -1833,13 +1880,18 @@ this._clearDragOverGroupingTimer(); this.#clearPinnedDropIndicatorTimer(); - let isPinned = draggedTab.pinned; - let numPinned = gBrowser.pinnedTabCount; + let isPinned = draggedTab?.group ? draggedTab.group.pinned : draggedTab.pinned; + let numPinned = gBrowser._numVisiblePinTabsWithoutCollapsed; + let essential = draggedTab.hasAttribute("zen-essential"); + const isDraggingFolder = isTabGroupLabel(draggedTab) && draggedTab.group?.isZenFolder; let allTabs = this._tabbrowserTabs.ariaFocusableItems; let tabs = allTabs.slice( - isPinned ? 0 : numPinned, - isPinned ? numPinned : undefined + (isPinned && essential) ? 0 : gBrowser._numZenEssentials, + isPinned ? (essential ? gBrowser._numZenEssentials : (isDraggingFolder ? numPinned : undefined)) : undefined ); + if (draggedTab.group?.hasAttribute("split-view-group")) { + draggedTab = draggedTab.group.labelElement; + } if (this._rtlMode) { tabs.reverse(); @@ -1854,7 +1906,7 @@ let translateAxis = this._tabbrowserTabs.verticalMode ? "translateY" : "translateX"; - let { width: tabWidth, height: tabHeight } = bounds(draggedTab); + let { width: tabWidth, height: tabHeight } = bounds(draggedTab.group?.hasAttribute("split-view-group") ? draggedTab.group : draggedTab); let tabSize = this._tabbrowserTabs.verticalMode ? tabHeight : tabWidth; let translateX = event.screenX - dragData.screenX; let translateY = event.screenY - dragData.screenY; @@ -1870,6 +1922,12 @@ ); let lastMovingTab = movingTabs.at(-1); let firstMovingTab = movingTabs[0]; + if (lastMovingTab.group?.hasAttribute("split-view-group")) { + lastMovingTab = lastMovingTab.group; + } + if (firstMovingTab.group?.hasAttribute("split-view-group")) { + firstMovingTab = firstMovingTab.group; + } let endEdge = ele => ele[screenAxis] + bounds(ele)[size]; let lastMovingTabScreen = endEdge(lastMovingTab); let firstMovingTabScreen = firstMovingTab[screenAxis]; @@ -1884,6 +1942,13 @@ let endBound = this._rtlMode ? endEdge(this._tabbrowserTabs) - lastMovingTabScreen : periphery[screenAxis] - 1 - lastMovingTabScreen; + { + let firstTab = tabs.at(this._rtlMode ? -1 : 0); + let lastTab = tabs.at(this._rtlMode ? 0 : -1); + startBound = firstTab[screenAxis] - firstMovingTabScreen; + endBound = endEdge(lastTab) - lastMovingTabScreen; + endBound = gZenPinnedTabManager.getLastTabBound(endBound, lastTab, isDraggingFolder); + } translate = Math.min(Math.max(translate, startBound), endBound); // Center the tab under the cursor if the tab is not under the cursor while dragging @@ -2075,6 +2140,8 @@ }; let dropElement = getOverlappedElement(); + if (dropElement?.hasAttribute("split-view-group")) dropElement = dropElement.labelElement; + gZenPinnedTabManager.animateSeparatorMove(movingTabs, dropElement, isPinned, event); let newDropElementIndex; if (dropElement) { @@ -2157,7 +2224,7 @@ ? Services.prefs.getIntPref( "browser.tabs.dragDrop.moveOverThresholdPercent" ) / 100 - : 0.5; + : Services.prefs.getIntPref('zen.view.drag-and-drop.move-over-threshold') / 100; moveOverThreshold = Math.min(1, Math.max(0, moveOverThreshold)); let shouldMoveOver = overlapPercent > moveOverThreshold; if (logicalForward && shouldMoveOver) { @@ -2190,6 +2257,7 @@ // If dragging a group over another group, don't make it look like it is // possible to drop the dragged group inside the other group. if ( + false && isTabGroupLabel(draggedTab) && dropElement?.group && (!dropElement.group.collapsed || @@ -2216,20 +2284,13 @@ let isOutOfBounds = isPinned ? dropElement.elementIndex >= numPinned : dropElement.elementIndex < numPinned; - if (isOutOfBounds) { - // Drop after last pinned tab - dropElement = this._tabbrowserTabs.ariaFocusableItems[numPinned - 1]; - dropBefore = false; - } } - if ( - gBrowser._tabGroupsEnabled && - isTab(draggedTab) && - !isPinned && - (!numPinned || newDropElementIndex >= numPinned) - ) { + if (isTab(draggedTab) || isTabGroupLabel(draggedTab)) { let dragOverGroupingThreshold = 1 - moveOverThreshold; + if (draggedTab && !dropElement?.group) { + gZenFolders.highlightGroupOnDragOver(null); + } let groupingDelay = Services.prefs.getIntPref( "browser.tabs.dragDrop.createGroup.delayMS" ); @@ -2237,6 +2298,7 @@ // When dragging tab(s) over an ungrouped tab, signal to the user // that dropping the tab(s) will create a new tab group. let shouldCreateGroupOnDrop = + false && !movingTabsSet.has(dropElement) && isTab(dropElement) && !dropElement?.group && @@ -2245,6 +2307,7 @@ // When dragging tab(s) over a collapsed tab group label, signal to the // user that dropping the tab(s) will add them to the group. let shouldDropIntoCollapsedTabGroup = + false && isTabGroupLabel(dropElement) && dropElement.group.collapsed && overlapPercent > dragOverGroupingThreshold; @@ -2302,6 +2365,14 @@ dropElement = dropElementGroup.tabs[0]; dropBefore = true; } + ({ dropElement, colorCode, dropBefore } = gZenFolders.handleDragOverTabGroupLabel( + dropElement, + draggedTab, + overlapPercent, + movingTabs, + dropBefore, + colorCode + )); } this._setDragOverGroupColor(colorCode); this._tabbrowserTabs.toggleAttribute( @@ -2324,10 +2395,11 @@ dragData.dropBefore = dropBefore; dragData.animDropElementIndex = newDropElementIndex; + gZenFolders.setFolderIndentation(movingTabs, dropElement); // Shift background tabs to leave a gap where the dragged tab // would currently be dropped. for (let item of tabs) { - if (item == draggedTab) { + if (item == draggedTab || (item.group?.hasAttribute("split-view-group") && item.group == draggedTab.group)) { continue; } @@ -2417,11 +2489,13 @@ } finishAnimateTabMove() { + gZenPinnedTabManager.onDragFinish(); if (!this.#isMovingTab()) { return; } this.#setMovingTabMode(false); + gZenFolders.highlightGroupOnDragOver(null); for (let item of this._tabbrowserTabs.ariaFocusableItems) { this._resetGroupTarget(item); @@ -2457,7 +2531,7 @@ tab.style.left = ""; tab.style.top = ""; tab.style.maxWidth = ""; - tab.removeAttribute("dragtarget"); + tab.removeAttribute("zen-dragtarget"); } for (let label of draggedTabDocument.getElementsByClassName( "tab-group-label-container" @@ -2467,7 +2541,7 @@ label.style.left = ""; label.style.top = ""; label.style.maxWidth = ""; - label.removeAttribute("dragtarget"); + label.removeAttribute("zen-dragtarget"); } let periphery = draggedTabDocument.getElementById( "tabbrowser-arrowscrollbox-periphery"