Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Maintenance: Update favicon and document title when session is close to expiring #2810

merged 7 commits into from May 1, 2020


Copy link

@kevinrobinson kevinrobinson commented Apr 22, 2020

Who is this PR for?

Educators within remote meetings

What problem does this PR fix?

If someone is running a meeting remotely (eg, MTSS meeting), they start writing a note but then switch to other tabs or browser windows, and not be able to find the original tab they were working in. In that circumstance, there's currently no indication to them that the tab is about to expire (session timeout is currently 30 minutes).

What does this PR do?

This updates the favicon and document title so that there's a visual indication if this is a background tab.

Screenshot (if adding a client-side feature)

This is on IE11, the UX is similar on other browsers as well.

1. in background tab

Screen Shot 2020-04-22 at 12 15 53 PM

2. favicon and document title change

Screen Shot 2020-04-22 at 12 12 45 PM

3. clicking in the tab, the timeout message is visible

Screen Shot 2020-04-22 at 12 13 02 PM

4. clicking revives the session and reset the favicon and title

Screen Shot 2020-04-22 at 12 13 19 PM


Which features or pages does this PR touch?

  • Core

Does this PR use tests to help verify we can deploy these changes quickly and confidently?

  • Manual testing made more sense here

@kevinrobinson kevinrobinson merged commit c3e39e7 into master May 1, 2020
10 checks passed
Copy link
Contributor Author

kevinrobinson commented May 1, 2020


@kevinrobinson kevinrobinson deleted the feature/session-renewal-alert branch May 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
None yet

Successfully merging this pull request may close these issues.

None yet

1 participant