@khanacademy/math-input 20.0.2 → 20.1.0

package/dist/components/input/math-input.d.ts

@@ -3,6 +3,7 @@ import { MathInputI18nContext } from "../i18n-context";
 import type { KeypadAPI, KeypadContextType } from "../../types";
 type Props = {
     keypadElement?: KeypadAPI;
+    ariaLabel: string;
     onBlur: () => void;
     onChange: (value: string, callback: any) => void;
     onFocus: () => void;

package/dist/components/input/math-wrapper.d.ts

@@ -10,7 +10,7 @@ declare class MathWrapper {
     mathField: MathFieldInterface;
     callbacks: any;
     mobileKeyTranslator: Record<Key, MathFieldUpdaterCallback>;
-    constructor(element: any, strings: MathInputStrings, locale: string, callbacks?: {});
+    constructor(mathFieldMount: any, ariaLabel: string, strings: MathInputStrings, locale: string, callbacks?: {});
     focus(): void;
     blur(): void;
     /**

package/dist/es/index.js

@@ -15,7 +15,7 @@ import PropTypes from 'prop-types';
 
 // This file is processed by a Rollup plugin (replace) to inject the production
 const libName = "@khanacademy/math-input";
-const libVersion = "20.0.2";
+const libVersion = "20.1.0";
 addLibraryVersionToPerseusDebug(libName, libVersion);
 
 function _extends() {
@@ -1402,12 +1402,13 @@ const getKeyTranslator = locale => ({
  * from MathQuill changes.
  */
 class MathWrapper {
-  constructor(element, strings, locale, callbacks = {}) {
+  constructor(mathFieldMount, ariaLabel, strings, locale, callbacks = {}) {
+    var _this$mathField;
     this.mathField = void 0;
     // MathQuill MathField input
     this.callbacks = void 0;
     this.mobileKeyTranslator = void 0;
-    this.mathField = createMathField(element, locale, strings, () => {
+    this.mathField = createMathField(mathFieldMount, locale, strings, () => {
       return {
         // use a span instead of a textarea so that we don't bring up the
         // native keyboard on mobile when selecting the input
@@ -1416,6 +1417,7 @@ class MathWrapper {
         }
       };
     });
+    (_this$mathField = this.mathField) == null || _this$mathField.setAriaLabel(ariaLabel);
     this.callbacks = callbacks;
     this.mobileKeyTranslator = _extends({}, getKeyTranslator(locale), {
       // note(Matthew): our mobile backspace logic is really complicated
@@ -2179,7 +2181,7 @@ class MathInput extends React.Component {
   }
   componentDidMount() {
     this._isMounted = true;
-    this.mathField = new MathWrapper(this._mathContainer, this.context.strings, this.context.locale, {
+    this.mathField = new MathWrapper(this._mathContainer, this.props.ariaLabel, this.context.strings, this.context.locale, {
       onCursorMove: cursor => {
         // TODO(charlie): It's not great that there is so much coupling
         // between this keypad and the input behavior. We should wrap
@@ -2609,11 +2611,25 @@ function TabbarItem(props) {
   } = props;
   const tabRef = useRef(null);
   useEffect(() => {
+    let timeout;
     if (role === "tab" && focus) {
-      var _tabRef$current;
-      // Move element into view when it is focused
-      tabRef == null || (_tabRef$current = tabRef.current) == null || _tabRef$current.focus();
+      /**
+       * When tabs are within a WonderBlocks Popover component, the
+       * manner in which the component is rendered and moved causes
+       * focus to snap to the bottom of the page on first focus.
+       *
+       * This timeout moves around that by delaying the focus enough
+       * to wait for the WonderBlock Popover to move to the correct
+       * location and scroll the user to the correct location.
+       * */
+      timeout = setTimeout(() => {
+        if (tabRef != null && tabRef.current) {
+          // Move element into view when it is focused
+          tabRef == null || tabRef.current.focus();
+        }
+      }, 0);
     }
+    return () => clearTimeout(timeout);
   }, [role, focus, tabRef]);
   return /*#__PURE__*/React.createElement(Clickable, {
     onClick: onClick,