@khanacademy/perseus 29.0.0 → 30.0.1

package/dist/article-renderer.d.ts

@@ -7,18 +7,15 @@ import { PerseusI18nContext } from "./components/i18n-context";
 import { DependenciesContext } from "./dependencies";
 import Renderer from "./renderer";
 import type { PerseusRenderer } from "./perseus-types";
-import type { APIOptions, PerseusDependenciesV2 } from "./types";
+import type { PerseusDependenciesV2, SharedRendererProps } from "./types";
 import type { KeypadAPI } from "@khanacademy/math-input";
 import type { KeypadContextRendererInterface } from "@khanacademy/perseus-core";
-import type { LinterContextProps } from "@khanacademy/perseus-linter";
-type Props = Partial<React.ContextType<typeof DependenciesContext>> & {
-    apiOptions: APIOptions;
+type Props = Partial<React.ContextType<typeof DependenciesContext>> & SharedRendererProps & {
     json: PerseusRenderer | ReadonlyArray<PerseusRenderer>;
     /**
      * @deprecated Does nothing
      */
     useNewStyles: boolean;
-    linterContext: LinterContextProps;
     legacyPerseusLint?: ReadonlyArray<string>;
     keypadElement?: KeypadAPI | null | undefined;
     dependencies: PerseusDependenciesV2;

package/dist/components/lint.d.ts

@@ -14,6 +14,9 @@ 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
@@ -36,8 +39,12 @@ type Props = {
  * 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> {
+declare class Lint extends React.Component<Props, State> {
     _positionTimeout: number | undefined;
+    state: State;
+    componentDidMount(): void;
+    componentWillUnmount(): void;
+    getPosition: () => void;
     renderLink: (arg1: any) => React.ReactElement;
     render(): React.ReactNode;
 }

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, font } from '@khanacademy/wonder-blocks-tokens';
+import { color, spacing } from '@khanacademy/wonder-blocks-tokens';
 import $ from 'jquery';
 import { v4 } from 'uuid';
 import PropTypes from 'prop-types';
@@ -247,7 +247,10 @@ const mockStrings = {
   somethingWrong: "Something went wrong.",
   videoWrapper: "Khan Academy video wrapper",
   mathInputTitle: "mathematics keyboard",
-  mathInputDescription: "Use keyboard/mouse to interact with math-based input fields"
+  mathInputDescription: "Use keyboard/mouse to interact with math-based input fields",
+  sin: "sin",
+  cos: "cos",
+  tan: "tan"
 };
 
 /**
@@ -328,7 +331,7 @@ class InnerMathInput extends React.Component {
       const {
         locale
       } = this.context;
-      const customKeyTranslator = _extends({}, getKeyTranslator(locale), {
+      const customKeyTranslator = _extends({}, getKeyTranslator(locale, this.context.strings), {
         // If there's something in the input that can become part of a
         // fraction, typing "/" puts it in the numerator. If not, typing
         // "/" does nothing. In that case, enter a \frac.
@@ -447,7 +450,7 @@ class InnerMathInput extends React.Component {
       const {
         locale
       } = this.context;
-      const translator = getKeyTranslator(locale)[key];
+      const translator = getKeyTranslator(locale, this.context.strings)[key];
       const mathField = this.mathField();
       if (mathField) {
         if (translator) {
@@ -18171,9 +18174,27 @@ 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;
@@ -18190,23 +18211,34 @@ class Lint extends React.Component {
         warningText = "Recommendation";
         warningTextStyle = styles$B.warning;
       }
-      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", {
+      return /*#__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))));
+      }, 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)
+      })));
     };
   }
+  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.
@@ -18407,11 +18439,54 @@ 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
@@ -44201,7 +44276,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 = "29.0.0";
+const libVersion = "30.0.1";
 addLibraryVersionToPerseusDebug(libName, libVersion);
 
 const apiVersion = {