@khanacademy/perseus-editor 2.0.1 → 2.0.3

package/dist/index.js

@@ -1056,7 +1056,6 @@ class Editor extends React__namespace.Component {
       }, function () {
         perseus.Util.textarea.moveCursor(
         // findDOMNode can return Text but we know this is Element
-        // $FlowFixMe[incompatible-call]
         // @ts-expect-error [FEI-5003] - TS2345 - Argument of type 'Element | Text | null' is not assignable to parameter of type 'HTMLTextAreaElement'.
         textarea,
         // We want to put the cursor after the widget
@@ -1901,7 +1900,6 @@ class ArticleEditor extends React__namespace.Component {
 }
 _defineProperty(ArticleEditor, "defaultProps", {
   contentPaths: [],
-  // $FlowFixMe[incompatible-exact]
   json: [{}],
   mode: "edit",
   screen: "desktop",
@@ -4610,7 +4608,7 @@ class ModeDropdown extends React__namespace.Component {
     _defineProperty(this, "_handleSelectMode", event => {
       if (this.props.onChange) {
         // event.target.value corresponds to the options' values below which
-        // are limited to EditorMode, but flow doesn't know that so we have
+        // are limited to EditorMode, but TypeScript doesn't know that so we have
         // to cast through any here.
         const value = event.target.value;
         this.props.onChange(value);
@@ -4682,7 +4680,6 @@ const Header = _ref => {
   } = _ref;
   const headerLevel = Math.min(depth, 5) + 1;
   const HeaderTag = "h".concat(headerLevel);
-  // @ts-expect-error [FEI-5003] - TS2559 - Type '{ children?: ReactNode; }' has no properties in common with type 'IntrinsicAttributes'.
   return /*#__PURE__*/React__namespace.createElement(HeaderTag, props);
 };
 
@@ -4846,7 +4843,7 @@ const NodeContent = props => {
 
   // All uses of `data` have been cast through any which isn't safe. This was
   // done to avoid introducing new logic which may have resulted in a change
-  // of behavior.  Also, there doesn't appear to be a way to tell flow the
+  // of behavior.  Also, there doesn't appear to be a way to tell TypeScript the
   // difference between and `TagsNode` and `ItemArrayNode`, see
   // perseus-all-package/multi-items/item-types.js.
   if (shape.type === "content") {
@@ -4925,12 +4922,6 @@ const withStickiness = WrappedComponent => {
     render() {
       return (
         /*#__PURE__*/
-        /**
-         * $FlowFixMe[incompatible-exact]
-         * TODO(somewhatabstract, JIRA-XXXX):
-         * Cannot create WrappedComponent element because inexact props
-         * [1] is incompatible with exact WithStickiness [2].
-         */
         // @ts-expect-error [FEI-5003] - TS2322 - Type 'Readonly<Config> & { sticky: boolean; children?: ReactNode; }' is not assignable to type 'IntrinsicAttributes & LibraryManagedAttributes<Component, PropsWithChildren<WithStickiness<Config>>>'.
         React__namespace.createElement(WrappedComponent, _extends({}, this.props, {
           sticky: this.state.sticky
@@ -4959,9 +4950,6 @@ const ItemNodeContent = withStickiness(props => {
     className: aphrodite.css(sticky && styles$c.sticky)
   },
   /*#__PURE__*/
-  // $FlowFixMe[incompatible-exact]: {...data} pass too many props
-  // $FlowFixMe[incompatible-type]
-  // $FlowFixMe[prop-missing]
   // @ts-expect-error [FEI-5003] - TS2769 - No overload matches this call.
   React__namespace.createElement(Editor, _extends({}, data, {
     onChange: newVal => actions.mergeValueAtPath(path, newVal),
@@ -5065,10 +5053,7 @@ const ArrayNodeContent = props => {
       onClick: () => actions.moveArrayElementUp(subpath)
     }, /*#__PURE__*/React__namespace.createElement("div", {
       className: aphrodite.css(styles$c.verticalFlip)
-    }, /*#__PURE__*/React__namespace.createElement(InlineIcon$7, iconChevronDown)))),
-    // $FlowFixMe[unsafe-addition]
-    // $FlowFixMe[prop-missing]
-    i < data.length - 1 && /*#__PURE__*/React__namespace.createElement("div", {
+    }, /*#__PURE__*/React__namespace.createElement(InlineIcon$7, iconChevronDown)))), i < data.length - 1 && /*#__PURE__*/React__namespace.createElement("div", {
       key: "moveArrayElementDown",
       className: aphrodite.css(styles$c.control)
     }, /*#__PURE__*/React__namespace.createElement(SimpleButton, {
@@ -5705,14 +5690,10 @@ class PairEditor$1 extends React__namespace.Component {
     return /*#__PURE__*/React__namespace.createElement("fieldset", {
       className: "pair-editor"
     }, /*#__PURE__*/React__namespace.createElement("label", null, "Name:", " ", /*#__PURE__*/React__namespace.createElement(BlurInput, {
-      value: this.props.name
-      // $FlowFixMe[incompatible-type] single-param call returns a callback
-      ,
+      value: this.props.name,
       onChange: this.change("name")
     })), /*#__PURE__*/React__namespace.createElement("label", null, " ", "Value:", " ", /*#__PURE__*/React__namespace.createElement(BlurInput, {
-      value: this.props.value
-      // $FlowFixMe[incompatible-type] single-param call returns a callback
-      ,
+      value: this.props.value,
       onChange: this.change("value")
     })));
   }
@@ -6869,10 +6850,7 @@ class ExpressionEditor extends React__namespace.Component {
     })), /*#__PURE__*/React__namespace.createElement(InfoTip$j, null, /*#__PURE__*/React__namespace.createElement("p", null, "Single-letter variables listed here will be interpreted as functions. This let us know that f(x) means \"f of x\" and not \"f times x\"."))), /*#__PURE__*/React__namespace.createElement("div", null, /*#__PURE__*/React__namespace.createElement("div", null, "Button sets:"), buttonSetChoices), this.state.isTex && /*#__PURE__*/React__namespace.createElement(TexButtons, {
       className: "math-input-buttons",
       sets: this.props.buttonSets,
-      convertDotToTimes: this.props.times
-      // onInsert is typed as `(string | ((input: $FlowFixMe) => void)) => void`
-      // $FlowFixMe[incompatible-type]
-      ,
+      convertDotToTimes: this.props.times,
       onInsert: this.handleTexInsert
     }), /*#__PURE__*/React__namespace.createElement("h3", {
       className: "expression-editor-h3"
@@ -7775,7 +7753,6 @@ class GrapherEditor extends React__namespace.Component {
       // NOTE(jeremy): This editor doesn't pass in a bunch of
       // standard props that the Renderer provides normally (eg.
       // alignment, findWidgets, etc).
-      // $FlowFixMe[prop-missing]
       // @ts-expect-error [FEI-5003] - TS2769 - No overload matches this call.
       React__namespace.createElement(Grapher, _extends({}, graphProps, {
         apiOptions: this.props.apiOptions,
@@ -7915,14 +7892,10 @@ class PairEditor extends React__namespace.Component {
   }
   render() {
     return /*#__PURE__*/React__namespace.createElement("fieldset", null, /*#__PURE__*/React__namespace.createElement("label", null, "Name:", /*#__PURE__*/React__namespace.createElement(BlurInput, {
-      value: this.props.name
-      // $FlowFixMe[incompatible-type] single-param call returns a callback
-      ,
+      value: this.props.name,
       onChange: this.change("name")
     })), /*#__PURE__*/React__namespace.createElement("label", null, "Value:", /*#__PURE__*/React__namespace.createElement(BlurInput, {
-      value: this.props.value
-      // $FlowFixMe[incompatible-type] single-param call returns a callback
-      ,
+      value: this.props.value,
       onChange: this.change("value")
     })));
   }
@@ -8409,9 +8382,7 @@ class InputNumberEditor extends React__namespace.Component {
       disabled: !this.props.inexact,
       defaultValue: this.props.maxError,
       onBlur: e => {
-        const ans = "" + (
-        // $FlowFixMe[incompatible-type]
-        perseus.Util.firstNumericalParse(e.target.value) || 0);
+        const ans = "" + (perseus.Util.firstNumericalParse(e.target.value) || 0);
         e.target.value = ans;
         this.props.onChange({
           maxError: ans
@@ -9812,7 +9783,6 @@ const InteractiveGraphEditor = createReactClass__default["default"]({
       // There are a bunch of props that renderer.jsx passes to widgets via
       // getWidgetProps() and widget-container.jsx that the editors don't
       // bother passing.
-      // $FlowFixMe[prop-missing]
       // @ts-expect-error [FEI-5003] - TS2769 - No overload matches this call.
       React__namespace.createElement(InteractiveGraph, _extends({}, graphProps, {
         containerSizeClass: sizeClass,
@@ -10713,9 +10683,6 @@ class Option extends React__namespace.Component {
       ariaLabel
     } = this.props;
     return /*#__PURE__*/React__namespace.createElement("button", {
-      // TODO(mdr): We found a new Flow error when upgrading:
-      //     "node (null) This type is incompatible with this.node (HTMLDivElement)"
-      // $FlowFixMe[incompatible-type](0.52.0->0.53.0)
       // @ts-expect-error [FEI-5003] - TS2322 - Type 'HTMLButtonElement | null' is not assignable to type 'HTMLDivElement'.
       ref: node => this.node = node,
       value: value,
@@ -10726,11 +10693,6 @@ class Option extends React__namespace.Component {
       ,
       onClick: value => {
         if (!disabled && onClick) {
-          // TODO(mdr): We found a new Flow error here when upgrading
-          //     from 0.35.0 to 0.52.0: "value (unused function
-          //     argument)". This comment was automatically added to
-          //     this file, to suppress the error for now. Please
-          //     consider fixing!
           // @ts-expect-error [FEI-5003] - TS2554 - Expected 0 arguments, but got 1.
           onClick(value);
         }
@@ -10991,7 +10953,6 @@ class Marker extends React__namespace.Component {
     }), /*#__PURE__*/React__namespace.createElement(OptionGroup, {
       onSelected: this.handleSelectAnswer
       // TODO(WB-1096): make selectedValues immutable in wonder-blocks
-      // $FlowFixMe[incompatible-type]
       // @ts-expect-error [FEI-5003] - TS2769 - No overload matches this call.
       ,
       selectedValues: answers
@@ -12411,7 +12372,6 @@ const NumericInputEditor = createReactClass__default["default"]({
       size: "normal",
       coefficient: false,
       labelText: "",
-      multipleNumberInput: false,
       rightAlign: false
     };
   },
@@ -12605,14 +12565,6 @@ const NumericInputEditor = createReactClass__default["default"]({
     return /*#__PURE__*/React__namespace.createElement("div", {
       className: "perseus-input-number-editor"
     }, /*#__PURE__*/React__namespace.createElement("div", {
-      ref: e => this.multInputOption = e
-    }, /*#__PURE__*/React__namespace.createElement("select", {
-      onChange: this.onMultipleInputChange
-    }, /*#__PURE__*/React__namespace.createElement("option", {
-      value: "simple-numeric-input"
-    }, "Ask for one correct solution"), /*#__PURE__*/React__namespace.createElement("option", {
-      value: "multiple-numeric-input"
-    }, "Ask for all correct solutions"))), /*#__PURE__*/React__namespace.createElement("div", {
       className: "ui-title"
     }, "User input"), /*#__PURE__*/React__namespace.createElement("div", {
       className: "msg-title"
@@ -12658,18 +12610,6 @@ const NumericInputEditor = createReactClass__default["default"]({
       simplify: newStatus === "correct" ? "required" : "accepted"
     });
   },
-  onMultipleInputChange: function (event) {
-    const newOption = event.target.value;
-    if (newOption === "multiple-numeric-input") {
-      this.props.onChange({
-        multipleNumberInput: true
-      });
-    } else {
-      this.props.onChange({
-        multipleNumberInput: false
-      });
-    }
-  },
   updateAnswer: function (choiceIndex, update) {
     if (!___default["default"].isObject(update)) {
       return ___default["default"].partial((choiceIndex, key, value) => {
@@ -14815,7 +14755,6 @@ class TransformationsShapeEditor extends React__namespace.Component {
     });
     _defineProperty(this, "updateCoords", () => {
       this.props.onChange({
-        // $FlowFixMe[prop-missing]
         // @ts-expect-error [FEI-5003] - TS2339 - Property 'shape' does not exist on type 'TransformationsShapeEditor'.
         shape: this.shape.toJSON()
       });