Skip to content

Produce message if locale isn't set when using direct access to MathJax modules.#1484

Merged
dpvc merged 2 commits into
feature/localefrom
feature/locale-message
Jun 11, 2026
Merged

Produce message if locale isn't set when using direct access to MathJax modules.#1484
dpvc merged 2 commits into
feature/localefrom
feature/locale-message

Conversation

@dpvc

@dpvc dpvc commented May 25, 2026

Copy link
Copy Markdown
Member

This PR introduces an error message about not having set up localization, if that is the case. This applies only to node applications that use direct access to the MathJax modules (since the component-based apps in node or the browser will have Locale.setLocale() performed automatically). In this PR, when a MathDocument is created, the Locale is checked and it setLocale() has never been called, the message is generated. When we have a URL to point to, it should be added to the message.

@dpvc dpvc requested a review from zorkow May 25, 2026 12:23

@zorkow zorkow left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm.
I think this is the branch I need to merge for PR #1496

Base automatically changed from fix/test-format to feature/locale June 11, 2026 18:25
@dpvc dpvc merged commit dc214ab into feature/locale Jun 11, 2026
@dpvc dpvc deleted the feature/locale-message branch June 11, 2026 18:30
@dpvc dpvc added this to the v4.2 milestone Jun 11, 2026
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