Skip to content

Conversation

@lutien
Copy link
Contributor

@lutien lutien commented Oct 15, 2025

Tests to illustrate how the API will work for the PR in the html spec.
More tests will be updated during work on https://bugzilla.mozilla.org/show_bug.cgi?id=1994396.

Copy link
Contributor

@juliandescottes juliandescottes left a comment

Choose a reason for hiding this comment

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

Thanks Sasha, I feel like there is something wrong with the usage of the fixture to check the default values. It probably predates the PR, but can you check?

Copy link
Contributor

@juliandescottes juliandescottes left a comment

Choose a reason for hiding this comment

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

Thanks Sasha, LGTM, one minor suggestion.

@lutien
Copy link
Contributor Author

lutien commented Oct 30, 2025

Closing in favour of #55768.

@lutien lutien closed this Oct 30, 2025
@lutien lutien deleted the add-tests-for-navigator-language-to-setLocaleOverride branch October 30, 2025 16:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants