[Replay Details - UI] Reflect pause/resume for mobile playback

getsentry / sentry

Developer-first error tracking and performance monitoring

https://sentry.io

Other

36.96k stars 3.97k forks source link

[Replay Details - UI] Reflect pause/resume for mobile playback #69249

Open romtsn opened 1 week ago

romtsn commented 1 week ago

Because we pause/resume replay when an app goes foreground/background, we should reflect this somehow on the playback (e.g. gray out the video view and also the replay timeline part of it). We could do that based off of the breadcrumb that the SDKs send.

Currently these breadcrumbs have:

category: navigation
data["state"]: foreground|background

Open question: is this breadcrumb data sufficient or should we introduce a new category/type or add more data?

michellewzhang commented 1 week ago

ui.blur & ui.focus exist for web replays but we don't display them. we could reuse these for mobile; they correlate to background & foreground
would be helpful to show, visually in the timeline and possibly an overlay on the video, where there are "gaps" (which likely result from switching from foreground/background)
- would require a large amount of refactoring wrt the scrubber (e.g. we'd probably want to disable clicking into a gap)

will need to chat with @Jesse-Box on design thoughts