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

Safari iOS doesn't support api.WorkerGlobalScope.languagechange_event #24816

Merged
merged 2 commits into from
Oct 25, 2024

Conversation

ddbeck
Copy link
Collaborator

@ddbeck ddbeck commented Oct 24, 2024

Summary

This was probably never supported.

Test results and supporting details

This is very difficult to test because it requires changing the system language settings, which I can't do in BrowserStack. And I don't have a device that I can directly test with. But I learned a few things:

  • In a window, languagechange fires in contemporary macOS Safari.
  • In a worker, languagechange does not fire in macOS Safari and onlanguagechange doesn't appear to be real (e.g., you can assign arbitrary types to it, unlike the window equivalent).
  • Following the blames shows this data was originally migrated from the wiki. Add compat data for WorkerGlobalScope (Worker API) #982
  • Searching webkit/webkit shows evidence of the window exposure, like tests, but I couldn't find anything even hinting at worker exposure.

Taken together, I'm rather certain that Safari doesn't actually support this and it's spurious data from the wiki days.

Related issues

Discovered reviewing web-platform-dx/web-features#1931

@github-actions github-actions bot added the data:api Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API label Oct 24, 2024
Copy link
Contributor

@caugner caugner left a comment

Choose a reason for hiding this comment

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

Reasoning is plausible.

@caugner caugner changed the title api.WorkerGlobalScope.languagechange_event: drop iOS erroneous support Safari iOS doesn't support api.WorkerGlobalScope.languagechange_event Oct 25, 2024
@caugner caugner merged commit 9bb4102 into mdn:main Oct 25, 2024
6 checks passed
@ddbeck ddbeck deleted the languagechange-event branch October 28, 2024 10:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data:api Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants