@khanacademy/perseus-editor 2.5.2 → 2.6.1

package/CHANGELOG.md

@@ -1,5 +1,27 @@
 # @khanacademy/perseus-editor
 
+## 2.6.1
+
+### Patch Changes
+
+-   89f54569: Types and tests for Plotter and Plotter-Editor
+-   89f54569: Bugfix to hide drag instructions in static plotter
+-   Updated dependencies [89f54569]
+-   Updated dependencies [ea9dac75]
+-   Updated dependencies [89f54569]
+    -   @khanacademy/perseus@7.2.1
+
+## 2.6.0
+
+### Minor Changes
+
+-   e08f85ee: Add testing and improve error logging in the image editor
+
+### Patch Changes
+
+-   Updated dependencies [e08f85ee]
+    -   @khanacademy/perseus@7.2.0
+
 ## 2.5.2
 
 ### Patch Changes

package/dist/es/index.js

@@ -18371,7 +18371,7 @@ mhchemParser.stateMachines = {
         }
       },
       oxidation$: {
-        "0": {
+        0: {
           action_: "oxidation-output"
         }
       },
@@ -18438,7 +18438,7 @@ mhchemParser.stateMachines = {
           action_: ["output", "d="],
           nextState: "d"
         },
-        "3": {
+        3: {
           action_: ["sb=false", "output", "operator"],
           nextState: "0"
         }
@@ -18532,7 +18532,7 @@ mhchemParser.stateMachines = {
           }],
           nextState: "3"
         },
