PostHog / posthog-js

Send usage data from your web app or site to PostHog, with autocapture.
https://posthog.com/docs/libraries/js
Other
299 stars 125 forks source link

fix: get computed style error #1433

Closed pauldambra closed 1 month ago

pauldambra commented 1 month ago

see https://posthog.sentry.io/issues/5858293571/events/latest/?project=1899813&query=is%3Aunresolved+issue.priority%3A%5Bhigh%2C+medium%5D+getComputedStyle&referrer=latest-event&statsPeriod=90d&stream_index=0&utc=true

and https://posthoghelp.zendesk.com/agent/tickets/17649

we very infrequently see a Window.getComputedStyle: Argument 1 does not implement interface Element. error.

there are some small differences between similar routes in heatmaps and autocapture

the error seems to be in autocapture, so let's make heatmaps safer

vercel[bot] commented 1 month ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
posthog-js ✅ Ready (Inspect) Visit Preview Sep 23, 2024 9:42pm
github-actions[bot] commented 1 month ago

Size Change: -378 B (-0.01%)

Total Size: 2.79 MB

Filename Size Change
dist/array.full.js 350 kB -42 B (-0.01%)
dist/array.full.no-external.js 348 kB -42 B (-0.01%)
dist/array.js 165 kB -42 B (-0.03%)
dist/array.no-external.js 164 kB -42 B (-0.03%)
dist/main.js 165 kB -42 B (-0.03%)
dist/module.full.js 349 kB -42 B (-0.01%)
dist/module.full.no-external.js 348 kB -42 B (-0.01%)
dist/module.js 165 kB -42 B (-0.03%)
dist/module.no-external.js 164 kB -42 B (-0.03%)
ℹ️ View Unchanged | Filename | Size | | :--- | :---: | | `dist/all-external-dependencies.js` | 191 kB | | `dist/exception-autocapture.js` | 10.5 kB | | `dist/external-scripts-loader.js` | 2.35 kB | | `dist/recorder-v2.js` | 111 kB | | `dist/recorder.js` | 111 kB | | `dist/surveys-preview.js` | 59.8 kB | | `dist/surveys.js` | 66 kB | | `dist/tracing-headers.js` | 8.36 kB | | `dist/web-vitals.js` | 10.3 kB |

compressed-size-action

sentry-io[bot] commented 1 month ago

Sentry Issue: POSTHOG-1KPQ