diff --git a/browser/components/tabbrowser/content/tabs.js b/browser/components/tabbrowser/content/tabs.js index 1fcebe3962398ff1b7cadef657ac8b68a80e720d..bd64221c60aa2c2f4efdf765299ed03077710ff9 100644 --- a/browser/components/tabbrowser/content/tabs.js +++ b/browser/components/tabbrowser/content/tabs.js @@ -289,6 +289,7 @@ on_TabGroupCollapse(event) { this._invalidateCachedVisibleTabs(); this._unlockTabSizing(); + return; // If the user's selected tab is in the collapsing group, kick them off // the tab. If no tabs exist outside the group, create a new one and @@ -345,7 +346,7 @@ // and we're not hitting the scroll buttons. if ( event.button != 0 || - event.target != this.arrowScrollbox || + !event.target.classList.contains("zen-workspace-normal-tabs-section") || event.composedTarget.localName == "toolbarbutton" ) { return; @@ -695,7 +696,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.visibleTabs.slice(0, gBrowser.pinnedTabCount); + let pinnedTabs = this.ariaFocusableItems.slice(0, gBrowser._numZenEssentials); let tabsPerRow = 0; let position = RTL_UI ? window.windowUtils.getBoundsWithoutFlushing( @@ -780,7 +781,7 @@ } else if (isTabGroupLabel(tab) && !tab.group.collapsed) { this._lockTabSizing(); this.#keepTabSizeLocked = true; - tab.group.collapsed = true; + tab.group.collapsed = !tab.group.hasAttribute("split-view-group"); expandGroupOnDrop = true; } } @@ -879,7 +880,7 @@ ? event.screenY - window.screenY - tabOffset : event.screenY - window.screenY, scrollPos: - this.verticalMode && tab.pinned + this.verticalMode && tab.pinned && false ? this.pinnedTabsContainer.scrollPosition : this.arrowScrollbox.scrollPosition, screenX: event.screenX, @@ -933,6 +934,10 @@ } let draggedTab = event.dataTransfer.mozGetDataAt(TAB_DROP_TYPE, 0); + if (draggedTab && effects === "move") { + gZenPinnedTabManager.applyDragoverClass(event, draggedTab); + gZenViewSplitter.onBrowserDragEndToSplit(event); + } if ( (effects == "move" || effects == "copy") && document == draggedTab.ownerDocument && @@ -1089,6 +1094,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 = []; @@ -1128,10 +1145,11 @@ } } else { let isPinned = draggedTab.pinned; - let numPinned = gBrowser.pinnedTabCount; + let numPinned = gBrowser._numVisiblePinTabsWithoutCollapsed; + let essential = draggedTab.hasAttribute("zen-essential"); let tabs = this.ariaFocusableItems.slice( - isPinned ? 0 : numPinned, - isPinned ? numPinned : undefined + isPinned ? (essential ? 0 : gBrowser._numZenEssentials) : numPinned, + isPinned ? (essential ? gBrowser._numZenEssentials : numPinned) : undefined ); let size = this.verticalMode ? "height" : "width"; let screenAxis = this.verticalMode ? "screenY" : "screenX"; @@ -1180,7 +1198,7 @@ (oldTranslateX && oldTranslateX != newTranslateX) || (oldTranslateY && oldTranslateY != newTranslateY); } else if (this.verticalMode) { - shouldTranslate &&= oldTranslateY && oldTranslateY != newTranslateY; + shouldTranslate &&= oldTranslateY && oldTranslateY != newTranslateY && movingTabs.length === 1; } else { shouldTranslate &&= oldTranslateX && oldTranslateX != newTranslateX; } @@ -1349,6 +1367,7 @@ let nextItem = this.ariaFocusableItems[newIndex]; let tabGroup = isTab(nextItem) && nextItem.group; + if (gZenViewSplitter.handleTabDrop(event, urls, replace, inBackground)) return; gBrowser.loadTabs(urls, { inBackground, replace, @@ -1381,6 +1400,17 @@ this.finishMoveTogetherSelectedTabs(draggedTab); this.finishAnimateTabMove(); + + if (!dt.mozUserCancelled && dt.dropEffect == "none" && !this._isCustomizing) { + const moved = gZenViewSplitter.moveTabToSplitView(event, draggedTab); + if (moved) { + delete draggedTab._dragData; + return; + } + } else if (dt.mozUserCancelled) { + if (gZenViewSplitter._lastOpenedTab) gZenViewSplitter._lastOpenedTab._visuallySelected = false; + } + this.#expandGroupOnDrop(draggedTab); if ( @@ -1607,7 +1637,7 @@ } get newTabButton() { - return this.querySelector("#tabs-newtab-button"); + return gZenWorkspaces.activeWorkspaceStrip.querySelector("#tabs-newtab-button"); } get verticalMode() { @@ -1623,6 +1653,7 @@ } get overflowing() { + gZenWorkspaces.updateOverflowingTabs(); return this.hasAttribute("overflow"); } @@ -1631,26 +1662,54 @@ if (this.#allTabs) { return this.#allTabs; } - let children = Array.from(this.arrowScrollbox.children); + let children = gZenWorkspaces.tabboxChildren; // remove arrowScrollbox periphery element children.pop(); // explode tab groups // Iterate backwards over the array to preserve indices while we modify // things in place - for (let i = children.length - 1; i >= 0; i--) { - if (children[i].tagName == "tab-group") { - children.splice(i, 1, ...children[i].tabs); + const pinnedTabs = [...gZenWorkspaces.getCurrentEssentialsContainer().children, ...this.pinnedTabsContainer.children]; + const expandTabs = (tabs) => { + for (let i = tabs.length - 1; i >= 0; i--) { + const tab = tabs[i]; + if (isTabGroup(tab)) { + // remove the group from the list + tabs.splice(i, 1); + // add the tabs in the group to the list + tabs.splice(i, 0, ...tab.tabs); + } } } - - this.#allTabs = [...this.pinnedTabsContainer.children, ...children]; + expandTabs(pinnedTabs); + expandTabs(children); + const allTabs = [ + ...pinnedTabs, + ...children, + ]; + const lastPinnedTabIdx = pinnedTabs.length - 1; + let i = 0; + for (const tab of [...allTabs]) { + // add glance tabs (tabs inside tabs) to the list + const glanceTab = tab.glanceTab; + if (glanceTab) { + // insert right after the parent tab. note: it must be inserted before + // the last pinned tab so it can be inserted in the correct order + allTabs.splice(Math.max(i++ + 1, lastPinnedTabIdx), 0, glanceTab); + } else if (tab.classList.contains("pinned-tabs-container-separator")) { + // remove the separator from the list + allTabs.splice(i, 1); + i--; + } + i++; + } + this.#allTabs = allTabs; return this.#allTabs; } get allGroups() { let children = Array.from(this.arrowScrollbox.children); - return children.filter(node => node.tagName == "tab-group"); + return gZenWorkspaces.allTabGroups; } /** @@ -1717,20 +1776,17 @@ let elementIndex = 0; - for (let i = 0; i < this.pinnedTabsContainer.childElementCount; i++) { - this.pinnedTabsContainer.children[i].elementIndex = elementIndex++; - } - let children = Array.from(this.arrowScrollbox.children); + let children = gZenWorkspaces.tabboxChildrenWithoutEmpty; let focusableItems = []; - for (let child of children) { + for (let child of [...gZenWorkspaces.getCurrentEssentialsContainer().children, ...this.pinnedTabsContainer.children, ...children]) { if (isTab(child) && child.visible) { child.elementIndex = elementIndex++; focusableItems.push(child); } else if (isTabGroup(child)) { child.labelElement.elementIndex = elementIndex++; focusableItems.push(child.labelElement); - if (!child.collapsed) { + if (!child.collapsed && !child.hasAttribute("split-view-group")) { let visibleTabsInGroup = child.tabs.filter(tab => tab.visible); visibleTabsInGroup.forEach(tab => { tab.elementIndex = elementIndex++; @@ -1740,10 +1796,7 @@ } } - this.#focusableItems = [ - ...this.pinnedTabsContainer.children, - ...focusableItems, - ]; + this.#focusableItems = focusableItems; return this.#focusableItems; } @@ -1751,6 +1804,7 @@ _invalidateCachedTabs() { this.#allTabs = null; this._invalidateCachedVisibleTabs(); + gZenWorkspaces._allStoredTabs = null; } _invalidateCachedVisibleTabs() { @@ -1766,8 +1820,8 @@ #isContainerVerticalPinnedGrid(tab) { return ( this.verticalMode && - tab.hasAttribute("pinned") && - this.hasAttribute("expanded") && + (tab.hasAttribute("zen-essential")) && + (this.hasAttribute("expanded") || document.documentElement.hasAttribute("zen-sidebar-expanded")) && !this.expandOnHover ); } @@ -1783,7 +1837,7 @@ if (node == null) { // We have a container for non-tab elements at the end of the scrollbox. - node = this.arrowScrollbox.lastChild; + node = gZenWorkspaces.activeWorkspaceStrip.lastChild; } node.before(tab); @@ -1878,7 +1932,7 @@ // There are separate "new tab" buttons for horizontal tabs toolbar, vertical tabs and // for when the tab strip is overflowed (which is shared by vertical and horizontal tabs); // Attach the long click popup to all of them. - const newTab = document.getElementById("new-tab-button"); + const newTab = gZenWorkspaces.activeWorkspaceStrip.querySelector("#tabs-newtab-button"); const newTab2 = this.newTabButton; const newTabVertical = document.getElementById( "vertical-tabs-newtab-button" @@ -1973,10 +2027,12 @@ _handleTabSelect(aInstant) { let selectedTab = this.selectedItem; + if (!selectedTab) return; if (this.overflowing) { this.arrowScrollbox.ensureElementIsVisible(selectedTab, aInstant); } + gZenCompactModeManager.flashSidebarIfNecessary(aInstant); selectedTab._notselectedsinceload = false; } @@ -2130,7 +2186,7 @@ return; } - let tabs = this.visibleTabs.slice(0, gBrowser.pinnedTabCount); + let tabs = this.ariaFocusableItems.slice(0, gBrowser._numZenEssentials); let directionX = screenX > dragData.animLastScreenX; let directionY = screenY > dragData.animLastScreenY; @@ -2139,6 +2195,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; @@ -2168,7 +2226,7 @@ let translateX = screenX - dragData.screenX; let translateY = screenY - dragData.screenY; translateY += - this.pinnedTabsContainer.scrollPosition - dragData.scrollPos; + dragData.scrollPos; let firstBoundX = firstTabInRow.screenX - firstMovingTabScreenX; let firstBoundY = firstTabInRow.screenY - firstMovingTabScreenY; let lastBoundX = @@ -2294,7 +2352,7 @@ } dragData.animDropElementIndex = newIndex; - dragData.dropElement = tabs[newIndex]; + dragData.dropElement = tabs[Math.min(newIndex, tabs.length - 1)]; dragData.dropBefore = newIndex < tabs.length; // Shift background tabs to leave a gap where the dragged tab @@ -2327,12 +2385,16 @@ this.#clearDragOverCreateGroupTimer(); - 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"); let tabs = this.ariaFocusableItems.slice( - isPinned ? 0 : numPinned, - isPinned ? numPinned : undefined + isPinned ? (essential ? 0 : gBrowser._numZenEssentials) : numPinned, + isPinned ? (essential ? gBrowser._numZenEssentials : numPinned) : undefined ); + if (draggedTab.group?.hasAttribute("split-view-group")) { + draggedTab = draggedTab.group.labelElement; + } if (this.#rtlMode) { tabs.reverse(); @@ -2346,7 +2408,7 @@ let size = this.verticalMode ? "height" : "width"; let translateAxis = this.verticalMode ? "translateY" : "translateX"; let scrollDirection = this.verticalMode ? "scrollTop" : "scrollLeft"; - let { width: tabWidth, height: tabHeight } = bounds(draggedTab); + let { width: tabWidth, height: tabHeight } = bounds(draggedTab.group?.hasAttribute("split-view-group") ? draggedTab.group : draggedTab); let translateX = event.screenX - dragData.screenX; let translateY = event.screenY - dragData.screenY; @@ -2360,12 +2422,21 @@ let lastTab = tabs.at(-1); 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]; let shiftSize = lastMovingTabScreen - firstMovingTabScreen; + if (firstMovingTab.hasAttribute("split-view-group")) { + shiftSize += 5; // A hack to allow more space for the group + } let translate = screen - dragData[screenAxis]; - if (!isPinned) { + if (true) { translate += this.arrowScrollbox.scrollbox[scrollDirection] - dragData.scrollPos; } else if (isPinned && this.verticalMode) { @@ -2384,6 +2455,9 @@ // Shift the `.tab-group-label-container` to shift the label element. item = item.parentElement; } + if (item.parentElement?.hasAttribute("split-view-group")) { + item = item.parentElement; + } item.style.transform = `${translateAxis}(${translate}px)`; } @@ -2521,6 +2595,9 @@ break; } let element = tabs[mid]; + if (element?.group?.hasAttribute("split-view-group")) { + element = element.group.labelElement; + } let elementForSize = isTabGroupLabel(element) ? element.parentElement : element; @@ -2604,7 +2681,7 @@ let shouldCreateGroupOnDrop; let dropBefore; if (dropElement) { - let dropElementForOverlap = isTabGroupLabel(dropElement) + let dropElementForOverlap = isTabGroupLabel(dropElement) && !dropElement.group?.hasAttribute("split-view-group") ? dropElement.parentElement : dropElement; @@ -2686,12 +2763,7 @@ } } - if ( - gBrowser._tabGroupsEnabled && - isTab(draggedTab) && - !isPinned && - (!numPinned || newDropElementIndex > numPinned) - ) { + if (isTab(draggedTab)) { let dragOverGroupingThreshold = 1 - moveOverThreshold; // When dragging tab(s) over an ungrouped tab, signal to the user @@ -2739,7 +2811,7 @@ // Dropping right before the tab group. dropElement = dropElementGroup; colorCode = undefined; - } else if (dropElementGroup.collapsed) { + } else if (dropElement?.group?.hasAttribute("split-view-group")) { // Dropping right after the collapsed tab group. dropElement = dropElementGroup; colorCode = undefined; @@ -2769,7 +2841,7 @@ // 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; } @@ -2778,6 +2850,9 @@ if (isTabGroupLabel(item)) { // Shift the `.tab-group-label-container` to shift the label element. item = item.parentElement; + if (item.parentElement?.hasAttribute("split-view-group")) { + item = item.parentElement; + } } item.style.transform = transform; } @@ -2830,8 +2905,9 @@ ); } - finishAnimateTabMove() { - if (!this.#isMovingTab()) { + finishAnimateTabMove(always = false) { + gZenPinnedTabManager.removeTabContainersDragoverClass(); + if (!this.#isMovingTab() && !always) { return; } @@ -2843,6 +2919,12 @@ item = item.parentElement; } item.style.transform = ""; + if (item.closest("tab-group")?.hasAttribute("split-view-group")) item.closest("tab-group").style.transform = ""; + if (item.closest("tab-group")?.hasAttribute("split-view-group")) { + for (let tab of item.closest("tab-group").tabs) { + tab.style.transform = ""; + } + } item.removeAttribute("dragover-createGroup"); } this.removeAttribute("movingtab-createGroup"); @@ -2889,7 +2971,7 @@ let postTransitionCleanup = () => { movingTab._moveTogetherSelectedTabsData.animate = false; }; - if (gReduceMotion) { + if (true || gReduceMotion) { postTransitionCleanup(); } else { let onTransitionEnd = transitionendEvent => { @@ -3062,7 +3144,7 @@ } _notifyBackgroundTab(aTab) { - if (aTab.pinned || !aTab.visible || !this.overflowing) { + if (aTab.hasAttribute("zen-essential") || !aTab.visible || !this.overflowing) { return; } @@ -3188,6 +3270,9 @@ return null; } } + if (target?.group?.hasAttribute("split-view-group")) { + target = target.group.labelElement; + } return target; }