tor-browser

commit 1ec0d52bbd4ddb9ec2219fbee6f0ae31d043bf69
parent 6a98e783e1394d59748e1bb3108de0e69f865352
Author: Nikki Sharpley <nsharpley@mozilla.com>
Date:   Fri,  7 Nov 2025 14:47:10 +0000

Bug 1998519 - Apply overflow-clip-margin of 4px to dragtarget tab only r=tabbrowser-reviewers,desktop-theme-reviewers,kcochrane

Differential Revision: https://phabricator.services.mozilla.com/D271620

Diffstat:
M
browser/components/tabbrowser/test/browser/tabs/browser_tab_tooltips.js
 | 
6
+++---
M
browser/themes/shared/tabbrowser/tabs.css
 | 
4
+++-

2 files changed, 6 insertions(+), 4 deletions(-)
diff --git a/browser/components/tabbrowser/test/browser/tabs/browser_tab_tooltips.js b/browser/components/tabbrowser/test/browser/tabs/browser_tab_tooltips.js
@@ -12,12 +12,12 @@ function openTooltip(node) {
     event => event.originalTarget.nodeName == "tooltip"
   );
   window.windowUtils.disableNonTestMouseEvents(true);
-  EventUtils.synthesizeMouse(node, 10, 10, { type: "mouseover" });
-  EventUtils.synthesizeMouse(node, 10, 10, { type: "mousemove" });
+  EventUtils.synthesizeMouse(node, 2, 2, { type: "mouseover" });
+  EventUtils.synthesizeMouse(node, 4, 4, { type: "mousemove" });
   EventUtils.synthesizeMouse(node, MOUSE_OFFSET, MOUSE_OFFSET, {
     type: "mousemove",
   });
-  EventUtils.synthesizeMouse(node, 10, 10, { type: "mouseout" });
+  EventUtils.synthesizeMouse(node, 2, 2, { type: "mouseout" });
   window.windowUtils.disableNonTestMouseEvents(false);
   return tooltipShownPromise;
 }
diff --git a/browser/themes/shared/tabbrowser/tabs.css b/browser/themes/shared/tabbrowser/tabs.css
@@ -30,7 +30,7 @@
   --tab-pinned-min-width-expanded: calc(var(--tab-pinned-expanded-background-width) + 2 * var(--tab-pinned-margin-inline-expanded));
   --tab-pinned-container-margin-inline-expanded: var(--space-small);
   --tab-border-radius: var(--toolbarbutton-border-radius);
-  --tab-overflow-clip-margin: 4px;
+  --tab-overflow-clip-margin: 2px;
   --tab-close-button-padding: 6px;
   --tab-block-margin: 4px;
   --tab-icon-end-margin: 5.5px;
@@ -216,6 +216,8 @@
   }
 
   &[dragtarget] {
+    /* Margin needs to be at least 4px to not clip multiselect stacking effect */
+    overflow-clip-margin: 4px;
     z-index: 3 !important;
     position: absolute !important;
     pointer-events: none; /* avoid blocking dragover events on scroll buttons */