fix(VMenu, VTooltip): apply scroll-strategy reposition for horizontal overflow (#21309)

J-Sek · web-flow · commit bd486588ae66 · 2025-07-31T01:34:13.000+02:00
fixes #20625
diff --git a/packages/vuetify/src/util/getScrollParent.ts b/packages/vuetify/src/util/getScrollParent.ts
@@ -25,7 +25,9 @@ export function hasScrollbar (el?: Element | null) {
   if (!el || el.nodeType !== Node.ELEMENT_NODE) return false
 
   const style = window.getComputedStyle(el)
-  return style.overflowY === 'scroll' || (style.overflowY === 'auto' && el.scrollHeight > el.clientHeight)
+  const hasVerticalScrollbar = style.overflowY === 'scroll' || (style.overflowY === 'auto' && el.scrollHeight > el.clientHeight)
+  const hasHorizontalScrollbar = style.overflowX === 'scroll' || (style.overflowX === 'auto' && el.scrollWidth > el.clientWidth)
+  return hasVerticalScrollbar || hasHorizontalScrollbar
 }
 
 function isPotentiallyScrollable (el?: Element | null) {

Original file line number	Diff line number	Diff line change
`@@ -25,7 +25,9 @@ export function hasScrollbar (el?: Element \| null) {`
`25`	`25`	`if (!el \|\| el.nodeType !== Node.ELEMENT_NODE) return false`
`26`	`26`
`27`	`27`	`const style = window.getComputedStyle(el)`
`28`		`- return style.overflowY === 'scroll' \|\| (style.overflowY === 'auto' && el.scrollHeight > el.clientHeight)`
	`28`	`+ const hasVerticalScrollbar = style.overflowY === 'scroll' \|\| (style.overflowY === 'auto' && el.scrollHeight > el.clientHeight)`
	`29`	`+ const hasHorizontalScrollbar = style.overflowX === 'scroll' \|\| (style.overflowX === 'auto' && el.scrollWidth > el.clientWidth)`
	`30`	`+ return hasVerticalScrollbar \|\| hasHorizontalScrollbar`
`29`	`31`	`}`
`30`	`32`
`31`	`33`	`function isPotentiallyScrollable (el?: Element \| null) {`