@khanacademy/perseus 28.1.0 → 29.0.0

package/dist/components/lint.d.ts

@@ -14,9 +14,6 @@ type Props = {
     blockHighlight?: boolean;
     severity?: Severity;
 };
-type State = {
-    tooltipAbove: boolean;
-};
 /**
  * This component renders "lint" nodes in a markdown parse tree. Lint nodes
  * are inserted into the tree by the Perseus linter (see
@@ -39,12 +36,8 @@ type State = {
  * that has a right margin (like anything blockquoted) the circle will appear
  * to the left of where it belongs.  And if there is more
  **/
-declare class Lint extends React.Component<Props, State> {
+declare class Lint extends React.Component<Props> {
     _positionTimeout: number | undefined;
-    state: State;
-    componentDidMount(): void;
-    componentWillUnmount(): void;
-    getPosition: () => void;
     renderLink: (arg1: any) => React.ReactElement;
     render(): React.ReactNode;
 }

package/dist/es/index.css

@@ -1500,7 +1500,6 @@ body.mobile ul.inequalities-one-line-radios > li {
 .perseus-clearfix:after {
   clear: both;
 }
-/* Fall back colors? Eg educator, talks & interviews */
 .framework-perseus code {
   font-family: Courier, monospace;
 }
@@ -2418,7 +2417,7 @@ img.zoom-img {
   width: 40px;
 }
 .framework-perseus .number-input.invalid-input {
-  background-color: #cf5044;
+  background-color: #ffbaba;
   outline-color: red;
 }
 .framework-perseus .number-input.mini {
@@ -2604,7 +2603,7 @@ img.zoom-img {
   z-index: 2;
 }
 body.mobile .categorizer-container td.category input[type="radio"]:checked + span:before {
-  color: #005a88;
+  color: #1c758a;
 }
 body.mobile .categorizer-container td.category input[type="radio"] + span:active:before {
   color: #666;
@@ -2677,7 +2676,7 @@ body.mobile .categorizer-container td.category input[type="radio"] + span:active
   margin-left: 0;
 }
 .framework-perseus .perseus-graded-group.answer-incorrect {
-  border-left: 3px solid #b2392e;
+  border-left: 3px solid #ff8787;
   margin-left: 0;
 }
 .framework-perseus .perseus-graded-group .group-icon {
@@ -3490,7 +3489,7 @@ body.mobile .framework-perseus table.perseus-widget-table-of-values.non-markdown
   background-color: #fcc;
 }
 .perseus-error {
-  background: #cf5044;
+  background: #ffbaba;
   border: 2px solid red;
   border-radius: 5px;
   padding: 20px;

package/dist/es/index.js

@@ -15,7 +15,7 @@ import ReactDOM__default from 'react-dom';
 import _, { debounce as debounce$1 } from 'underscore';
 import Clickable from '@khanacademy/wonder-blocks-clickable';
 import { Popover, PopoverContentCore, PopoverContent } from '@khanacademy/wonder-blocks-popover';
-import { color, spacing } from '@khanacademy/wonder-blocks-tokens';
+import { color, spacing, font } from '@khanacademy/wonder-blocks-tokens';
 import $ from 'jquery';
 import { v4 } from 'uuid';
 import PropTypes from 'prop-types';
@@ -766,7 +766,7 @@ const denylist = [
 // (technically the renderer still adds them)
 "key", "ref",
 // added by src/renderer.jsx
-"containerSizeClass", "widgetId", "onChange", "problemNum", "apiOptions", "questionCompleted", "findWidgets", "hintMode",
+"containerSizeClass", "widgetId", "onChange", "problemNum", "apiOptions", "questionCompleted", "findWidgets",
 // added by src/editor.jsx, for widgets removing themselves
 // this is soooo not the right place for this, but alas.
 "onRemove",
@@ -1293,7 +1293,7 @@ const KhanMath = {
    * with an approx symbol if num had to be rounded, and trailing 0s
    */
   toFixedApprox: function (num, precision) {
-    // TODO(aria): Make this locale-dependent like KhanUtil.localeToFixed
+    // TODO(aria): Make this locale-dependent
     const fixedStr = num.toFixed(precision);
     if (number.equal(+fixedStr, num)) {
       return fixedStr;
@@ -18171,27 +18171,9 @@ class Lint extends React.Component {
   constructor(...args) {
     super(...args);
     this._positionTimeout = void 0;
-    this.state = {
-      tooltipAbove: true
-    };
-    // We can't call setState in componentDidMount without risking a render
-    // thrash, and we can't call getBoundingClientRect in render, so we
-    // borrow a timeout approach from learnstorm-dashboard.jsx and set our
-    // state once the component has mounted and we can get what we need.
-    this.getPosition = () => {
-      // @ts-expect-error - TS2531 - Object is possibly 'null'. | TS2339 - Property 'getBoundingClientRect' does not exist on type 'Element | Text'.
-      const rect = ReactDOM__default.findDOMNode(this).getBoundingClientRect();
-      // TODO(scottgrant): This is a magic number! We don't know the size
-      // of the tooltip at this point, so we're arbitrarily choosing a
-      // point at which to flip the tooltip's position.
-      this.setState({
-        tooltipAbove: rect.top > 100
-      });
-    };
     // Render the <a> element that holds the indicator icon and the tooltip
     // We pass different styles for the inline and block cases
     this.renderLink = style => {
-      const tooltipAbove = this.state.tooltipAbove;
       let severityStyle;
       let warningText;
       let warningTextStyle;
@@ -18208,34 +18190,23 @@ class Lint extends React.Component {
         warningText = "Recommendation";
         warningTextStyle = styles$B.warning;
       }
-      return /*#__PURE__*/React.createElement("a", {
+      return /*#__PURE__*/React.createElement(Tooltip$1, {
+        backgroundColor: "offBlack",
+        content: /*#__PURE__*/React.createElement(React.Fragment, null, this.props.message.split("\n\n").map((m, i) => /*#__PURE__*/React.createElement("p", {
+          key: i,
+          className: css(styles$B.tooltipParagraph)
+        }, /*#__PURE__*/React.createElement("span", {
+          className: css(warningTextStyle)
+        }, warningText, ":", " "), m)))
+      }, /*#__PURE__*/React.createElement("a", {
         href: `https://khanacademy.org/r/linter-rules#${this.props.ruleName}`,
         target: "lint-help-window",
         className: css(style)
       }, /*#__PURE__*/React.createElement("span", {
         className: css(styles$B.indicator, severityStyle)
-      }, this.props.severity === 1 && /*#__PURE__*/React.createElement(InlineIcon, exclamationIcon)), /*#__PURE__*/React.createElement("div", {
-        className: css(styles$B.tooltip, tooltipAbove && styles$B.tooltipAbove)
-      }, this.props.message.split("\n\n").map((m, i) => /*#__PURE__*/React.createElement("p", {
-        key: i,
-        className: css(styles$B.tooltipParagraph)
-      }, /*#__PURE__*/React.createElement("span", {
-        className: css(warningTextStyle)
-      }, warningText, ":", " "), m)), /*#__PURE__*/React.createElement("div", {
-        className: css(styles$B.tail, tooltipAbove && styles$B.tailAbove)
-      })));
+      }, this.props.severity === 1 && /*#__PURE__*/React.createElement(InlineIcon, exclamationIcon))));
     };
   }
-  componentDidMount() {
-    // TODO(somewhatabstract): Use WB timing
-    // eslint-disable-next-line no-restricted-syntax
-    this._positionTimeout = window.setTimeout(this.getPosition);
-  }
-  componentWillUnmount() {
-    // TODO(somewhatabstract): Use WB timing
-    // eslint-disable-next-line no-restricted-syntax
-    window.clearTimeout(this._positionTimeout);
-  }
   // The main render method surrounds linty content with a block or
   // inline container and the link element that displays the indicator
   // and holds the tooltip.
@@ -18436,54 +18407,11 @@ const styles$B = StyleSheet.create({
   indicatorGuideline: {
     backgroundColor: "#ffbe26"
   },
-  // These are the styles for the tooltip
-  tooltip: {
-    // Absolute positioning relative to the lint indicator circle.
-    position: "absolute",
-    right: -12,
-    // The tooltip is hidden by default; only displayed on hover
-    display: "none",
-    // When it is displayed, it goes on top!
-    zIndex: 1000,
-    // These styles control what the tooltip looks like
-    color: white$1,
-    backgroundColor: gray17,
-    opacity: 0.9,
-    fontFamily: baseFontFamily$1,
-    fontSize: "12px",
-    lineHeight: "15px",
-    width: "320px",
-    borderRadius: "4px"
-  },
-  // If we're going to render the tooltip above the warning circle, we use
-  // the previous rules in tooltip, but change the position slightly.
-  tooltipAbove: {
-    bottom: 32
-  },
-  // We give the tooltip a little triangular "tail" that points down at
-  // the lint indicator circle. This is inside the tooltip and positioned
-  // relative to it. It also shares the opacity of the tooltip. We're using
-  // the standard CSS trick for drawing triangles with a thick border.
-  tail: {
-    position: "absolute",
-    top: -12,
-    right: 16,
-    width: 0,
-    height: 0,
-    // This is the CSS triangle trick
-    borderLeft: "8px solid transparent",
-    borderRight: "8px solid transparent",
-    borderBottom: "12px solid " + gray17
-  },
-  tailAbove: {
-    bottom: -12,
-    borderBottom: "none",
-    borderTop: "12px solid " + gray17,
-    top: "auto"
-  },
   // Each warning in the tooltip is its own <p>. They are 12 pixels from
   // the edges of the tooltip and 12 pixels from each other.
   tooltipParagraph: {
+    fontFamily: font.family.sans,
+    color: color.white,
     margin: 12
   },
   // The text "Warning" inside the tooltip is highlighted like this
@@ -18972,7 +18900,7 @@ const {
 // Note: these size cutoffs represent content-width cutoffs as specified in
 // http://zpl.io/1mVmvU (broken link - we no longer use Zeplin)
 // TODO(benkomalo): these values aren't used in JS outside of this file, but
-// are coupled to the values in styles/articles.less - DRY it up at some point
+// are coupled to the values in styles/styles.less - DRY it up at some point
 const smMax = articleMaxWidthTableInPx;
 const mdMax = articleMaxWidthInPx;
 const containerSizeClass = {
@@ -19763,9 +19691,7 @@ class Renderer extends React.Component {
       return _extends({}, widgetProps, {
         widgetId: id,
         alignment: widgetInfo && widgetInfo.alignment,
-        // When determining if a widget is static, we verify that the widget is not an
-        // exercise question by verifying that it has no problem number.
-        static: widgetInfo && widgetInfo.static && !this.props.problemNum,
+        static: widgetInfo == null ? void 0 : widgetInfo.static,
         problemNum: this.props.problemNum,
         apiOptions: this.getApiOptions(),
         keypadElement: this.props.keypadElement,
@@ -19775,7 +19701,6 @@ class Renderer extends React.Component {
         onBlur: _.partial(this._onWidgetBlur, id),
         findWidgets: this.findWidgets,
         reviewModeRubric: reviewModeRubric,
-        hintMode: this.props.hintMode,
         onChange: (newProps, cb, silent = false) => {
           this._setWidgetProps(id, newProps, cb, silent);
         },
@@ -20150,7 +20075,7 @@ class Renderer extends React.Component {
         }
         // Widgets can contain text nodes, so we don't center them with
         // markdown magic here.
-        // Instead, we center them with css magic in articles.less
+        // Instead, we center them with css magic in styles.less
         // /cry(aria)
         this._foundTextNodes = true;
         if (_.contains(this.widgetIds, node.id)) {
@@ -20978,7 +20903,6 @@ Renderer.defaultProps = {
   findExternalWidgets: () => [],
   alwaysUpdate: false,
   reviewMode: false,
-  hintMode: false,
   serializedState: null,
   onSerializedStateUpdated: () => {},
   linterContext: PerseusLinter.linterContextDefault
@@ -22581,7 +22505,7 @@ var sharedStyles = StyleSheet.create({
     zIndex: zIndexAboveScratchpad
   },
   blankBackground: {
-    // TODO(emily): Use KhanUtil._BACKGROUND?
+    // TODO(emily): Use WB colors?
     backgroundColor: "#FDFDFD"
   },
   perseusSrOnly: {
@@ -22596,7 +22520,7 @@ var sharedStyles = StyleSheet.create({
   },
   responsiveLabel: {
     // NOTE(charlie): The values used here should be kept in sync with the
-    // caption text sizing in Perseus' articles.less.
+    // caption text sizing in Perseus' styles.less.
     // TODO(charlie): Migrate the captions over to using this style.
     [mediaQueries.smOrSmaller]: {
       fontSize: 14,
@@ -26915,7 +26839,7 @@ const propTransform$2 = editorProps => {
 
 // Note that in addition to the standard staticTransform, in static
 // mode we set static=true for the graph's handles in FunctionGrapher.
-const staticTransform$3 = editorProps => {
+const staticTransform$4 = editorProps => {
   return _extends({}, propTransform$2(editorProps), {
     // Don't display graph type choices if we're in static mode
     availableTypes: [editorProps.correct.type],
@@ -26928,7 +26852,7 @@ var Grapher$1 = {
   displayName: "Grapher",
   widget: Grapher,
   transform: propTransform$2,
-  staticTransform: staticTransform$3
+  staticTransform: staticTransform$4
 };
 
 // exports has no 'transform'
@@ -29696,6 +29620,18 @@ const LegacyGrid = ({
 
 // Range is replaced within this file with Interval, but it is used elsewhere
 // and exported from the package, so we need to keep it around.
+/**
+ * A "PerseusItem" is a classic Perseus item. It is rendered by the
+ * `ServerItemRenderer` and the layout is pre-set.
+ *
+ * To render more complex Perseus items, see the `Item` type in the multi item
+ * area.
+ */
+/**
+ * A "MultiItem" is an advanced Perseus item. It is rendered by the
+ * `MultiRenderer` and you can control the layout of individual parts of the
+ * item.
+ */
 const ItemExtras = [
 // The user might benefit from using a Scientific Calculator.  Provided on Khan Academy when true
 "calculator",
@@ -31868,7 +31804,6 @@ const MafsGraph = props => {
     state,
     dispatch,
     labels,
-    hintMode,
     readOnly
   } = props;
   const [width, height] = props.box;
@@ -31885,7 +31820,7 @@ const MafsGraph = props => {
       width,
       height,
       labels,
-      disableKeyboardInteraction: hintMode || readOnly
+      disableKeyboardInteraction: readOnly || !!props.static
     }
   }, /*#__PURE__*/React.createElement(View, {
     className: "mafs-graph",
@@ -31897,7 +31832,8 @@ const MafsGraph = props => {
       boxSizing: "content-box",
       marginLeft: "20px",
       marginBottom: "20px",
-      pointerEvents: hintMode ? "none" : "auto"
+      pointerEvents: props.static ? "none" : "auto",
+      userSelect: "none"
     }
   }, /*#__PURE__*/React.createElement(LegacyGrid, {
     box: props.box,
@@ -33013,6 +32949,17 @@ const StatefulMafsGraph = /*#__PURE__*/React.forwardRef((props, ref) => {
       dispatch(reinitialize(latestPropsRef.current));
     }
   }, [graph.type, numPoints, numSegments, numSides, snapTo, showAngles, showSides, latestPropsRef, graph.startCoords]);
+
+  // If the graph is static, it always displays the correct answer. This is
+  // standard behavior for Perseus widgets (e.g. compare the Radio widget).
+  if (props.static) {
+    return /*#__PURE__*/React.createElement(MafsGraph, _extends({}, props, {
+      state: initializeGraphState(_extends({}, props, {
+        graph: props.correct
+      })),
+      dispatch: dispatch
+    }));
+  }
   return /*#__PURE__*/React.createElement(MafsGraph, _extends({}, props, {
     state: state,
     dispatch: dispatch
@@ -34332,7 +34279,6 @@ class InteractiveGraph extends React.Component {
         snapStep: snapStep,
         box: box,
         showTooltips: !!this.props.showTooltips,
-        hintMode: this.props.hintMode,
         readOnly: (_this$props$apiOption2 = this.props.apiOptions) == null ? void 0 : _this$props$apiOption2.readOnly
       }));
     }
@@ -34922,10 +34868,13 @@ function shouldUseMafs(mafsFlags, graph) {
       return Boolean(mafsFlags[graph.type]);
   }
 }
+// We don't need to change any of the original props for static mode
+const staticTransform$3 = _.identity;
 var InteractiveGraph$1 = {
   name: "interactive-graph",
   displayName: "Interactive graph",
-  widget: InteractiveGraph
+  widget: InteractiveGraph,
+  staticTransform: staticTransform$3
 };
 
 const black = "#000000";
@@ -44252,7 +44201,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 = "28.1.0";
+const libVersion = "29.0.0";
 addLibraryVersionToPerseusDebug(libName, libVersion);
 
 const apiVersion = {
@@ -44494,7 +44443,6 @@ class HintRenderer extends React.Component {
       // eslint-disable-next-line react/no-string-refs
       , {
         ref: "renderer",
-        hintMode: true,
         widgets: hint.widgets,
         content: hint.content || "",
         images: hint.images,
@@ -47663,7 +47611,6 @@ const genericPerseusItemData = {
     minor: 1
   },
   hints: [],
-  _multi: null,
   answer: null
 };