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

feat(web): Support Safari >= 16.0 again #1348

Merged
merged 1 commit into from
Dec 20, 2023

Conversation

corneliusroemer
Copy link
Member

The reason we deprecated Safari was that it didn't support shared web workers Since v16.0 shared web workers are supported so we no longer need to show the browser warning Alternatively, we could discourage less loudly, saying that Safari is not tested (for >=16) while still erroring for lower versions.

The reason we deprecated Safari was that it didn't support shared web workers
Since v16.0 shared web workers are supported so we no longer need to show the browser warning 
Alternatively, we could discourage less loudly, saying that Safari is not tested (for >=16) while still erroring for lower versions.
Copy link

vercel bot commented Dec 18, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
nextclade ✅ Ready (Inspect) Visit Preview Dec 18, 2023 5:37pm

@ivan-aksamentov
Copy link
Member

@corneliusroemer There has been problems beyond web workers, for example with CSS. Do the /results and /tree pages look reasonable on Safari >= 16?

@corneliusroemer
Copy link
Member Author

Everything looks good on Safari 17.2 - I can keep using it there for a bit to see how robust it is.

image image

@ivan-aksamentov ivan-aksamentov merged commit df21391 into master Dec 20, 2023
17 checks passed
@ivan-aksamentov ivan-aksamentov deleted the support-safari-16-again branch December 20, 2023 00:33
ivan-aksamentov added a commit that referenced this pull request Mar 5, 2024
Followup of #1348

There are reports that WebWorkers don't work in Nextclade on Safari 16.3.

I was able to confirm that they do work on 16.5, so I am setting this version as the lowest supported. I don't have access to versions lower than 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.

2 participants