@khanacademy/perseus 38.0.0 → 39.0.0

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

@@ -5,7 +5,7 @@ import type { LegacyButtonSets } from "../perseus-types";
 import type { PerseusDependenciesV2 } from "../types";
 import type { Keys, MathFieldInterface } from "@khanacademy/math-input";
 type ButtonsVisibleType = "always" | "never" | "focused";
-export type KeypadButtonSets = {
+type KeypadButtonSets = {
     advancedRelations?: boolean;
     basicRelations?: boolean;
     divisionKey?: boolean;

package/dist/components/sortable.d.ts

@@ -1,7 +1,7 @@
 import * as React from "react";
 import type { Position } from "../util";
 import type { LinterContextProps } from "@khanacademy/perseus-linter";
-export type Layout = "horizontal" | "vertical";
+type Layout = "horizontal" | "vertical";
 declare enum ItemState {
     STATIC = "static",
     DRAGGING = "dragging",

package/dist/definition-context.d.ts

@@ -9,7 +9,7 @@ type DefintionContext = {
 type ProviderState = {
     activeDefinitionId: string | null | undefined;
 };
-export type ProviderProps = {
+type ProviderProps = {
     children: any;
 };
 export declare class DefinitionProvider extends React.Component<ProviderProps, ProviderState> {

package/dist/es/index.js

@@ -2402,11 +2402,6 @@ class Expression extends React.Component {
     // @ts-expect-error - TS2339 - Property 'insert' does not exist on type 'ReactInstance'.
     this.refs.input.insert(keyPressed);
   }
-  // TODO(Nicole): I believe this is going away and won't be needed anymore
-  getGrammarTypeForPath(inputPath) {
-    /* c8 ignore next */
-    return "expression";
-  }
   setInputValue(path, newValue, cb) {
     this.props.onChange({
       value: newValue
@@ -2878,7 +2873,9 @@ function flattenScores(widgetScoreMap) {
 function isCorrect(score) {
   return score.type === "points" && score.earned >= score.total;
 }
-function keScoreFromPerseusScore(score, guess, state) {
+function keScoreFromPerseusScore(score,
+// It's weird, but this is what we're passing it
+guess, state) {
   if (score.type === "points") {
     return {
       empty: false,
@@ -3892,36 +3889,14 @@ const ZoomService = new ZoomServiceClass();
 const baseFontFamily$1 = "'Lato', sans-serif";
 // @boldFontFamily:        "Lato-Bold", "Lato", sans-serif;
 const boldFontFamily = "'Lato-Bold', 'Lato', sans-serif";
-// @green:                 #76a005;
-const green$1 = "#76A005";
 // @kaGreen:               #71b307;
 const kaGreen$1 = "#71B307";
-// @blue:                  #1c758a;
-const blue$1 = "#1C758A";
-// @gray:                  #aaa;
-const gray$1 = "#AAAAAA";
-// @red:                   #ffbaba;
-const red$1 = "#FFBABA";
-// @grayLight:             #aaa;
-const grayLight$1 = "#AAAAAA";
-// @grayLighter:           #ddd;
-const grayLighter$1 = "#DDDDDD";
-// @learnstormBlue:        #4898fc;
-const learnstormBlue$1 = "#4898FC";
 const white$1 = "#FFFFFF";
-const gray98 = "#FAFAFA";
 const gray97 = "#F6F7F7";
-const gray95 = "#F0F1F2";
-const gray90 = "#E3E5E6";
 const gray85 = "#D6D8DA";
 const gray76 = "#BABEC2";
 const gray68 = "#888D93";
-const gray41 = "#626569";
-const gray25 = "#3B3e40";
 const gray17 = "#21242c";
-const black$1 = "#000000";
-const warning1 = "#F86700";
-const warning3 = "#C75300";
 
 // @pure-sm-min: 568px;
 const pureSmMin$2 = "568px";
@@ -3941,16 +3916,10 @@ const pureMdMax$2 = "1023px";
 const pureLgMax$2 = "1279px";
 // @tableBackgroundAccent: #f9f9f9; // for striping
 const tableBackgroundAccent$1 = "#F9F9F9";
-// @zIndexScratchPad: 1;
-const zIndexScratchPad = 1;
 // @zIndexAboveScratchpad: @zIndexScratchPad + 1;
 const zIndexAboveScratchpad$1 = 2;
 // @zIndexInteractiveComponent: @zIndexAboveScratchpad + 1;
 const zIndexInteractiveComponent$1 = 3;
-// @zIndexCurrentlyDragging: @zIndexInteractiveComponent + 1;
-const zIndexCurrentlyDragging = 4;
-// @zIndexCalculator: @zIndexCurrentlyDragging + 1;
-const zIndexCalculator = 5;
 // @phoneMargin: 16px;
 const phoneMargin = 16;
 const negativePhoneMargin = -16;
@@ -3978,28 +3947,13 @@ var constants$1 = /*#__PURE__*/Object.freeze({
   __proto__: null,
   baseFontFamily: baseFontFamily$1,
   boldFontFamily: boldFontFamily,
-  green: green$1,
   kaGreen: kaGreen$1,
-  blue: blue$1,
-  gray: gray$1,
-  red: red$1,
-  grayLight: grayLight$1,
-  grayLighter: grayLighter$1,
-  learnstormBlue: learnstormBlue$1,
   white: white$1,
-  gray98: gray98,
   gray97: gray97,
-  gray95: gray95,
-  gray90: gray90,
   gray85: gray85,
   gray76: gray76,
   gray68: gray68,
-  gray41: gray41,
-  gray25: gray25,
   gray17: gray17,
-  black: black$1,
-  warning1: warning1,
-  warning3: warning3,
   pureSmMin: pureSmMin$2,
   pureMdMin: pureMdMin$2,
   pureLgMin: pureLgMin$2,
@@ -4009,11 +3963,8 @@ var constants$1 = /*#__PURE__*/Object.freeze({
   pureMdMax: pureMdMax$2,
   pureLgMax: pureLgMax$2,
   tableBackgroundAccent: tableBackgroundAccent$1,
-  zIndexScratchPad: zIndexScratchPad,
   zIndexAboveScratchpad: zIndexAboveScratchpad$1,
   zIndexInteractiveComponent: zIndexInteractiveComponent$1,
-  zIndexCurrentlyDragging: zIndexCurrentlyDragging,
-  zIndexCalculator: zIndexCalculator,
   phoneMargin: phoneMargin,
   negativePhoneMargin: negativePhoneMargin,
   hintBorderWidth: hintBorderWidth,
@@ -19756,6 +19707,7 @@ class Renderer extends React.Component {
         onBlur: _.partial(this._onWidgetBlur, id),
         findWidgets: this.findWidgets,
         reviewModeRubric: reviewModeRubric,
+        reviewMode: this.props.reviewMode,
         onChange: (newProps, cb, silent = false) => {
           this._setWidgetProps(id, newProps, cb, silent);
         },
@@ -20435,14 +20387,6 @@ class Renderer extends React.Component {
         return ReactDOM__default.findDOMNode(widget);
       }
     };
-    this.getGrammarTypeForPath = path => {
-      // @ts-expect-error - TS2345 - Argument of type 'FocusPath' is not assignable to parameter of type 'List<any>'.
-      const widgetId = _.first(path);
-      // @ts-expect-error - TS2345 - Argument of type 'FocusPath' is not assignable to parameter of type 'List<any>'.
-      const interWidgetPath = _.rest(path);
-      const widget = this.getWidgetInstance(widgetId);
-      return widget == null || widget.getGrammarTypeForPath == null ? void 0 : widget.getGrammarTypeForPath(interWidgetPath);
-    };
     this.getInputPaths = () => {
       const inputPaths = [];
       _.each(this.widgetIds, widgetId => {
@@ -20582,11 +20526,6 @@ class Renderer extends React.Component {
     this.getWidgetIds = () => {
       return this.widgetIds;
     };
-    this.guessAndScore = () => {
-      const totalGuess = this.getUserInput();
-      const totalScore = this.score();
-      return [totalGuess, totalScore];
-    };
     this.examples = () => {
       const widgetIds = this.widgetIds;
       const examples = _.compact(_.map(widgetIds, widgetId => {
@@ -20808,6 +20747,11 @@ class Renderer extends React.Component {
     const combinedScore = Util.flattenScores(scores);
     return combinedScore;
   }
+  guessAndScore() {
+    const totalGuess = this.getUserInput();
+    const totalScore = this.score();
+    return [totalGuess, totalScore];
+  }
   render() {
     const apiOptions = this.getApiOptions();
     if (this.reuseMarkdown) {
@@ -21676,7 +21620,7 @@ const answerTypes = {
     forms: "pi"
   }
 };
-function inputNumberValidator(state, rubric, strings) {
+function inputNumberValidator(userInput, rubric, strings) {
   if (rubric.answerType == null) {
     rubric.answerType = "number";
   }
@@ -21694,7 +21638,7 @@ function inputNumberValidator(state, rubric, strings) {
 
   // We may have received TeX; try to parse it before grading.
   // If `currentValue` is not TeX, this should be a no-op.
-  const currentValue = ParseTex$1(state.currentValue);
+  const currentValue = ParseTex$1(userInput.currentValue);
   const result = val(currentValue);
 
   // TODO(eater): Seems silly to translate result to this invalid/points
@@ -21787,12 +21731,6 @@ class InputNumber extends React.Component {
       /* c8 ignore next */
       return [[]];
     };
-    // Note: We believe that this isn't used anywhere but are leaving it during
-    // the initial refactor
-    this.getGrammarTypeForPath = path => {
-      /* c8 ignore next */
-      return "number";
-    };
     this.setInputValue = (path, newValue, cb) => {
       this.props.onChange({
         currentValue: newValue
@@ -21836,17 +21774,13 @@ class InputNumber extends React.Component {
       }
       return input;
     }
-    // HACK(johnsullivan): Create a function with shared logic between
-    // this and NumericInput.
-    // TODO(jeremy): Deprecate this widget and prefer numeric-input.
-    const rubric = this.props.reviewModeRubric;
 
     // Note: This is _very_ similar to what `numeric-input.jsx` does. If
     // you modify this, double-check if you also need to modify that
     // component.
     const inputStyles = [styles$A.default, this.props.size === "small" ? styles$A.small : null, this.props.rightAlign ? styles$A.rightAlign : styles$A.leftAlign];
     // Unanswered
-    if (rubric && !this.props.currentValue) {
+    if (this.props.reviewMode && !this.props.currentValue) {
       inputStyles.push(styles$A.answerStateUnanswered);
     }
     return /*#__PURE__*/React.createElement(InputWithExamples
@@ -22149,10 +22083,6 @@ class NumericInput extends React.Component {
       /* c8 ignore next */
       return [[]];
     };
-    this.getGrammarTypeForPath = inputPath => {
-      /* c8 ignore next */
-      return "number";
-    };
     this.setInputValue = (path, newValue, cb) => {
       /* c8 ignore next */
       this.props.onChange({
@@ -22339,7 +22269,7 @@ var NumericInput$1 = {
  * Several widgets don't have "right"/"wrong" validation logic,
  * so this just says to move on past those widgets
  *
- * TODO(LEMS-2396) widgets that use this probably shouldn't have any
+ * TODO(LEMS-2543) widgets that use this probably shouldn't have any
  * validation logic and the thing validating an exercise
  * should just know to skip these
  */
@@ -24401,7 +24331,7 @@ class Passage extends React.Component {
     const enabled = this.state.stylesAreApplied;
 
     // Highlights are read-only in review mode.
-    const editable = !this.props.reviewModeRubric;
+    const editable = !this.props.reviewMode;
     return /*#__PURE__*/React.createElement(HighlightableContent, {
       editable: editable,
       enabled: enabled,
@@ -25660,6 +25590,7 @@ const BaseRadio = function BaseRadio(props) {
   const {
     apiOptions,
     reviewModeRubric,
+    reviewMode,
     choices,
     editMode,
     multipleSelect,
@@ -25772,7 +25703,6 @@ const BaseRadio = function BaseRadio(props) {
   });
 
   // some commonly used shorthands
-  const reviewMode = !!reviewModeRubric;
   const isMobile = apiOptions.isMobile;
   const firstChoiceHighlighted = choices[0].highlighted;
   const lastChoiceHighlighted = choices[choices.length - 1].highlighted;
@@ -26340,6 +26270,7 @@ class Radio$1 extends React.Component {
       }));
     }
     const choicesProp = choices.map((choice, i) => {
+      var _this$props$reviewMod;
       const content = choice.isNoneOfTheAbove && !choice.content ? strings.noneOfTheAbove : choice.content;
       const {
         selected,
@@ -26350,7 +26281,7 @@ class Radio$1 extends React.Component {
         highlighted,
         previouslyAnswered
       } = choiceStates[i];
-      const reviewChoice = this.props.reviewModeRubric && this.props.reviewModeRubric.choices[i];
+      const reviewChoice = (_this$props$reviewMod = this.props.reviewModeRubric) == null ? void 0 : _this$props$reviewMod.choices[i];
       return {
         content: this._renderRenderer(content),
         checked: selected,
@@ -26385,6 +26316,7 @@ class Radio$1 extends React.Component {
       choices: choicesProp,
       onChange: this.updateChoices,
       reviewModeRubric: this.props.reviewModeRubric,
+      reviewMode: this.props.reviewMode,
       deselectEnabled: this.props.deselectEnabled,
       apiOptions: this.props.apiOptions,
       isLastUsedWidget: this.props.isLastUsedWidget,
@@ -29121,7 +29053,7 @@ class Group extends React.Component {
       ref: ref => this.rendererRef = ref,
       apiOptions: apiOptions,
       findExternalWidgets: this.props.findWidgets,
-      reviewMode: !!this.props.reviewModeRubric,
+      reviewMode: this.props.reviewMode,
       onInteractWithWidget: onInteractWithWidget,
       linterContext: this.props.linterContext,
       strings: this.context.strings
@@ -38693,10 +38625,12 @@ function scoreMarker(marker) {
   }
   return score;
 }
-function labelImageValidator(state, rubric) {
+
+// TODO(LEMS-2440): May need to pull answers out of PerseusLabelImageWidgetOptions[markers] for the rubric
+function labelImageValidator(userInput, rubric) {
   let numAnswered = 0;
   let numCorrect = 0;
-  for (const marker of state.markers) {
+  for (const marker of userInput.markers) {
     const score = scoreMarker(marker);
     if (score.hasAnswers) {
       numAnswered++;
@@ -38707,7 +38641,7 @@ function labelImageValidator(state, rubric) {
   }
 
   // We expect all question markers to be answered before grading.
-  if (numAnswered !== state.markers.length) {
+  if (numAnswered !== userInput.markers.length) {
     return {
       type: "invalid",
       message: null
@@ -38717,7 +38651,7 @@ function labelImageValidator(state, rubric) {
     type: "points",
     // Markers with no expected answers are graded as correct if user
     // makes no answer selection.
-    earned: numCorrect === state.markers.length ? 1 : 0,
+    earned: numCorrect === userInput.markers.length ? 1 : 0,
     total: 1,
     message: null
   };
@@ -38981,8 +38915,6 @@ const styles$b = StyleSheet.create({
  * No diagonals allowed.
  */
 
-// TODO: should this be using WidgetProps / PerseusLabelImageWidgetOptions?
-
 class LabelImage extends React.Component {
   /**
    * Test whether point is contained within triangle.
@@ -39186,6 +39118,8 @@ class LabelImage extends React.Component {
       markers
     };
   }
+
+  // TODO(LEMS-2544): Investigate impact on scoring; possibly pull out &/or remove rubric parameter.
   showRationalesForCurrentlySelectedChoices(rubric) {
     const {
       markers
@@ -40611,9 +40545,6 @@ class Matrix extends React.Component {
       });
       return inputPaths;
     };
-    this.getGrammarTypeForPath = inputPath => {
-      return "number";
-    };
     this._handleFocus = (row, col) => {
       this.props.onFocus(getInputPath$1(row, col));
     };
@@ -41026,7 +40957,7 @@ class Measurer extends React.Component {
 }
 Measurer.defaultProps = {
   box: [480, 480],
-  image: {},
+  image: defaultImage,
   showProtractor: true,
   protractorX: 7.5,
   protractorY: 0.5,
@@ -42101,10 +42032,12 @@ const TickMarks = Graphie.createSimpleClass((graphie, props) => {
   };
 
   // Generate an array of tick numbers:
-  //    `Array(props.numDivisions)` makes an array of null values - one for every division marker
+  //    `Array(Math.round(numDivisions))` makes an array of null values - one for every division marker
   //    `.keys()` gets the index values for each marker placeholder
   //    `.map()` converts the index values into actual tick numbers
-  const initialTicks = [...Array(numDivisions).keys()].map(index => range[0] + index * tickStep);
+  // NOTE: 'numDivisions' can sometimes be a non-integer (i.e. 7.000001).
+  //       Using Math.round() to ensure that an integer is used in the Array setup.
+  const initialTicks = [...Array(Math.round(numDivisions)).keys()].map(index => range[0] + index * tickStep);
 
   // .sort() comparator
   const byNumericAscending = (a, b) => a - b;
@@ -42409,11 +42342,6 @@ class NumberLine extends React.Component {
     }
     return null;
   }
-  getGrammarTypeForPath(inputPath) {
-    if ((inputPath == null ? void 0 : inputPath.length) === 1 && inputPath[0] === "tick-ctrl") {
-      return "number";
-    }
-  }
   getUserInput() {
     return {
       numLinePosition: this.props.numLinePosition,
@@ -44507,9 +44435,6 @@ class Table extends React.Component {
       });
       return inputPaths;
     };
-    this.getGrammarTypeForPath = inputPath => {
-      return "number";
-    };
     this.setInputValue = (path, newValue, cb) => {
       // Extract row, column information
       const row = getRowFromPath(path);
@@ -44827,7 +44752,7 @@ const init = function init(options) {
 
 // This file is processed by a Rollup plugin (replace) to inject the production
 const libName = "@khanacademy/perseus";
-const libVersion = "38.0.0";
+const libVersion = "39.0.0";
 addLibraryVersionToPerseusDebug(libName, libVersion);
 
 const apiVersion = {
@@ -45549,9 +45474,6 @@ class ServerItemRenderer extends React.Component {
   getDOMNodeForPath(path) {
     return this.questionRenderer.getDOMNodeForPath(path);
   }
-  getGrammarTypeForPath(path) {
-    return this.questionRenderer.getGrammarTypeForPath(path);
-  }
   getInputPaths() {
     const questionAreaInputPaths = this.questionRenderer.getInputPaths();
     return questionAreaInputPaths;
@@ -46828,7 +46750,6 @@ const styles = StyleSheet.create({
  * If you're writing new components, though, consider using the Item and Shape
  * types instead.
  */
-
 /**
  * Return a PropType that accepts Items of the given shape, and rejects other
  * objects.
@@ -47060,6 +46981,21 @@ function violatingWidgets(itemData) {
 }
 
 /* eslint-disable eqeqeq, @typescript-eslint/no-unused-vars */
+/*
+ * A class for formatting number to significant digits.
+ * Copyright (C) 2001 Stephen Ostermiller
+ * http://ostermiller.org/contact.pl?regarding?JavaScript+Significant+Figures
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ */
 
 /**
  * Format a floating point for display using the specified