replayio / website

Code for our demo site on replay.io/demo
https://website-git-master.recordreplay.vercel.app
1 stars 1 forks source link

Add recording prompt for non-mac Replay users #162

Closed ryanjduffy closed 3 years ago

ryanjduffy commented 3 years ago

Adds a prompt to start recording when in Replay on non-mac platforms.

image

vercel[bot] commented 3 years ago

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/recordreplay/website/8oJdRFBfFkyKde2hW3AvxJqz2TWC
✅ Preview: https://website-git-ryan-add-nonmac-copy-recordreplay.vercel.app

ryanjduffy commented 3 years ago

Fixes https://github.com/RecordReplay/devtools/issues/4117

ryanjduffy commented 3 years ago

@jonbell-lot23 image

ryanjduffy commented 3 years ago

i'll add that it fades in after loading to help draw attention to it.

ryanjduffy commented 3 years ago

Going to merge but we can easily update further now that we have CD in place

jonbell-lot23 commented 3 years ago

Looks good, thanks!

We should consider a further design where the demo doesn't actually load until after the page reloads. I know we can't tell when the page is being recorded, but we do at least know that no one on a non-Mac is going to end up on this page with a recorder going the first time. To consider for a later iteration.

ryanjduffy commented 3 years ago

:+1: Not sure yet how we'll do that but I get the idea.

jonbell-lot23 commented 3 years ago

If it were me I'd set a little cookie so on the first time someone is told

  1. HI PRESS RECORD

And then when the page reloads, the cookie spots itself, and then can show the second bit:

  1. MAKE SURE YOU'RE RECORDING, THEN CLICK THESE BACK TO THE FUTURE DUDES