Skip to content

Comments

Implemented the title change to "✅Speed Test Completed" to alert the user in case of a Tab change by the user#88

Closed
lakshay776 wants to merge 1 commit intom-lab:mainfrom
lakshay776:issue37
Closed

Implemented the title change to "✅Speed Test Completed" to alert the user in case of a Tab change by the user#88
lakshay776 wants to merge 1 commit intom-lab:mainfrom
lakshay776:issue37

Conversation

@lakshay776
Copy link
Contributor

  • Implemented a title change to "✅Speed Test Completed" when the test is completed to notify the user of a test completion when they are on another tab
  • This was done by checking if the document.hidden is true which checks if the user has opened the speed test tab or not
  • if it is hidden then the title is changed otherwise it is rechecked and reverted to the original title
if (document.hidden) {
        var currentTitle = document.title;
        document.title = '✅ Speed Test Completed';
        document.addEventListener('visibilitychange', function onReturn() {
          if (!document.hidden) {
            document.title = currentTitle;
            document.removeEventListener('visibilitychange', onReturn);
          }
        });
      }

fixes #37

@lakshay776
Copy link
Contributor Author

@robertodauria I have implemented the feature. Kindly review the PR and give your feedback!

@robertodauria
Copy link
Contributor

robertodauria commented Feb 20, 2026

Thanks for the contribution. Closing this as it is superseded by #83, which rewrites the codebase removing Angular, jQuery, Gulp, and the old app/ directory structure. The files modified by this PR either no longer exist after that change or the change has already been made.

We aim to merge #83 next week, which will give contributors a stable, modern base to work from. Thoughtful, targeted contributions that build on where the project is heading are much more valuable. We'd encourage you to review #83 and open any future PRs on top of that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Consider adding a "test completed" message

2 participants