From ec8fafa9e785f6817bf6d0058a6953b0091a1370 Mon Sep 17 00:00:00 2001 From: Kael Date: Wed, 2 Jun 2021 20:03:00 +1000 Subject: [PATCH] fix(menuable): don't account for app offset in fixed context fixes #13740 --- packages/vuetify/src/mixins/menuable/index.ts | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/packages/vuetify/src/mixins/menuable/index.ts b/packages/vuetify/src/mixins/menuable/index.ts index 66247f798ff..a6bb8b75e6e 100644 --- a/packages/vuetify/src/mixins/menuable/index.ts +++ b/packages/vuetify/src/mixins/menuable/index.ts @@ -404,12 +404,14 @@ export default baseMixins.extend().extend({ // Display and hide to get dimensions this.sneakPeek(() => { - if (this.$refs.content && this.$refs.content.offsetParent) { - const offsetRect = this.getRoundedBoundedClientRect(this.$refs.content.offsetParent) - - this.relativeYOffset = window.pageYOffset + offsetRect.top - dimensions.activator.top -= this.relativeYOffset - dimensions.activator.left -= window.pageXOffset + offsetRect.left + if (this.$refs.content) { + if (this.$refs.content.offsetParent) { + const offsetRect = this.getRoundedBoundedClientRect(this.$refs.content.offsetParent) + + this.relativeYOffset = window.pageYOffset + offsetRect.top + dimensions.activator.top -= this.relativeYOffset + dimensions.activator.left -= window.pageXOffset + offsetRect.left + } dimensions.content = this.measure(this.$refs.content) }