Skip to content

Check if serviceWorker is supported#9

Open
tutacat wants to merge 2 commits intoOffroaders123:mainfrom
tutacat:patch-1
Open

Check if serviceWorker is supported#9
tutacat wants to merge 2 commits intoOffroaders123:mainfrom
tutacat:patch-1

Conversation

@tutacat
Copy link
Copy Markdown

@tutacat tutacat commented Aug 19, 2024

Tested working on gecko/firefox and blink/chromium

Copy link
Copy Markdown
Author

@tutacat tutacat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oops, newline

@Offroaders123
Copy link
Copy Markdown
Owner

Thanks for the contribution!

I think I didn't add the check for it initially because it seems like all modern browsers support Service Workers nowadays. Is there another situation where navigator.serviceWorker might not be declared?

Information about it does seem to show that the check is used commonplace, however much of the original documentation for Service Workers seems to be from 2016-2018.

I could see a take of wanting to use Dovetail on older browsers that don't support Service Workers, but I'm also not sure how much marketshare those realistic have anymore. Dovetail's backing relies on fairly modern APIs that are in modern browsers too, so much of the actual app wouldn't work even if the Service Worker call didn't break the whole site.

Thanks again for contributing this, I don't mean to be over analyzing this, I just want to see if there's any more info I might not know about whether the check is still viable nowadays. Always something to learn from 😁

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.

2 participants