fix(plugin-autocapture-browser): apply rage clicks to window over viewport

packages/plugin-autocapture-browser/src/autocapture/track-action-click.ts

@@ -1,4 +1,4 @@
-import { AllWindowObservables, AutoCaptureOptionsWithDefaults } from 'src/autocapture-plugin';
+import { AllWindowObservables, AutoCaptureOptionsWithDefaults } from '../autocapture-plugin';
 import { BrowserClient, ActionType, merge, asyncMap } from '@amplitude/analytics-core';
 import {
   ElementBasedTimestampedEvent,

packages/plugin-autocapture-browser/src/autocapture/track-change.ts

@@ -1,7 +1,11 @@
-import { AllWindowObservables } from 'src/autocapture-plugin';
-import { ElementBasedTimestampedEvent, type evaluateTriggersFn } from 'src/helpers';
+import { AllWindowObservables } from '../autocapture-plugin';
+import {
+  filterOutNonTrackableEvents,
+  shouldTrackEvent,
+  ElementBasedTimestampedEvent,
+  type evaluateTriggersFn,
+} from '../helpers';
 import { BrowserClient, ActionType } from '@amplitude/analytics-core';
-import { filterOutNonTrackableEvents, shouldTrackEvent } from '../helpers';
 import { AMPLITUDE_ELEMENT_CHANGED_EVENT } from '../constants';
 
 export function trackChange({

packages/plugin-autocapture-browser/src/autocapture/track-click.ts

@@ -1,5 +1,5 @@
-import { AllWindowObservables } from 'src/autocapture-plugin';
-import { type evaluateTriggersFn } from 'src/helpers';
+import { AllWindowObservables } from '../autocapture-plugin';
+import { ElementBasedEvent, ElementBasedTimestampedEvent, type evaluateTriggersFn } from '../helpers';
 import { Observable, BrowserClient } from '@amplitude/analytics-core';
 import { filterOutNonTrackableEvents, shouldTrackEvent } from '../helpers';
 import { AMPLITUDE_ELEMENT_CLICKED_EVENT } from '../constants';
@@ -28,7 +28,7 @@ export function trackClicks({
   const clicks: Observable<typeof clickObservableFiltered extends Observable<infer U> ? U : never> =
     clickObservableFiltered;
 
-  return clicks.subscribe((click) => {
+  return clicks.subscribe((click: ElementBasedTimestampedEvent<ElementBasedEvent>) => {
     /* istanbul ignore next */
     amplitude?.track(AMPLITUDE_ELEMENT_CLICKED_EVENT, click.targetElementProperties);
   });

packages/plugin-autocapture-browser/src/autocapture/track-dead-click.ts

@@ -1,4 +1,4 @@
-import { AllWindowObservables } from 'src/frustration-plugin';
+import { AllWindowObservables } from '../frustration-plugin';
 import { BrowserClient, ActionType, merge, asyncMap } from '@amplitude/analytics-core';
 import { ElementBasedTimestampedEvent, filterOutNonTrackableEvents, shouldTrackEvent } from '../helpers';
 import { AMPLITUDE_ELEMENT_DEAD_CLICKED_EVENT } from '../constants';

packages/plugin-autocapture-browser/src/autocapture/track-rage-click.ts

@@ -1,4 +1,4 @@
-import { AllWindowObservables } from 'src/frustration-plugin';
+import { AllWindowObservables } from '../frustration-plugin';
 import {
   BrowserClient,
   asyncMap,
@@ -48,11 +48,11 @@ type RageClickEvent = {
 };
 
 function addCoordinates(regionBox: ClickRegionBoundingBox, click: ClickEvent) {
-  const { clientX, clientY } = click.event as MouseEvent;
-  regionBox.yMin = Math.min(regionBox.yMin ?? clientY, clientY);
-  regionBox.yMax = Math.max(regionBox.yMax ?? clientY, clientY);
-  regionBox.xMin = Math.min(regionBox.xMin ?? clientX, clientX);
-  regionBox.xMax = Math.max(regionBox.xMax ?? clientX, clientX);
+  const { pageX, pageY } = click.event as MouseEvent;
+  regionBox.yMin = Math.min(regionBox.yMin ?? pageY, pageY);
+  regionBox.yMax = Math.max(regionBox.yMax ?? pageY, pageY);
+  regionBox.xMin = Math.min(regionBox.xMin ?? pageX, pageX);
+  regionBox.xMax = Math.max(regionBox.xMax ?? pageX, pageX);
   regionBox.isOutOfBounds =
     regionBox.yMax - regionBox.yMin > RAGE_CLICK_OUT_OF_BOUNDS_THRESHOLD ||
     regionBox.xMax - regionBox.xMin > RAGE_CLICK_OUT_OF_BOUNDS_THRESHOLD;
@@ -71,8 +71,8 @@ function getRageClickAnalyticsEvent(clickWindow: ClickEvent[]) {
     '[Amplitude] End Time': new Date(lastClick.timestamp).toISOString(),
     '[Amplitude] Duration': lastClick.timestamp - firstClick.timestamp,
     '[Amplitude] Clicks': clickWindow.map((click) => ({
-      X: (click.event as MouseEvent).clientX,
-      Y: (click.event as MouseEvent).clientY,
+      X: (click.event as MouseEvent).pageX,
+      Y: (click.event as MouseEvent).pageY,
       Time: click.timestamp,
     })),
     '[Amplitude] Click Count': clickWindow.length,

packages/plugin-autocapture-browser/src/pageActions/matchEventToFilter.ts

@@ -1,5 +1,5 @@
 import type { Filter } from '@amplitude/analytics-core/lib/esm/types/element-interactions';
-import { ElementBasedTimestampedEvent, ElementBasedEvent } from 'src/helpers';
+import { ElementBasedTimestampedEvent, ElementBasedEvent } from '../helpers';
 
 /**
  * Matches an event to a single filter

packages/plugin-autocapture-browser/src/pageActions/triggers.ts

@@ -1,5 +1,5 @@
 import type { ElementInteractionsOptions, LabeledEvent, Trigger } from '@amplitude/analytics-core';
-import type { ElementBasedTimestampedEvent, ElementBasedEvent } from 'src/helpers';
+import type { ElementBasedTimestampedEvent, ElementBasedEvent } from '../helpers';
 import { matchEventToFilter } from './matchEventToFilter';
 import { executeActions } from './actions';
 import type { DataExtractor } from '../data-extractor';