@khanacademy/math-input 17.0.5 → 17.0.7

package/dist/index.js

@@ -9,7 +9,7 @@ var wonderStuffCore = require('@khanacademy/wonder-stuff-core');
 var aphrodite = require('aphrodite');
 var React = require('react');
 var ReactDOM = require('react-dom');
-var $ = require('jquery');
+var mathjaxRenderer = require('@khanacademy/mathjax-renderer');
 var MathQuill = require('mathquill');
 var wonderBlocksCore = require('@khanacademy/wonder-blocks-core');
 var Clickable = require('@khanacademy/wonder-blocks-clickable');
@@ -41,14 +41,13 @@ var Color__default = /*#__PURE__*/_interopDefaultLegacy(Color);
 var i18n__namespace = /*#__PURE__*/_interopNamespace(i18n);
 var React__namespace = /*#__PURE__*/_interopNamespace(React);
 var ReactDOM__default = /*#__PURE__*/_interopDefaultLegacy(ReactDOM);
-var $__default = /*#__PURE__*/_interopDefaultLegacy($);
 var MathQuill__default = /*#__PURE__*/_interopDefaultLegacy(MathQuill);
 var Clickable__default = /*#__PURE__*/_interopDefaultLegacy(Clickable);
 var PropTypes__default = /*#__PURE__*/_interopDefaultLegacy(PropTypes);
 
 // This file is processed by a Rollup plugin (replace) to inject the production
 const libName = "@khanacademy/math-input";
-const libVersion = "17.0.5";
+const libVersion = "17.0.7";
 perseusCore.addLibraryVersionToPerseusDebug(libName, libVersion);
 
 function _extends() {
@@ -371,6 +370,49 @@ let CursorContext = /*#__PURE__*/function (CursorContext) {
   return CursorContext;
 }({});
 
+const DecimalSeparator = {
+  COMMA: ",",
+  PERIOD: "."
+};
+
+// NOTES(kevinb):
+// - In order to get the correct decimal separator for the current locale,
+//   the locale must bet set using `setLocale(kaLocale)` which can be
+//   imported from wonder-blocks-i18n.
+// - Some languages/locales use different decimal separators than the ones
+//   listed here.  Much of the Arab world uses U+066C.
+const decimalSeparator = i18n.getDecimalSeparator() === "," ? DecimalSeparator.COMMA : DecimalSeparator.PERIOD;
+const CDOT_ONLY = ["az", "cs", "da", "de", "hu", "hy", "kk", "ky", "lt", "lv", "nb", "sk", "sr", "sv", "uz"];
+const TIMES_ONLY = ["fr", "tr", "pt-pt"];
+
+/**
+ * convertDotToTimes (aka `times`) is an option the content creators have to
+ * use × (TIMES) rather than · (CDOT) for multiplication (for younger learners).
+ * Some locales _only_ use one or the other for all multiplication regardless
+ * of age.
+ *
+ * convertDotToTimesByLocale overrides convertDotToTimes for those locales.
+ *
+ * @param {boolean} convertDotToTimes - the setting set by content creators
+ * @returns {boolean} - true to convert to × (TIMES), false to use · (CDOT)
+ */
+function convertDotToTimesByLocale(convertDotToTimes) {
+  const locale = i18n.getLocale();
+  if (CDOT_ONLY.includes(locale)) {
+    return false;
+  }
+  if (TIMES_ONLY.includes(locale)) {
+    return true;
+  }
+  return convertDotToTimes;
+}
+
+/**
+ * Use this to avoid running code that should not run in Jest.
+ **/
+const inJest = typeof process !== "undefined" && !!process?.env?.JEST_WORKER_ID;
+// Explicitly checking for undefined because Cypress throws an error
+
 // We only need one MathQuill instance (referred to as MQ in the docs)
 // and that contains some MQ constants and the MathField constructor
 const mathQuillInstance = MathQuill__default["default"].getInterface(3);
@@ -422,6 +464,12 @@ function createMathField(container, configCallback) {
   const mathField = mathQuillInstance.MathField(container, config)
   // translated in ./math-input.tsx
   .setAriaLabel(i18n__namespace._("Math input box"));
+
+  // We should avoid running SpeechRuleEngine.setup() in Jest. It makes an
+  //   HTTP request to fetch non-english speech rules, and cannot be easily
+  //   mocked in consuming packages now that we do not bundle source code.
+  //   When it eventually times out, it will cause arbitrary test failures.
+  !inJest && mathjaxRenderer.SpeechRuleEngine.setup().then(SRE => mathField.setMathspeakOverride(SRE.texToSpeech));
   return mathField;
 }
 
@@ -440,9 +488,12 @@ let MathFieldActionType = /*#__PURE__*/function (MathFieldActionType) {
 }({});
 
 /**
- * The MathQuill MathField Cursor
- * it's not part of the public API for MathQuill,
- * we reach into the internals to get it
+ * The MathQuill API (see mathuill.d.ts) does not include types
+ * for cursor() and controller(), and adding these types there
+ * in the MathQuill repo causes unexpected conflicts with other types.
+ *
+ * We don't want to use the cursor and controller default type `any`
+ * so we declare the types here.
  */
 
 const Numerals = ["0", "1", "2", "3", "4", "5", "6", "7", "8", "9"];
@@ -857,43 +908,6 @@ function handleBackspace(mathField) {
   }
 }
 
-const DecimalSeparator = {
-  COMMA: ",",
-  PERIOD: "."
-};
-
-// NOTES(kevinb):
-// - In order to get the correct decimal separator for the current locale,
-//   the locale must bet set using `setLocale(kaLocale)` which can be
-//   imported from wonder-blocks-i18n.
-// - Some languages/locales use different decimal separators than the ones
-//   listed here.  Much of the Arab world uses U+066C.
-const decimalSeparator = i18n.getDecimalSeparator() === "," ? DecimalSeparator.COMMA : DecimalSeparator.PERIOD;
-const CDOT_ONLY = ["az", "cs", "da", "de", "hu", "hy", "kk", "ky", "lt", "lv", "nb", "sk", "sr", "sv", "uz"];
-const TIMES_ONLY = ["fr", "tr", "pt-pt"];
-
-/**
- * convertDotToTimes (aka `times`) is an option the content creators have to
- * use × (TIMES) rather than · (CDOT) for multiplication (for younger learners).
- * Some locales _only_ use one or the other for all multiplication regardless
- * of age.
- *
- * convertDotToTimesByLocale overrides convertDotToTimes for those locales.
- *
- * @param {boolean} convertDotToTimes - the setting set by content creators
- * @returns {boolean} - true to convert to × (TIMES), false to use · (CDOT)
- */
-function convertDotToTimesByLocale(convertDotToTimes) {
-  const locale = i18n.getLocale();
-  if (CDOT_ONLY.includes(locale)) {
-    return false;
-  }
-  if (TIMES_ONLY.includes(locale)) {
-    return true;
-  }
-  return convertDotToTimes;
-}
-
 function handleLeftArrow(mathField, cursor) {
   // If we're inside a function, and just after the left parentheses, we
   // need to skip the entire function name, rather than move the cursor
@@ -1358,7 +1372,7 @@ class MathWrapper {
         const controller = this.mathField.controller();
         const pageX = x - document.body.scrollLeft;
         const pageY = y - document.body.scrollTop;
-        controller.seek($__default["default"](el), pageX, pageY).cursor.startSelection();
+        controller.seek(el, pageX, pageY).cursor.startSelection();
 
         // Unless that would leave us mid-command, in which case, we
         // need to adjust and place the cursor inside the parens