-        "3": {
+        3: {
           action_: {
             type_: "bond",
             option: "-"
@@ -18632,7 +18632,7 @@ mhchemParser.stateMachines = {
         a: {
           nextState: "as"
         },
-        "0": {
+        0: {
           action_: "sb=false"
         },
         "1|2": {
@@ -19243,12 +19243,12 @@ mhchemParser.stateMachines = {
         "*": {}
       },
       "1/2$": {
-        "0": {
+        0: {
           action_: "1/2"
         }
       },
       else: {
-        "0": {
+        0: {
           nextState: "1",
           revisit: true
         }
@@ -19281,12 +19281,12 @@ mhchemParser.stateMachines = {
         "*": {}
       },
       "1/2$": {
-        "0": {
+        0: {
           action_: "1/2"
         }
       },
       else: {
-        "0": {
+        0: {
           nextState: "1",
           revisit: true
         }
@@ -19387,30 +19387,30 @@ mhchemParser.stateMachines = {
         }
       },
       i$: {
-        "0": {
+        0: {
           nextState: "!f",
           revisit: true
         }
       },
       "(KV letters),": {
-        "0": {
+        0: {
           action_: "rm",
           nextState: "0"
         }
       },
       formula$: {
-        "0": {
+        0: {
           nextState: "f",
           revisit: true
         }
       },
       "1/2$": {
-        "0": {
+        0: {
           action_: "1/2"
         }
       },
       else: {
-        "0": {
+        0: {
           nextState: "!f",
           revisit: true
         }
@@ -19496,19 +19496,19 @@ mhchemParser.stateMachines = {
         "*": {}
       },
       x$: {
-        "0": {
+        0: {
           nextState: "!f",
           revisit: true
         }
       },
       formula$: {
-        "0": {
+        0: {
           nextState: "f",
           revisit: true
         }
       },
       else: {
-        "0": {
+        0: {
           nextState: "!f",
           revisit: true
         }
@@ -19761,13 +19761,13 @@ mhchemParser.stateMachines = {
         }
       },
       "(-)(9)^(-9)": {
-        "0": {
+        0: {
           action_: "number^",
           nextState: "a"
         }
       },
       "(-)(9.,9)(e)(99)": {
-        "0": {
+        0: {
           action_: "enumber",
           nextState: "a"
         }
@@ -19955,16 +19955,16 @@ mhchemParser.stateMachines = {
         }
       },
       "^{(...)}|^(-1)": {
-        "1": {
+        1: {
           action_: "^(-1)"
         }
       },
       "-9.,9": {
-        "0": {
+        0: {
           action_: "rm=",
           nextState: "0"
         },
-        "1": {
+        1: {
           action_: "^(-1)",
           nextState: "0"
         }
@@ -20014,7 +20014,7 @@ mhchemParser.stateMachines = {
   "pu-9,9": {
     transitions: mhchemParser.createTransitions({
       empty: {
-        "0": {
+        0: {
           action_: "output-0"
         },
         o: {
@@ -20022,13 +20022,13 @@ mhchemParser.stateMachines = {
         }
       },
       ",": {
-        "0": {
+        0: {
           action_: ["output-0", "comma"],
           nextState: "o"
         }
       },
       ".": {
-        "0": {
+        0: {
           action_: ["output-0", "copy"],
           nextState: "o"
         }
@@ -23297,10 +23297,7 @@ TagsDiff.defaultProps = {
   title: "Question tags"
 };
 
-/**
- * A side by side diff view for Perseus exercise items
- * that do not have the standard question layout.
- */
+/* eslint-disable @khanacademy/ts-no-error-suppressions */
 /**
  * Outputs true if path begins with beginPath, false otherwise.
  */
@@ -23487,7 +23484,7 @@ class StructuredItemDiff extends React.Component {
   }
 }
 
-/* eslint-disable @babel/no-invalid-this */
+/* eslint-disable @khanacademy/ts-no-error-suppressions */
 const {
   InfoTip: InfoTip$n,
   InlineIcon: InlineIcon$8
@@ -26108,6 +26105,7 @@ DefinitionEditor.defaultProps = {
   definition: ""
 };
 
+/* eslint-disable @khanacademy/ts-no-error-suppressions */
 const {
   InfoTip: InfoTip$k,
   InlineIcon: InlineIcon$6
@@ -28342,7 +28340,7 @@ const ImageEditor = createReactClass({
       this.setUrl(url, size[0], size[1], true);
     } catch (error) {
       this.setState({
-        backgroundImageError: `There was an error loading the image URL: ${error}`
+        backgroundImageError: `There was an error loading the image URL: ${JSON.stringify(error, null, 2)}`
       });
     }
   },
@@ -32184,6 +32182,7 @@ MoleculeWidgetEditor.propTypes = _extends({}, Changeable.propTypes, {
 });
 MoleculeWidgetEditor.widgetName = "molecule-renderer";
 
+/* eslint-disable @khanacademy/ts-no-error-suppressions */
 const {
   ButtonGroup: ButtonGroup$1,
   InfoTip: InfoTip$a,
@@ -32836,7 +32835,7 @@ const NumericInputEditor = createReactClass({
   }
 });
 
-/* eslint-disable one-var, react/forbid-prop-types, react/sort-comp */
+/* eslint-disable @khanacademy/ts-no-error-suppressions */
 const {
   InfoTip: InfoTip$8,
   TextListEditor: TextListEditor$2
@@ -33116,12 +33115,15 @@ const {
   TextListEditor: TextListEditor$1
 } = components;
 const Plotter = PlotterWidget.widget;
-const BAR = "bar",
-  LINE = "line",
-  PIC = "pic",
-  HISTOGRAM = "histogram",
-  DOTPLOT = "dotplot";
-
+const BAR = "bar";
+const LINE = "line";
+const PIC = "pic";
+const HISTOGRAM = "histogram";
+const DOTPLOT = "dotplot";
+const plotTypes = [BAR, LINE, PIC, HISTOGRAM, DOTPLOT];
+const STARTING = "starting";
+const CORRECT = "correct";
+const editingStates = [STARTING, CORRECT];
 // Return a copy of array with length n, padded with given value
 function padArray(array, n, value) {
   const copy = _.clone(array);
@@ -33136,29 +33138,12 @@ const editorDefaults = {
   maxY: 10,
   snapsPerLine: 2
 };
-const widgetPropTypes = {
-  type: PropTypes.oneOf([BAR, LINE, PIC, HISTOGRAM, DOTPLOT]),
-  labels: PropTypes.arrayOf(PropTypes.string),
-  categories: PropTypes.arrayOf(PropTypes.oneOfType([PropTypes.number, PropTypes.string])),
-  scaleY: PropTypes.number,
-  maxY: PropTypes.number,
-  snapsPerLine: PropTypes.number,
-  picSize: PropTypes.number,
-  pixBoxHeight: PropTypes.number,
-  picUrl: PropTypes.string,
-  plotDimensions: PropTypes.arrayOf(PropTypes.number),
-  labelInterval: PropTypes.number,
-  starting: PropTypes.arrayOf(PropTypes.number),
-  correct: PropTypes.arrayOf(PropTypes.number),
-  static: PropTypes.bool,
-  onChange: PropTypes.func
-};
 const formatNumber = num => "$" + number.round(num, 2) + "$";
 class PlotterEditor extends React.Component {
   constructor(...args) {
     super(...args);
     this.state = {
-      editing: this.props.static ? "starting" : "correct",
+      editing: this.props.static ? STARTING : CORRECT,
       pic: null,
       loadedUrl: null,
       minX: null,
@@ -33202,6 +33187,7 @@ class PlotterEditor extends React.Component {
       let categories;
       if (type === HISTOGRAM) {
         // Switching to histogram, add a label (0) to the left
+        // @ts-expect-error - TS2769
         categories = [formatNumber(0)].concat(this.props.categories);
         this.props.onChange({
           type: type,
@@ -33338,29 +33324,21 @@ class PlotterEditor extends React.Component {
     }, this.props);
     return /*#__PURE__*/React.createElement("div", {
       className: "perseus-widget-plotter-editor"
-    }, /*#__PURE__*/React.createElement("div", null, "Chart type:", " ", _.map([BAR, LINE, PIC, HISTOGRAM, DOTPLOT], function (type) {
+    }, /*#__PURE__*/React.createElement("div", null, "Chart type:", " ", plotTypes.map(type => {
       return /*#__PURE__*/React.createElement("label", {
         key: type
       }, /*#__PURE__*/React.createElement("input", {
         type: "radio",
-        name: "chart-type"
-        // @ts-expect-error [FEI-5003] - TS2683 - 'this' implicitly has type 'any' because it does not have a type annotation.
-        ,
+        name: "chart-type",
         checked: this.props.type === type,
-        onChange: _.partial(
-        // @ts-expect-error [FEI-5003] - TS2683 - 'this' implicitly has type 'any' because it does not have a type annotation.
-        this.changeType, type)
+        onChange: _.partial(this.changeType, type)
       }), type);
-    }, this)), /*#__PURE__*/React.createElement("div", null, "Labels:", " ", _.map(["x", "y"], function (axis, i) {
+    }, this)), /*#__PURE__*/React.createElement("div", null, "Labels:", " ", ["x", "y"].map((axis, i) => {
       return /*#__PURE__*/React.createElement("label", {
         key: axis
       }, axis + ":", /*#__PURE__*/React.createElement("input", {
         type: "text",
-        onChange: _.partial(
-        // @ts-expect-error [FEI-5003] - TS2683 - 'this' implicitly has type 'any' because it does not have a type annotation.
-        this.changeLabel, i)
-        // @ts-expect-error [FEI-5003] - TS2683 - 'this' implicitly has type 'any' because it does not have a type annotation.
-        ,
+        onChange: _.partial(this.changeLabel, i),
         defaultValue: this.props.labels[i]
       }));
     }, this)), setFromScale && /*#__PURE__*/React.createElement("div", {
@@ -33411,12 +33389,12 @@ class PlotterEditor extends React.Component {
       type: "text",
       onChange: this.changeSnaps,
       defaultValue: this.props.snapsPerLine
-    })), /*#__PURE__*/React.createElement(InfoTip$5, null, /*#__PURE__*/React.createElement("p", null, "Creates the specified number of divisions between the horizontal lines. Fewer snaps between lines makes the graph easier for the student to create correctly."))), /*#__PURE__*/React.createElement("div", null, "Editing values:", " ", ["correct", "starting"].map(editing => /*#__PURE__*/React.createElement("label", {
+    })), /*#__PURE__*/React.createElement(InfoTip$5, null, /*#__PURE__*/React.createElement("p", null, "Creates the specified number of divisions between the horizontal lines. Fewer snaps between lines makes the graph easier for the student to create correctly."))), /*#__PURE__*/React.createElement("div", null, "Editing values:", " ", editingStates.map(editing => /*#__PURE__*/React.createElement("label", {
       key: editing
     }, /*#__PURE__*/React.createElement("input", {
       type: "radio",
-      disabled: editing === "correct" && this.props.static,
-      checked: this.props.static ? editing === "starting" : this.state.editing === editing,
+      disabled: editing === CORRECT && this.props.static,
+      checked: this.props.static ? editing === STARTING : this.state.editing === editing,
       onChange: e => this.changeEditing(editing)
     }), editing)), /*#__PURE__*/React.createElement(InfoTip$5, null, /*#__PURE__*/React.createElement("p", null, "Use this toggle to switch between editing the correct answer (what the student will be graded on) and the starting values (what the student will see plotted when they start the problem). Note: These cannot be the same."), /*#__PURE__*/React.createElement("p", null, "In static mode, the starting values are rendered out to the displayed widget."))), /*#__PURE__*/React.createElement(Plotter, _extends({}, props, {
       starting: this.props[this.state.editing],
@@ -33424,7 +33402,6 @@ class PlotterEditor extends React.Component {
     })));
   }
 }
-PlotterEditor.propTypes = widgetPropTypes;
 PlotterEditor.widgetName = "plotter";
 PlotterEditor.defaultProps = _extends({}, editorDefaults, {
   correct: [1],
@@ -33436,6 +33413,7 @@ PlotterEditor.defaultProps = _extends({}, editorDefaults, {
   picBoxHeight: 36,
   plotDimensions: [275, 200],
   labelInterval: 1,
+  // @ts-expect-error - TS2322
   get picUrl() {
     const staticUrl = Dependencies.getDependencies().staticUrl;
     if (staticUrl) {
@@ -34108,7 +34086,7 @@ SimulatorEditor.defaultProps = {
   proportionOrPercentage: "proportion"
 };
 
-/* eslint-disable react/forbid-prop-types */
+/* eslint-disable @khanacademy/ts-no-error-suppressions */
 const {
   InfoTip: InfoTip$3,
   PropCheckBox: PropCheckBox$1,