@khanacademy/perseus 21.0.0 → 21.1.0

package/dist/es/index.js

@@ -7,7 +7,7 @@ import * as i18n from '@khanacademy/wonder-blocks-i18n';
 import Tooltip$1 from '@khanacademy/wonder-blocks-tooltip';
 import classNames from 'classnames';
 import * as React from 'react';
-import React__default, { useState, useEffect, useRef } from 'react';
+import React__default, { useState, useEffect, useRef, createContext } from 'react';
 import _, { debounce as debounce$1 } from 'underscore';
 import Clickable from '@khanacademy/wonder-blocks-clickable';
 import { Popover, PopoverContentCore, PopoverContent } from '@khanacademy/wonder-blocks-popover';
@@ -24901,9 +24901,22 @@ class Explanation extends React.Component {
     // Special styling is needed to fit the button in a block of text without throwing off the line spacing.
     // While the button is not normally included in a block of text, it needs to be able to accommodate such a case.
     const buttonStyleOverrides = {
-      height: "22px",
+      height: "auto",
       marginLeft: "-2px",
-      padding: "0 2px"
+      // transfer space on the left side to the padding
+      marginRight: "2px",
+      // adds space between the right border and any text that follows
+      paddingLeft: "2px" // adds space between the left border and the button text
+    };
+
+    const labelStyle = {
+      fontSize: "18px",
+      lineHeight: "20px",
+      // The following property adjusts the large space between the button text and the caret icon.
+      // Since we are unable to adjust the styling of the icon (where the extra space exists),
+      //      we are adjusting it on the text label by using a negative margin.
+      marginRight: "-6px",
+      "white-space": "normal"
     };
     const contentStyling = [styles$p.content, this.state.expanded ? styles$p.contentExpanded : styles$p.contentCollapsed, allowTransition && (this.state.expanded ? styles$p.transitionExpanded : styles$p.transitionCollapsed)];
     return /*#__PURE__*/React.createElement(UniqueIDProvider, {
@@ -24914,8 +24927,9 @@ class Explanation extends React.Component {
       "aria-controls": ids.get("content"),
       endIcon: caretIcon,
       kind: "tertiary",
+      labelStyle: labelStyle,
       onClick: this._onClick,
-      size: "large",
+      size: "small",
       style: buttonStyleOverrides
     }, promptText), /*#__PURE__*/React.createElement(View, {
       id: ids.get("content"),
@@ -28190,6 +28204,14 @@ const moveAll = delta => ({
   type: MOVE_ALL,
   delta
 });
+const MOVE_POINT = "move-point";
+function movePoint(index, destination) {
+  return {
+    type: MOVE_POINT,
+    index,
+    destination
+  };
+}
 
 /**
  * 44 is touch best practice and AAA compliant for WCAG
@@ -28412,7 +28434,9 @@ const SegmentGraph = props => {
     onMoveLine: delta => {
       dispatch(moveLine(i, delta));
     },
-    onMovePoint: (endpointIndex, destination) => dispatch(moveControlPoint(endpointIndex, destination, i)),
+    onMovePoint: (endpointIndex, destination) => {
+      dispatch(moveControlPoint(endpointIndex, destination, i));
+    },
     "data-testid": "segment" + i
   })));
 };
@@ -28768,11 +28792,73 @@ const PolygonGraph = props => {
   }), points.map((point, i) => /*#__PURE__*/React.createElement(StyledMovablePoint, {
     key: "point-" + i,
     point: point,
-    onMove: destination => dispatch(moveControlPoint(i, destination)),
+    onMove: destination => dispatch(movePoint(i, destination)),
     "data-testid": type + i
   })));
 };
 
+function PointGraph(props) {
+  const {
+    dispatch
+  } = props;
+  return /*#__PURE__*/React.createElement(React.Fragment, null, props.graphState.coords.map((point, i) => /*#__PURE__*/React.createElement(StyledMovablePoint, {
+    key: i,
+    point: point,
+    onMove: destination => dispatch(movePoint(i, destination))
+  })));
+}
+
+const GraphStateContext = /*#__PURE__*/createContext({
+  state: {},
+  dispatch: () => {}
+});
+function useGraphState() {
+  return React__default.useContext(GraphStateContext);
+}
+
+function Arrow(props) {
+  const {
+    userTransform,
+    viewTransform
+  } = useTransformContext();
+  const point = [props.x, props.y];
+  const userTransformedPoint = vec.transform(point, userTransform);
+  const viewTransformedPoint = vec.transform(userTransformedPoint, viewTransform);
+  return /*#__PURE__*/React__default.createElement("g", {
+    transform: `translate(${viewTransformedPoint[0]} ${viewTransformedPoint[1]}) rotate(${props.rotate}) scale(1.25)`
+  }, /*#__PURE__*/React__default.createElement("g", {
+    transform: "translate(-1)"
+  }, /*#__PURE__*/React__default.createElement("path", {
+    d: "M-3 4 C-2.75 2.5 0 0.25 0.75 0C0 -0.25 -2.75 -2.5 -3 -4",
+    fill: "none"
+  })));
+}
+function AxisArrows() {
+  const {
+    state
+  } = useGraphState();
+  const range = state.range;
+  const [xMin, xMax] = range[0];
+  const [yMin, yMax] = range[1];
+  return /*#__PURE__*/React__default.createElement(React__default.Fragment, null, /*#__PURE__*/React__default.createElement(Arrow, {
+    x: xMax,
+    y: 0,
+    rotate: 0
+  }), /*#__PURE__*/React__default.createElement(Arrow, {
+    x: 0,
+    y: yMin,
+    rotate: 90
+  }), /*#__PURE__*/React__default.createElement(Arrow, {
+    x: xMin,
+    y: 0,
+    rotate: 180
+  }), /*#__PURE__*/React__default.createElement(Arrow, {
+    x: 0,
+    y: yMax,
+    rotate: 270
+  }));
+}
+
 const renderLineLabel = (n, [min, max]) => n !== -1 && n !== min && n !== max;
 const axisOptions = (props, index) => ({
   axis: props.markings === "graph",
@@ -28780,10 +28866,10 @@ const axisOptions = (props, index) => ({
   subdivisions: props.step[index] / props.gridStep[index],
   labels: n => renderLineLabel(n, props.range[index]) ? n : ""
 });
-const Grid = props => props.markings === "none" ? null : /*#__PURE__*/React.createElement(Coordinates.Cartesian, {
+const Grid = props => props.markings === "none" ? null : /*#__PURE__*/React.createElement(React.Fragment, null, /*#__PURE__*/React.createElement(Coordinates.Cartesian, {
   xAxis: axisOptions(props, 0),
   yAxis: axisOptions(props, 1)
-});
+}), props.markings === "graph" && /*#__PURE__*/React.createElement(AxisArrows, null));
 
 // Displays a stylized open eye: lint warnings are visible
 const VisibleIcon = () => /*#__PURE__*/React.createElement("svg", {
@@ -29480,22 +29566,32 @@ const LegacyGrid = ({
   return null;
 };
 
-/** Determine if coords is type CollinearTuple[] */
-const isCollinearTuples = coords => Array.isArray(coords[0][0]);
-
-// Generic type makes returned state match input state
 function interactiveGraphReducer(state, action) {
+  switch (action.type) {
+    case MOVE_CONTROL_POINT:
+      return doMoveControlPoint(state, action);
+    case MOVE_LINE:
+      return doMoveLine(state, action);
+    case MOVE_ALL:
+      return doMoveAll(state, action);
+    case MOVE_POINT:
+      return doMovePoint(state, action);
+    default:
+      throw new UnreachableCaseError(action);
+  }
+}
+function doMoveControlPoint(state, action) {
   const {
     snapStep,
     range
   } = state;
-  if (!state.coords) {
-    throw new Error("Graph state must have been initialized with coords");
-  }
-  switch (action.type) {
-    case MOVE_CONTROL_POINT:
+  switch (state.type) {
+    case "segment":
+    case "linear":
+    case "linear-system":
+    case "ray":
       {
-        const newCoords = action.itemIndex !== undefined && state.coords && isCollinearTuples(state.coords) ? updateAtIndex({
+        const newCoords = updateAtIndex({
           array: state.coords,
           index: action.itemIndex,
           update: tuple => setAtIndex({
@@ -29510,23 +29606,9 @@ function interactiveGraphReducer(state, action) {
               })
             })
           })
-        }) : setAtIndex({
-          array: state.coords,
-          index: action.pointIndex,
-          newValue: snap({
-            snapStep,
-            point: bound$1({
-              snapStep,
-              range,
-              point: action.destination
-            })
-          })
         });
-
-        // Cannot type narrow both conditions within function parameters,
-        // so this may seem redundant, but it's necessary for type safety.
-        const coordsToCheck = isCollinearTuples(newCoords) && action.itemIndex !== undefined ? newCoords[action.itemIndex] : newCoords;
-        if (!isCollinearTuples(coordsToCheck) && coordsOverlap(coordsToCheck)) {
+        const coordsToCheck = newCoords[action.itemIndex];
+        if (coordsOverlap(coordsToCheck)) {
           return state;
         }
         return _extends({}, state, {
@@ -29534,16 +29616,30 @@ function interactiveGraphReducer(state, action) {
           coords: newCoords
         });
       }
-    case MOVE_LINE:
+    case "circle":
+      throw new Error("FIXME implement circle reducer");
+    case "point":
+    case "polygon":
+      throw new Error(`Don't use moveControlPoint for ${state.type} graphs. Use movePoint instead!`);
+    default:
+      throw new UnreachableCaseError(state);
+  }
+}
+function doMoveLine(state, action) {
+  const {
+    snapStep,
+    range
+  } = state;
+  switch (state.type) {
+    case "segment":
+    case "linear":
+    case "linear-system":
+    case "ray":
       {
-        var _state$coords;
-        if (!isCollinearTuples(state.coords)) {
-          throw new Error("Cannot call this action unless coords is array of CollinearTuple.");
-        }
         if (action.itemIndex === undefined) {
           throw new Error("Please provide index of line to move");
         }
-        const currentLine = (_state$coords = state.coords) == null ? void 0 : _state$coords[action.itemIndex];
+        const currentLine = state.coords[action.itemIndex];
         if (!currentLine) {
           throw new Error("No line to move");
         }
@@ -29564,15 +29660,25 @@ function interactiveGraphReducer(state, action) {
           newValue: newLine
         });
         return _extends({}, state, {
+          type: state.type,
           hasBeenInteractedWith: true,
           coords: newCoords
         });
       }
-    case MOVE_ALL:
+    default:
+      // The MoveLine action doesn't make sense for other graph types;
+      // ignore it if it somehow happens
+      return state;
+  }
+}
+function doMoveAll(state, action) {
+  const {
+    snapStep,
+    range
+  } = state;
+  switch (state.type) {
+    case "polygon":
       {
-        if (isCollinearTuples(state.coords)) {
-          throw new Error("Cannot call this action unless coords is array of vectors.");
-        }
         const change = getChange(state.coords, action.delta, {
           snapStep,
           range
@@ -29587,7 +29693,33 @@ function interactiveGraphReducer(state, action) {
         });
       }
     default:
-      throw new UnreachableCaseError(action);
+      // MoveAll is not supported for other state types; just ignore it.
+      return state;
+  }
+}
+function doMovePoint(state, action) {
+  switch (state.type) {
+    case "point":
+    case "polygon":
+      {
+        return _extends({}, state, {
+          hasBeenInteractedWith: true,
+          coords: setAtIndex({
+            array: state.coords,
+            index: action.index,
+            newValue: snap({
+              snapStep: state.snapStep,
+              point: bound$1({
+                snapStep: state.snapStep,
+                range: state.range,
+                point: action.destination
+              })
+            })
+          })
+        });
+      }
+    default:
+      throw new Error("The movePoint action is only for point and polygon graphs");
   }
 }
 const getChange = (coords, delta, constraintOpts) => {
@@ -29668,7 +29800,7 @@ function clamp(value, min, max) {
 const coordsOverlap = coords => coords.some((coord, i) => coords.some((c, j) => i !== j && vector$1.equal(coord, c)));
 function updateAtIndex(args) {
   const {
-    array = [],
+    array,
     index,
     update
   } = args;
@@ -29681,13 +29813,12 @@ function updateAtIndex(args) {
 }
 function setAtIndex(args) {
   const {
-    array = [],
+    array,
     index,
     newValue
   } = args;
   const copy = [...array];
   copy[index] = newValue;
-  // restoring readonly to array
   return copy;
 }
 
@@ -29728,19 +29859,32 @@ function initializeGraphState(params) {
         })
       };
     case "polygon":
-      return _extends({
+      return {
+        type: "polygon",
         hasBeenInteractedWith: false,
         range,
-        snapStep
-      }, graph, {
+        snapStep,
+        showAngles: Boolean(graph.showAngles),
+        showSides: Boolean(graph.showSides),
         coords: getPolygonCoords({
           graph,
           range,
           step
         })
-      });
-    case "angle":
+      };
     case "point":
+      return {
+        type: graph.type,
+        hasBeenInteractedWith: false,
+        range,
+        snapStep,
+        coords: getDefaultPoints({
+          graph,
+          step,
+          range
+        })
+      };
+    case "angle":
     case "circle":
     case "sinusoid":
     case "quadratic":
@@ -29749,6 +29893,46 @@ function initializeGraphState(params) {
       throw new UnreachableCaseError(graph);
   }
 }
+const getDefaultPoints = ({
+  graph,
+  range,
+  step
+}) => {
+  var _graph$coords;
+  const numPoints = graph.numPoints || 1;
+  let coords = (_graph$coords = graph.coords) == null ? void 0 : _graph$coords.slice();
+  if (coords) {
+    return coords;
+  }
+  switch (numPoints) {
+    case 1:
+      // Back in the day, one point's coords were in graph.coord
+      coords = [graph.coord || [0, 0]];
+      break;
+    case 2:
+      coords = [[-5, 0], [5, 0]];
+      break;
+    case 3:
+      coords = [[-5, 0], [0, 0], [5, 0]];
+      break;
+    case 4:
+      coords = [[-6, 0], [-2, 0], [2, 0], [6, 0]];
+      break;
+    case 5:
+      coords = [[-6, 0], [-3, 0], [0, 0], [3, 0], [6, 0]];
+      break;
+    case 6:
+      coords = [[-5, 0], [-3, 0], [-1, 0], [1, 0], [3, 0], [5, 0]];
+      break;
+    default:
+      coords = [];
+      break;
+  }
+  // Transform coords from their -10 to 10 space to 0 to 1
+  // because of the old graph.coord, and also it's easier.
+  const newCoords = normalizeCoords(coords, [[-10, 10], [-10, 10]]);
+  return normalizePoints(range, step, newCoords);
+};
 
 // TS v4 doesn't narrow return types, while v5 does.
 // Instead of updating to v5, using generic type to relate input and output types.
@@ -29769,15 +29953,13 @@ function getGradableGraph(state, initialGraph) {
     });
   }
   if (state.type === "linear" && initialGraph.type === "linear") {
-    var _state$coords;
     return _extends({}, initialGraph, {
-      coords: (_state$coords = state.coords) == null ? void 0 : _state$coords[0]
+      coords: state.coords[0]
     });
   }
   if (state.type === "ray" && initialGraph.type === "ray") {
-    var _state$coords2;
     return _extends({}, initialGraph, {
-      coords: (_state$coords2 = state.coords) == null ? void 0 : _state$coords2[0]
+      coords: state.coords[0]
     });
   }
   if (state.type === "polygon" && initialGraph.type === "polygon") {
@@ -29785,6 +29967,11 @@ function getGradableGraph(state, initialGraph) {
       coords: state.coords
     });
   }
+  if (state.type === "point" && initialGraph.type === "point") {
+    return _extends({}, initialGraph, {
+      coords: state.coords
+    });
+  }
   throw new Error("Mafs is not yet implemented for graph type: " + initialGraph.type);
 }
 const getDefaultSegments = ({
@@ -29822,10 +30009,10 @@ const getLineCoords = ({
   range,
   step
 }) => {
-  var _graph$coords;
+  var _graph$coords2;
   return (
     // Return two lines for a linear system, one for a ray or linear
-    ((_graph$coords = graph.coords) != null ? _graph$coords : graph.type === "linear-system") ? defaultLinearCoords.map(collinear => normalizePoints(range, step, collinear)) : [normalizePoints(range, step, defaultLinearCoords[0])]
+    ((_graph$coords2 = graph.coords) != null ? _graph$coords2 : graph.type === "linear-system") ? defaultLinearCoords.map(collinear => normalizePoints(range, step, collinear)) : [normalizePoints(range, step, defaultLinearCoords[0])]
   );
 };
 const getPolygonCoords = ({
@@ -29833,7 +30020,8 @@ const getPolygonCoords = ({
   range,
   step
 }) => {
-  let coords = graph.coords;
+  var _graph$coords3;
+  let coords = (_graph$coords3 = graph.coords) == null ? void 0 : _graph$coords3.slice();
   if (coords) {
     return coords;
   }
@@ -29887,6 +30075,13 @@ const renderGraph = props => {
         graphState: state,
         dispatch: dispatch
       });
+    case "point":
+      return /*#__PURE__*/React.createElement(PointGraph, {
+        graphState: state,
+        dispatch: dispatch
+      });
+    case "circle":
+      throw new Error("the circle graph type is not yet implemented");
     default:
       return new UnreachableCaseError(type);
   }
@@ -29894,10 +30089,24 @@ const renderGraph = props => {
 const MafsGraph = /*#__PURE__*/React.forwardRef((props, ref) => {
   const [width, height] = props.box;
   const [state, dispatch] = React.useReducer(interactiveGraphReducer, props, initializeGraphState);
+  const prevState = useRef(state);
+  useEffect(() => {
+    if (prevState.current !== state) {
+      props.onChange({
+        graph: state
+      });
+    }
+    prevState.current = state;
+  }, [props, state]);
   React.useImperativeHandle(ref, () => ({
     getUserInput: () => getGradableGraph(state, props.graph)
   }));
-  return /*#__PURE__*/React.createElement(View, {
+  return /*#__PURE__*/React.createElement(GraphStateContext.Provider, {
+    value: {
+      state,
+      dispatch
+    }
+  }, /*#__PURE__*/React.createElement(View, {
     style: {
       width,
       height,
@@ -29928,7 +30137,7 @@ const MafsGraph = /*#__PURE__*/React.forwardRef((props, ref) => {
   }), renderGraph({
     state,
     dispatch
-  }))));
+  })))));
 });
 
 const {
@@ -31065,6 +31274,7 @@ class LegacyInteractiveGraph extends React.Component {
       $(this.angle).on("move", () => {
         var _this$angle;
         this.onChange({
+          // @ts-expect-error Type '{ coords: any; type: "angle"; showAngles?: boolean | undefined; allowReflexAngles?: boolean | undefined; angleOffsetDeg?: number | undefined; snapDegrees?: number | undefined; match?: "congruent" | undefined; }' is not assignable to type 'InteractiveGraphState | undefined'.
           graph: _extends({}, graph, {
             coords: (_this$angle = this.angle) == null ? void 0 : _this$angle.getClockwiseCoords()
           })
@@ -31656,9 +31866,8 @@ class InteractiveGraph extends React.Component {
           };
         }
       } else if (userInput.type === "point" && rubric.correct.type === "point" && userInput.coords != null) {
-        var _userInput$coords;
         let correct = InteractiveGraph.getPointCoords(rubric.correct, component);
-        const guess = (_userInput$coords = userInput.coords) == null ? void 0 : _userInput$coords.slice();
+        const guess = userInput.coords.slice();
         correct = correct.slice();
         // Everything's already rounded so we shouldn't need to do an
         // eq() comparison but _.isEqual(0, -0) is false, so we'll use
@@ -31676,8 +31885,8 @@ class InteractiveGraph extends React.Component {
           };
         }
       } else if (userInput.type === "polygon" && rubric.correct.type === "polygon" && userInput.coords != null) {
-        var _userInput$coords2, _rubric$correct$coord;
-        const guess = (_userInput$coords2 = userInput.coords) == null ? void 0 : _userInput$coords2.slice();
+        var _userInput$coords, _rubric$correct$coord;
+        const guess = (_userInput$coords = userInput.coords) == null ? void 0 : _userInput$coords.slice();
         // @ts-expect-error - TS2322 - Type 'Coord[] | undefined' is not assignable to type 'Coord[]'.
         const correct = (_rubric$correct$coord = rubric.correct.coords) == null ? void 0 : _rubric$correct$coord.slice();
         let match;
@@ -41972,7 +42181,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 = "21.0.0";
+const libVersion = "21.1.0";
 addLibraryVersionToPerseusDebug(libName, libVersion);
 
 const apiVersion = {
@@ -44482,7 +44691,7 @@ function getAnswersFromWidgets(widgets) {
         // E.g. 'item 1\nitem 2\nitem 3'
         const orderer = widget;
         if ((_orderer$options = orderer.options) != null && _orderer$options.correctOptions) {
-          answers.push(orderer.options.correctOptions.map(option => option.content).join("\n"));
+          answers.push(joinOptionContents(orderer.options.correctOptions));
         }
         break;
       case "sorter":
@@ -44541,6 +44750,28 @@ function getAnswersFromWidgets(widgets) {
   return answers;
 }
 
+/**
+ * Join the content of the options together.
+ * @param {Array<{content: string}>} options
+ * @returns {string}
+ */
+const joinOptionContents = options => options.map(({
+  content
+}) => content).join("\n");
+
+/**
+ * Convert an index to an option letter.
+ * @param {number} index
+ * @returns {string}
+ * @example
+ * toOptionLetter(0) // 'A'
+ * toOptionLetter(1) // 'B'
+ * toOptionLetter(25) // 'Z'
+ * // Once the index goes past 25, it will start returning special characters
+ * toOptionLetter(26) // '['
+ */
+const toOptionLetter = index => String.fromCharCode("A".charCodeAt(0) + index);
+
 /**
  * Inject a string equivalent of the widgets into the content.
  *
@@ -44551,7 +44782,7 @@ function getAnswersFromWidgets(widgets) {
  * @param {PerseusRenderer["widgets"]} widgets
  * @returns
  */
-function injectWidgets(content, widgets) {
+function injectWidgets(content, widgets, widgetProps) {
   // The types for taskProgress.itemData are not well defined,
   // so there is a chance that widgets or content could be undefined.
   // In that scenario, simply exit early and return the content as is.
@@ -44571,12 +44802,20 @@ function injectWidgets(content, widgets) {
     switch (widget.type) {
       case "radio":
         // Replace radio with the radio options
-        // '[[☃ Radio 1]]' -> 'option 1\noption 2\noption 3'
+        // '[[☃ Radio 1]]' -> 'choice 1\nchoice 2\nchoice 3'
+        // or if the current widget state is available with proper order,
+        // '[[☃ Radio 1]]' ->
+        //   'Option A: choice 1\nOption B: choice 2\nOption C: choice 3'
         const radio = widget;
+        const radioProps = widgetProps == null ? void 0 : widgetProps[widgetID];
         if ((_radio$options = radio.options) != null && (_radio$options = _radio$options.choices) != null && _radio$options.length) {
           var _radio$options2;
-          let radioContext = radio.options.choices.map(choice => choice.content).join("\n");
-          if ((_radio$options2 = radio.options) != null && _radio$options2.randomize) {
+          let radioContext = joinOptionContents(radioProps ? radioProps.choices.map(({
+            content
+          }, i) => ({
+            content: `Option ${toOptionLetter(i)}: ${content}`
+          })) : radio.options.choices);
+          if (!radioProps && (_radio$options2 = radio.options) != null && _radio$options2.randomize) {
             radioContext += "\nThose options are displayed in a different order to the user. If the user says the letter, number, or ordinal number, always ask them clarify which option they are referring to.\n";
           }
           context = context.replace(`[[☃ ${widgetID}]]`, radioContext);
@@ -44685,7 +44924,7 @@ function injectWidgets(content, widgets) {
         // '[[☃ Orderer 1]]' -> 'option 1\noption 2\noption 3'
         const orderer = widget;
         if ((_orderer$options2 = orderer.options) != null && _orderer$options2.options) {
-          context = context.replace(`[[☃ ${widgetID}]]`, orderer.options.options.map(option => option.content).join("\n"));
+          context = context.replace(`[[☃ ${widgetID}]]`, joinOptionContents(orderer.options.options));
         }
         break;
       case "sorter":