Fix timer slowdown on inactive tab

[mohsin-r]

Related Item(s)

#541

Changes

• Fixed an issue where changing to a different tab would cause the timer on the modal to be displayed incorrectly.

Notes

• Credit and respect to https://github.com/turuslan/HackTimer for the code.

Testing

1. Open the dev site.
2. Load your favourite storyline.
3. Make a few changes.
4. Switch to another tab for a bit (around 1-2 minutes).
5. Go back to your tab without allowing your mouse to enter any part of the editor itself (otherwise the timer will restart).
6. Wait.
7. Verify that the warning modal displays when there are 2 minutes left in the session and you get kicked out with your changes saved successfully at 0 minutes left.
8. Feel free to try other cases. For example, you could confirm extend the session, then switch tabs again, come back and verify things are all good, and so on.

---

This change is 

[github-actions]

Your demo site is ready! 🚀 Visit it here: https://ramp4-pcar4.github.io/storylines-editor/541

[IshavSohal]

Reviewed 8 of 8 files at r1, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @mohsin-r)

[szczz]

Reviewed 5 of 8 files at r1, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @mohsin-r)

---

server/index.js line 601 at r1 (raw file):

    const payload = JSON.stringify(message);
    clients.forEach((client) => {
        if (client.readyState === 1) {

Lets use process.env.SERVER_CURR_ENV !== '#{CURR_ENV}#' ? WebSocket.OPEN : 1 here. I know these evaluate the same but we've tested the socket server with the former so I'd rather leave this as is.

[mohsin-r]

Reviewable status: 7 of 8 files reviewed, 1 unresolved discussion (waiting on @IshavSohal and @szczz)

---

server/index.js line 601 at r1 (raw file):

Previously, szczz (Matt Szczerba) wrote…

Lets use process.env.SERVER_CURR_ENV !== '#{CURR_ENV}#' ? WebSocket.OPEN : 1 here. I know these evaluate the same but we've tested the socket server with the former. so I'd rather leave this as is.

Donethanks.

[yileifeng]

Reviewed 7 of 8 files at r1, 1 of 1 files at r2, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @szczz)

[szczz]

Got redirected at 2 seconds left on the timer and the storyline I was using seems to be permanently locked. Which build is on dev at the moment?

Reviewed 2 of 8 files at r1, 1 of 1 files at r2, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @mohsin-r)

[spencerwahl]

Reviewed 7 of 8 files at r1, 1 of 1 files at r2, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @mohsin-r)