From effb0ea8294907244a07938e789c7a1be83d1eae Mon Sep 17 00:00:00 2001
From: Stefan Puchta <puchta@adobe.com>
Date: Fri, 8 Sep 2023 11:57:48 +0200
Subject: [PATCH] fix(tooltip): update default offset to match Spectrum defs

---
 packages/tooltip/src/Tooltip.ts | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/packages/tooltip/src/Tooltip.ts b/packages/tooltip/src/Tooltip.ts
index 9f1cc0c7d2..3db7ff4e7b 100644
--- a/packages/tooltip/src/Tooltip.ts
+++ b/packages/tooltip/src/Tooltip.ts
@@ -25,6 +25,10 @@ import type {
     OverlayOpenCloseDetail,
     Placement,
 } from '@spectrum-web-components/overlay';
+import {
+    IS_MOBILE,
+    MatchMediaController,
+} from '@spectrum-web-components/reactive-controllers/src/MatchMedia.js';
 
 import tooltipStyles from './tooltip.css.js';
 import { ifDefined } from 'lit/directives/if-defined.js';
@@ -117,6 +121,7 @@ export class Tooltip extends SpectrumElement {
     public static override get styles(): CSSResultArray {
         return [tooltipStyles];
     }
+    protected isMobile = new MatchMediaController(this, IS_MOBILE);
 
     /**
      * Automatically bind to the parent element of the assigned `slot` or the parent element of the `sp-tooltip`.
@@ -126,7 +131,7 @@ export class Tooltip extends SpectrumElement {
     public selfManaged = false;
 
     @property({ type: Number })
-    public offset = 0;
+    public offset = this.isMobile.matches ? 5 : 4;
 
     @property({ type: Boolean, reflect: true })
     public open = false;