Skip to content

Update Locale to handle synchronous JSON file loading, when possible.#1481

Open
dpvc wants to merge 1 commit into
feature/locale_finish_texfrom
feature/locale-sync
Open

Update Locale to handle synchronous JSON file loading, when possible.#1481
dpvc wants to merge 1 commit into
feature/locale_finish_texfrom
feature/locale-sync

Conversation

@dpvc
Copy link
Copy Markdown
Member

@dpvc dpvc commented May 20, 2026

This PR allows the Locale object to load the localization files synchronously, when the mathjax.asyncLoad method is marked as synchronous. It also fixes the JSDoc issue that I mentioned in the review of #1479.

@dpvc dpvc requested a review from zorkow May 20, 2026 20:38
@dpvc
Copy link
Copy Markdown
Member Author

dpvc commented May 21, 2026

I've added the Locale.throw() method I mentioned in #1479, and made Locale.error() use console.error() rather than throw. I also added Locale.warn() and Locale.log(), and added tests for these, as well as for the synchronous loading.

@dpvc
Copy link
Copy Markdown
Member Author

dpvc commented May 21, 2026

I will need to change this PR in light of our meeting today. So wait with it for now.

@dpvc dpvc force-pushed the feature/locale-sync branch 2 times, most recently from f350e8e to 227d6cb Compare May 23, 2026 10:19
@dpvc
Copy link
Copy Markdown
Member Author

dpvc commented May 23, 2026

OK, Ive reverted this branch to the original purpose of adding the synchronous loading of locale files, when possible. I will do the other changes in a separate PR, so this one can be reviewed at this point.

@dpvc dpvc force-pushed the feature/locale-sync branch 2 times, most recently from 22ff232 to 7f32b1d Compare May 25, 2026 13:30
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.

1 participant