feat(feedback): Extra check for iPad in screenshot support (getsentry#12593)

c298lee · web-flow · commit 44ef79a42558 · 2024-06-20T19:04:28.000Z
iPads on iPad OS 13+ will show up as Macintosh in the UserAgent since they show the desktop version of sites. This adds extra checks to distinguish if it's an iPad. Fixes [JAVASCRIPT-2TNR](https://sentry.sentry.io/feedback/?feedbackSlug=javascript:5511454761&project=11276)
diff --git a/packages/feedback/src/util/isScreenshotSupported.ts b/packages/feedback/src/util/isScreenshotSupported.ts
@@ -17,6 +17,15 @@ export function isScreenshotSupported(): boolean {
   if (/Android|webOS|iPhone|iPad|iPod|BlackBerry|IEMobile|Opera Mini/i.test(NAVIGATOR.userAgent)) {
     return false;
   }
+  /**
+   * User agent on iPads show as Macintosh, so we need extra checks
+   *
+   * https://forums.developer.apple.com/forums/thread/119186
+   * https://stackoverflow.com/questions/60482650/how-to-detect-ipad-useragent-on-safari-browser
+   */
+  if (/Macintosh/i.test(NAVIGATOR.userAgent) && NAVIGATOR.maxTouchPoints && NAVIGATOR.maxTouchPoints > 1) {
+    return false;
+  }
   if (!isSecureContext) {
     return false;
   }
