@khanacademy/perseus 21.1.0 → 21.2.1

package/dist/es/index.css

@@ -4284,6 +4284,7 @@ html body .mq-editable-field .mq-selection .mq-nthroot-container * {
     --movable-point-ring-radius: calc(2px + var(--movable-point-center-radius));
     --movable-point-halo-radius: calc(3px + var(--movable-point-ring-radius));
     --movable-point-hover-expansion: 2px;
+    --movable-point-focus-ring-offset: 2px;
 
     --movable-line-stroke-color: var(--mafs-blue);
     --movable-line-stroke-weight: 2px;
@@ -4305,7 +4306,13 @@ html body .mq-editable-field .mq-selection .mq-nthroot-container * {
     fill: transparent;
 }
 
-.MafsView :is(.movable-point-center, .movable-point-ring, .movable-point-halo) {
+.MafsView
+    :is(
+        .movable-point-center,
+        .movable-point-ring,
+        .movable-point-halo,
+        .movable-point-focus-outline
+    ) {
     transition: r 0.15s ease-out;
 }
 
@@ -4344,7 +4351,24 @@ html body .mq-editable-field .mq-selection .mq-nthroot-container * {
     );
 }
 
-.MafsView .movable-point:focus-visible .movable-point-halo {
-    outline: 2px solid blue;
-    border-radius: 99px;
+.MafsView .movable-point .movable-point-focus-outline {
+    visibility: hidden;
+    r: calc(
+        var(--movable-point-halo-radius) +
+            var(--movable-point-focus-ring-offset)
+    );
+    stroke-width: 2px;
+    fill: none;
+    stroke: var(--mafs-blue);
+}
+
+.MafsView .movable-point:hover .movable-point-focus-outline {
+    r: calc(
+        var(--movable-point-hover-expansion) + var(--movable-point-halo-radius) +
+            var(--movable-point-focus-ring-offset)
+    );
+}
+
+.MafsView .movable-point:focus-visible .movable-point-focus-outline {
+    visibility: visible;
 }

package/dist/es/index.js

@@ -3258,6 +3258,15 @@ const textarea = {
  */
 const unescapeMathMode$1 = label => label.startsWith("$") && label.endsWith("$") ? label.slice(1, -1) : label;
 const random$1 = seededRNG$1(new Date().getTime() & 0xffffffff);
+
+// TODO(benchristel): in the future, we may want to make deepClone work for
+// Record<string, Cloneable> as well. Currently, it only does arrays.
+function deepClone(obj) {
+  if (Array.isArray(obj)) {
+    return obj.map(deepClone);
+  }
+  return obj;
+}
 const Util = {
   inputPathsEqual,
   nestedMap: nestedMap$2,
@@ -3303,7 +3312,8 @@ const Util = {
   getDataUrl: getDataUrl,
   textarea,
   unescapeMathMode: unescapeMathMode$1,
-  random: random$1
+  random: random$1,
+  deepClone
 };
 
 // Derived from the MIT-licensed:
@@ -28212,6 +28222,28 @@ function movePoint(index, destination) {
     destination
   };
 }
+const CHANGE_SNAP_STEP = "change-snap-step";
+function changeSnapStep(snapStep) {
+  return {
+    type: CHANGE_SNAP_STEP,
+    snapStep
+  };
+}
+const CHANGE_RANGE = "change-range";
+function changeRange(range) {
+  return {
+    type: CHANGE_RANGE,
+    range
+  };
+}
+
+const GraphStateContext = /*#__PURE__*/createContext({
+  state: {},
+  dispatch: () => {}
+});
+function useGraphState() {
+  return React__default.useContext(GraphStateContext);
+}
 
 /**
  * 44 is touch best practice and AAA compliant for WCAG
@@ -28238,6 +28270,11 @@ const normalizeCoords = (coordsList, ranges) => coordsList.map(coords => coords.
   const extent = ranges[i][1] - ranges[i][0];
   return (coord + ranges[i][1]) / extent;
 }));
+function snap(snapStep, point) {
+  const [requestedX, requestedY] = point;
+  const [snapX, snapY] = snapStep;
+  return [Math.round(requestedX / snapX) * snapX, Math.round(requestedY / snapY) * snapY];
+}
 
 const useTransform = (...points) => {
   const {
@@ -28293,6 +28330,9 @@ const getLines = points => points.map((point, i) => {
 
 const defaultStroke = "var(--movable-line-stroke-color)";
 const MovableLine = props => {
+  const {
+    state
+  } = useGraphState();
   const {
     start,
     end,
@@ -28317,7 +28357,7 @@ const MovableLine = props => {
     onMove: newPoint => {
       onMove(vec.sub(newPoint, midpoint));
     },
-    constrain: p => p
+    constrain: p => snap(state.snapStep, p)
   });
   return /*#__PURE__*/React.createElement(React.Fragment, null, /*#__PURE__*/React.createElement("g", {
     ref: line,
@@ -28368,7 +28408,13 @@ function SVGLine(props) {
   });
 }
 
+// The hitbox size of 48px by 48px is preserved from the legacy interactive
+// graph.
+const hitboxSizePx = 48;
 const StyledMovablePoint = props => {
+  const {
+    state
+  } = useGraphState();
   const hitboxRef = useRef(null);
   const {
     point,
@@ -28381,7 +28427,7 @@ const StyledMovablePoint = props => {
     gestureTarget: hitboxRef,
     point,
     onMove,
-    constrain: p => p
+    constrain: p => snap(state.snapStep, p)
   });
   const [[x, y]] = useTransform(point);
   return /*#__PURE__*/React.createElement("g", {
@@ -28396,7 +28442,7 @@ const StyledMovablePoint = props => {
     }
   }, /*#__PURE__*/React.createElement("circle", {
     className: "movable-point-hitbox",
-    r: TARGET_SIZE / 2,
+    r: hitboxSizePx / 2,
     cx: x,
     cy: y
   }), /*#__PURE__*/React.createElement("circle", {
@@ -28407,6 +28453,10 @@ const StyledMovablePoint = props => {
     className: "movable-point-ring",
     cx: x,
     cy: y
+  }), /*#__PURE__*/React.createElement("circle", {
+    className: "movable-point-focus-outline",
+    cx: x,
+    cy: y
   }), /*#__PURE__*/React.createElement("circle", {
     className: "movable-point-center",
     cx: x,
@@ -28725,7 +28775,8 @@ const PolygonGraph = props => {
     type,
     showAngles,
     showSides,
-    range
+    range,
+    snapStep
   } = props.graphState;
   const points = coords != null ? coords : [[0, 0]];
   const pointsSum = points.reduce((acc, point) => vec.add(acc, point), [0, 0]);
@@ -28740,7 +28791,7 @@ const PolygonGraph = props => {
       const delta = vec.sub(newPoint, midpoint);
       dispatch(moveAll(delta));
     },
-    constrain: p => p
+    constrain: p => snap(snapStep, p)
   });
   const active = hovered || focused || dragging;
   const lines = getLines(points);
@@ -28808,14 +28859,6 @@ function PointGraph(props) {
   })));
 }
 
-const GraphStateContext = /*#__PURE__*/createContext({
-  state: {},
-  dispatch: () => {}
-});
-function useGraphState() {
-  return React__default.useContext(GraphStateContext);
-}
-
 function Arrow(props) {
   const {
     userTransform,
@@ -28859,17 +28902,120 @@ function AxisArrows() {
   }));
 }
 
-const renderLineLabel = (n, [min, max]) => n !== -1 && n !== min && n !== max;
-const axisOptions = (props, index) => ({
-  axis: props.markings === "graph",
-  lines: props.step[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(React.Fragment, null, /*#__PURE__*/React.createElement(Coordinates.Cartesian, {
-  xAxis: axisOptions(props, 0),
-  yAxis: axisOptions(props, 1)
-}), props.markings === "graph" && /*#__PURE__*/React.createElement(AxisArrows, null));
+const tickSize = 10;
+const tickStyle = {
+  stroke: "black",
+  strokeWidth: 1
+};
+const YGridTick = ({
+  y
+}) => {
+  const pointOnAxis = [0, y];
+  const [[xPosition, yPosition]] = useTransform(pointOnAxis);
+  return /*#__PURE__*/React.createElement("g", {
+    className: "y-axis-ticks"
+  }, /*#__PURE__*/React.createElement("line", {
+    x1: xPosition - tickSize / 2,
+    y1: yPosition,
+    x2: xPosition + tickSize / 2,
+    y2: yPosition,
+    style: tickStyle
+  }));
+};
+const XGridTick = ({
+  x
+}) => {
+  const pointOnAxis = [x, 0];
+  const [[xPosition, yPosition]] = useTransform(pointOnAxis);
+  return /*#__PURE__*/React.createElement("g", {
+    className: "x-axis-ticks"
+  }, /*#__PURE__*/React.createElement("line", {
+    x1: xPosition,
+    y1: yPosition + tickSize / 2,
+    x2: xPosition,
+    y2: yPosition - tickSize / 2,
+    style: tickStyle
+  }));
+};
+function generateTickLocations(tickStep, min, max) {
+  const ticks = [];
+  for (let i = 0 + tickStep; i < max; i += tickStep) {
+    ticks.push(i);
+  }
+  for (let i = 0 - tickStep; i > min; i -= tickStep) {
+    ticks.push(i);
+  }
+  return ticks;
+}
+const AxisTicks = props => {
+  const range = props.range;
+  const [xMin, xMax] = range[0];
+  const [yMin, yMax] = range[1];
+  const yTickStep = props.tickStep[1];
+  const xTickStep = props.tickStep[0];
+  const yGridTicks = generateTickLocations(yTickStep, yMin, yMax);
+  const xGridTicks = generateTickLocations(xTickStep, xMin, xMax);
+  return /*#__PURE__*/React.createElement("g", {
+    className: "axis-ticks"
+  }, yGridTicks.map(y => {
+    return /*#__PURE__*/React.createElement(YGridTick, {
+      y: y,
+      key: `y-grid-tick-${y}`
+    });
+  }), xGridTicks.map(x => {
+    return /*#__PURE__*/React.createElement(XGridTick, {
+      x: x,
+      key: `x-grid-tick-${x}`
+    });
+  }));
+};
+
+/**
+ * lineLabelText get the text that should
+ * be displayed for a specific tick place
+ *
+ * @param {number} n the tick number
+ * @param {number} tickStep how frequently a label should appear
+ * @param {vec.Vector2} range the min/max range on the axis
+ */
+const lineLabelText = (n, tickStep, range) => {
+  const [min, max] = range;
+  const isOnStep = n % tickStep === 0;
+  const isNegativeOne = n === -1;
+  const isMin = n === min;
+  const isMax = n === max;
+  const shouldRender = isOnStep && !isNegativeOne && !isMin && !isMax;
+  return shouldRender ? `${n}` : "";
+};
+
+/**
+ * axisOptions determine axis options for Mafs
+ *
+ * axisIndex is for grabbing data in an array that contains
+ * data for multiple axes. For example range: [[-10, 10], [-10, 10]]
+ * range[0] is data for the x axis and range[1] is data for the y axis
+ *
+ * @param {GridProps} props
+ * @param {number} axisIndex which axis we're getting options for
+ */
+const axisOptions = (props, axisIndex) => {
+  const axisStep = props.tickStep[axisIndex];
+  const axisRange = props.range[axisIndex];
+  return {
+    axis: props.markings === "graph",
+    lines: props.gridStep[axisIndex],
+    labels: n => lineLabelText(n, axisStep, axisRange)
+  };
+};
+const Grid = props => {
+  return props.markings === "none" ? null : /*#__PURE__*/React.createElement(React.Fragment, null, /*#__PURE__*/React.createElement(Coordinates.Cartesian, {
+    xAxis: axisOptions(props, 0),
+    yAxis: axisOptions(props, 1)
+  }), /*#__PURE__*/React.createElement(AxisTicks, {
+    range: props.range,
+    tickStep: props.tickStep
+  }), props.markings === "graph" && /*#__PURE__*/React.createElement(AxisArrows, null));
+};
 
 // Displays a stylized open eye: lint warnings are visible
 const VisibleIcon = () => /*#__PURE__*/React.createElement("svg", {
@@ -29576,6 +29722,10 @@ function interactiveGraphReducer(state, action) {
       return doMoveAll(state, action);
     case MOVE_POINT:
       return doMovePoint(state, action);
+    case CHANGE_SNAP_STEP:
+      return doChangeSnapStep(state, action);
+    case CHANGE_RANGE:
+      return doChangeRange(state, action);
     default:
       throw new UnreachableCaseError(action);
   }
@@ -29597,14 +29747,11 @@ function doMoveControlPoint(state, action) {
           update: tuple => setAtIndex({
             array: tuple,
             index: action.pointIndex,
-            newValue: snap({
+            newValue: snap(snapStep, bound$1({
               snapStep,
-              point: bound$1({
-                snapStep,
-                range,
-                point: action.destination
-              })
-            })
+              range,
+              point: action.destination
+            }))
           })
         });
         const coordsToCheck = newCoords[action.itemIndex];
@@ -29647,13 +29794,7 @@ function doMoveLine(state, action) {
           snapStep,
           range
         });
-        const newLine = [snap({
-          snapStep,
-          point: vec.add(currentLine[0], change)
-        }), snap({
-          snapStep,
-          point: vec.add(currentLine[1], change)
-        })];
+        const newLine = [snap(snapStep, vec.add(currentLine[0], change)), snap(snapStep, vec.add(currentLine[1], change))];
         const newCoords = setAtIndex({
           array: state.coords,
           index: action.itemIndex,
@@ -29683,10 +29824,7 @@ function doMoveAll(state, action) {
           snapStep,
           range
         });
-        const newCoords = state.coords.map(point => snap({
-          snapStep,
-          point: vec.add(point, change)
-        }));
+        const newCoords = state.coords.map(point => snap(snapStep, vec.add(point, change)));
         return _extends({}, state, {
           hasBeenInteractedWith: true,
           coords: newCoords
@@ -29707,14 +29845,11 @@ function doMovePoint(state, action) {
           coords: setAtIndex({
             array: state.coords,
             index: action.index,
-            newValue: snap({
+            newValue: snap(state.snapStep, bound$1({
               snapStep: state.snapStep,
-              point: bound$1({
-                snapStep: state.snapStep,
-                range: state.range,
-                point: action.destination
-              })
-            })
+              range: state.range,
+              point: action.destination
+            }))
           })
         });
       }
@@ -29722,6 +29857,26 @@ function doMovePoint(state, action) {
       throw new Error("The movePoint action is only for point and polygon graphs");
   }
 }
+function doChangeSnapStep(state, action) {
+  if (
+  // Deep equality check since these are arrays
+  _.isEqual(state.snapStep, action.snapStep)) {
+    return state;
+  }
+  return _extends({}, state, {
+    snapStep: action.snapStep
+  });
+}
+function doChangeRange(state, action) {
+  if (
+  // Deep equality check since these are arrays
+  _.isEqual(state.range, action.range)) {
+    return state;
+  }
+  return _extends({}, state, {
+    range: action.range
+  });
+}
 const getChange = (coords, delta, constraintOpts) => {
   const [deltaX, deltaY] = delta;
   const maxMoves = coords.map(point => maxMove(_extends({}, constraintOpts, {
@@ -29738,15 +29893,6 @@ const getChange = (coords, delta, constraintOpts) => {
   const dy = clamp(deltaY, minYMove, maxYMove);
   return [dx, dy];
 };
-function snap({
-  snapStep,
-  point
-}) {
-  const [requestedX, requestedY] = point;
-  const [snapX, snapY] = snapStep;
-  return [Math.round(requestedX / snapX) * snapX, Math.round(requestedY / snapY) * snapY];
-}
-
 // Returns the closest point to the given `point` that is within the graph
 // bounds given in `state`.
 function bound$1({
@@ -30098,6 +30244,18 @@ const MafsGraph = /*#__PURE__*/React.forwardRef((props, ref) => {
     }
     prevState.current = state;
   }, [props, state]);
+
+  // Destructuring first to keep useEffect from making excess calls
+  const [xSnap, ySnap] = props.snapStep;
+  useEffect(() => {
+    dispatch(changeSnapStep([xSnap, ySnap]));
+  }, [xSnap, ySnap]);
+
+  // Destructuring first to keep useEffect from making excess calls
+  const [[xMinRange, xMaxRange], [yMinRange, yMaxRange]] = props.range;
+  useEffect(() => {
+    dispatch(changeRange([[xMinRange, xMaxRange], [yMinRange, yMaxRange]]));
+  }, [xMinRange, xMaxRange, yMinRange, yMaxRange]);
   React.useImperativeHandle(ref, () => ({
     getUserInput: () => getGradableGraph(state, props.graph)
   }));
@@ -30132,7 +30290,13 @@ const MafsGraph = /*#__PURE__*/React.forwardRef((props, ref) => {
     zoom: false,
     width: width,
     height: height
-  }, /*#__PURE__*/React.createElement(SvgDefs, null), /*#__PURE__*/React.createElement(Grid, props), props.lockedFigures && /*#__PURE__*/React.createElement(GraphLockedLayer, {
+  }, /*#__PURE__*/React.createElement(SvgDefs, null), /*#__PURE__*/React.createElement(Grid, {
+    tickStep: props.step,
+    gridStep: props.gridStep,
+    range: props.range,
+    containerSizeClass: props.containerSizeClass,
+    markings: props.markings
+  }), props.lockedFigures && /*#__PURE__*/React.createElement(GraphLockedLayer, {
     lockedFigures: props.lockedFigures
   }), renderGraph({
     state,
@@ -31911,9 +32075,9 @@ class InteractiveGraph extends React.Component {
           };
         }
       } else if (userInput.type === "segment" && rubric.correct.type === "segment" && userInput.coords != null) {
-        var _rubric$correct$coord2;
-        let guess = userInput.coords.slice();
-        let correct = (_rubric$correct$coord2 = rubric.correct.coords) == null ? void 0 : _rubric$correct$coord2.slice();
+        var _rubric$correct;
+        let guess = Util.deepClone(userInput.coords);
+        let correct = Util.deepClone((_rubric$correct = rubric.correct) == null ? void 0 : _rubric$correct.coords);
         guess = _.invoke(guess, "sort").sort();
         // @ts-expect-error - TS2345 - Argument of type '(readonly Coord[])[] | undefined' is not assignable to parameter of type 'Collection<any>'.
         correct = _.invoke(correct, "sort").sort();
@@ -42181,7 +42345,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.1.0";
+const libVersion = "21.2.1";
 addLibraryVersionToPerseusDebug(libName, libVersion);
 
 const apiVersion = {
@@ -43305,6 +43469,9 @@ const ItemExtras = [
 // A background image applied to various widgets.
 
 const PerseusExpressionAnswerFormConsidered = ["correct", "wrong", "ungraded"];
+
+// 2D range: xMin, xMax, yMin, yMax
+
 const plotterPlotTypes = ["bar", "line", "pic", "histogram", "dotplot"];
 
 /**