@khanacademy/perseus 1.3.2 → 1.3.4

package/dist/index.css

@@ -1527,16 +1527,16 @@ body.mobile ul.inequalities-one-line-radios > li {
   font-family: inherit;
   font-size: 14px;
   line-height: 1.25;
-  color: #888D93;
+  color: #21242C;
 }
 .framework-perseus:not(.perseus-article):not(.perseus-mobile) .perseus-radio-rationale-content .perseus-renderer > .paragraph .paragraph {
   font-family: inherit;
   font-size: 14px;
   line-height: 1.25;
-  color: #888D93;
+  color: #21242C;
 }
 .framework-perseus:not(.perseus-article):not(.perseus-mobile) .perseus-radio-rationale-content .perseus-renderer > .paragraph .paragraph .katex {
-  color: #888D93;
+  color: #21242C;
 }
 .bibliotron-article.framework-perseus.perseus-article.perseus-mobile :not(blockquote) > div.paragraph,
 .framework-perseus:not(.perseus-article).perseus-mobile :not(blockquote) > div.paragraph {
@@ -1696,18 +1696,18 @@ body.mobile ul.inequalities-one-line-radios > li {
     font-family: inherit;
     font-size: 16px;
     line-height: 1.25;
-    color: #888D93;
+    color: #21242C;
   }
   .bibliotron-article.framework-perseus.perseus-article.perseus-mobile .perseus-radio-rationale-content .perseus-renderer > .paragraph .paragraph,
   .framework-perseus:not(.perseus-article).perseus-mobile .perseus-radio-rationale-content .perseus-renderer > .paragraph .paragraph {
     font-family: inherit;
     font-size: 16px;
     line-height: 1.25;
-    color: #888D93;
+    color: #21242C;
   }
   .bibliotron-article.framework-perseus.perseus-article.perseus-mobile .perseus-radio-rationale-content .perseus-renderer > .paragraph .paragraph .katex,
   .framework-perseus:not(.perseus-article).perseus-mobile .perseus-radio-rationale-content .perseus-renderer > .paragraph .paragraph .katex {
-    color: #888D93;
+    color: #21242C;
   }
   .bibliotron-article.framework-perseus.perseus-article.perseus-mobile .perseus-image-caption .paragraph .paragraph,
   .framework-perseus:not(.perseus-article).perseus-mobile .perseus-image-caption .paragraph .paragraph {
@@ -1859,18 +1859,18 @@ body.mobile ul.inequalities-one-line-radios > li {
     font-family: inherit;
     font-size: 18px;
     line-height: 1.25;
-    color: #888D93;
+    color: #21242C;
   }
   .bibliotron-article.framework-perseus.perseus-article.perseus-mobile .perseus-radio-rationale-content .perseus-renderer > .paragraph .paragraph,
   .framework-perseus:not(.perseus-article).perseus-mobile .perseus-radio-rationale-content .perseus-renderer > .paragraph .paragraph {
     font-family: inherit;
     font-size: 18px;
     line-height: 1.25;
-    color: #888D93;
+    color: #21242C;
   }
   .bibliotron-article.framework-perseus.perseus-article.perseus-mobile .perseus-radio-rationale-content .perseus-renderer > .paragraph .paragraph .katex,
   .framework-perseus:not(.perseus-article).perseus-mobile .perseus-radio-rationale-content .perseus-renderer > .paragraph .paragraph .katex {
-    color: #888D93;
+    color: #21242C;
   }
   .bibliotron-article.framework-perseus.perseus-article.perseus-mobile .perseus-image-caption .paragraph .paragraph,
   .framework-perseus:not(.perseus-article).perseus-mobile .perseus-image-caption .paragraph .paragraph {
@@ -2022,18 +2022,18 @@ body.mobile ul.inequalities-one-line-radios > li {
     font-family: inherit;
     font-size: 20px;
     line-height: 1.25;
-    color: #888D93;
+    color: #21242C;
   }
   .bibliotron-article.framework-perseus.perseus-article.perseus-mobile .perseus-radio-rationale-content .perseus-renderer > .paragraph .paragraph,
   .framework-perseus:not(.perseus-article).perseus-mobile .perseus-radio-rationale-content .perseus-renderer > .paragraph .paragraph {
     font-family: inherit;
     font-size: 20px;
     line-height: 1.25;
-    color: #888D93;
+    color: #21242C;
   }
   .bibliotron-article.framework-perseus.perseus-article.perseus-mobile .perseus-radio-rationale-content .perseus-renderer > .paragraph .paragraph .katex,
   .framework-perseus:not(.perseus-article).perseus-mobile .perseus-radio-rationale-content .perseus-renderer > .paragraph .paragraph .katex {
-    color: #888D93;
+    color: #21242C;
   }
   .bibliotron-article.framework-perseus.perseus-article.perseus-mobile .perseus-image-caption .paragraph .paragraph,
   .framework-perseus:not(.perseus-article).perseus-mobile .perseus-image-caption .paragraph .paragraph {

package/dist/index.js

@@ -19343,7 +19343,7 @@ class Renderer extends React__namespace.Component {
         // look like a widget ID (ie. `widget-type \d+`). I can't figure
         // out how to trigger this line of code though.
 
-        /* istanbul ignore next line */
+        /* c8 ignore next line */
         return {};
       }
 
@@ -19682,7 +19682,7 @@ class Renderer extends React__namespace.Component {
           // reached. As noted above, these are always QuestionParagraphs
           // now.
 
-          /* istanbul ignore if */
+          /* c8 ignore if */
 
           if (typeof nodeOut === "string" && lastWasString) {
             /**
@@ -22384,12 +22384,12 @@ class Expression extends React__namespace.Component {
     });
 
     _defineProperty(this, "_handleFocus", () => {
-      /* istanbul ignore next */
+      /* c8 ignore next */
       this.props.onFocus([]);
     });
 
     _defineProperty(this, "_handleBlur", () => {
-      /* istanbul ignore next */
+      /* c8 ignore next */
       this.props.onBlur([]);
     });
 
@@ -22423,12 +22423,12 @@ class Expression extends React__namespace.Component {
       // The widget itself is an input, so we return a single empty list to
       // indicate this.
 
-      /* istanbul ignore next */
+      /* c8 ignore next */
       return [[]];
     });
 
     _defineProperty(this, "getGrammarTypeForPath", inputPath => {
-      /* istanbul ignore next */
+      /* c8 ignore next */
       return "expression";
     });
 
@@ -22478,7 +22478,7 @@ class Expression extends React__namespace.Component {
       // in the function variables list for the expression.
 
       if (!expression.parsed) {
-        /* istanbul ignore next */
+        /* c8 ignore next */
         Log.error("Unable to parse solution answer for expression", Errors.InvalidInput, {
           loggedMetadata: {
             rubric: JSON.stringify(rubric)
@@ -22782,7 +22782,7 @@ const keypadConfigurationForProps = widgetOptions => {
 };
 
 const propUpgrades$2 = {
-  /* istanbul ignore next */
+  /* c8 ignore next */
   "1": v0props => ({
     times: v0props.times,
     buttonSets: v0props.buttonSets,
@@ -23050,12 +23050,12 @@ class InputNumber extends React__namespace.Component {
       // The widget itself is an input, so we return a single empty list to
       // indicate this.
 
-      /* istanbul ignore next */
+      /* c8 ignore next */
       return [[]];
     });
 
     _defineProperty(this, "getGrammarTypeForPath", path => {
-      /* istanbul ignore next */
+      /* c8 ignore next */
       return "number";
     });
 
@@ -23116,7 +23116,7 @@ class InputNumber extends React__namespace.Component {
     let correct = null;
     let answerBlurb = null; // SAT is deprecated and not testable
 
-    /* istanbul ignore if */
+    /* c8 ignore if */
 
     if (this.props.apiOptions.satStyling && rubric) {
       const score = this.simpleValidate(rubric);
@@ -23160,7 +23160,7 @@ class InputNumber extends React__namespace.Component {
       linterContext: this.props.linterContext
     }); // SAT is deprecated, answer blurb is an SAT feature
 
-    /* istanbul ignore if */
+    /* c8 ignore if */
 
     if (answerBlurb) {
       return /*#__PURE__*/React__namespace.createElement("span", {
@@ -23321,7 +23321,7 @@ class NumericInput extends React__namespace.Component {
       let answerBlurb;
       /* Ignore this block because SAT is deprecated */
 
-      /* istanbul ignore next */
+      /* c8 ignore next */
 
       if (this.props.apiOptions.satStyling && rubric) {
         const score = this.simpleValidate(rubric);
@@ -23420,17 +23420,17 @@ class NumericInput extends React__namespace.Component {
       // The widget itself is an input, so we return a single empty list to
       // indicate this.
 
-      /* istanbul ignore next */
+      /* c8 ignore next */
       return [[]];
     });
 
     _defineProperty(this, "getGrammarTypeForPath", inputPath => {
-      /* istanbul ignore next */
+      /* c8 ignore next */
       return "number";
     });
 
     _defineProperty(this, "setInputValue", (path, newValue, cb) => {
-      /* istanbul ignore next */
+      /* c8 ignore next */
       this.props.onChange({
         currentValue: newValue
       }, cb);
@@ -23639,7 +23639,7 @@ class NumericInput extends React__namespace.Component {
     });
     /* This only gets used with SAT Content */
 
-    /* istanbul ignore next if */
+    /* c8 ignore next if */
 
     if (answerBlurb) {
       return /*#__PURE__*/React__namespace.createElement("span", {
@@ -26872,8 +26872,8 @@ function LibraryChoiceIcon(props) {
   }, /*#__PURE__*/React__namespace.createElement("div", {
     style: dynamicStyles,
     "data-test-id": "choice-icon__library-choice-icon",
-    className: aphrodite.css(styles$r.circle, showCorrectness && correct && styles$r.circleCorrect, showCorrectness && !correct && styles$r.circleIncorrect, showCorrectness && !correct && (checked || previouslyAnswered) && styles$r.circleIncorrectAnswered) // used in BaseRadio to check if we actually clicked on the
-    // radio icon
+    className: aphrodite.css(styles$r.circle, showCorrectness && correct && styles$r.circleCorrect, showCorrectness && !correct && styles$r.circleIncorrect, showCorrectness && !correct && (checked || previouslyAnswered) && styles$r.circleIncorrectAnswered) // used in BaseRadio editMode to check
+    // if we actually clicked on the radio icon
     ,
     "data-is-radio-icon": true
   }, /*#__PURE__*/React__namespace.createElement("div", {
@@ -27298,7 +27298,8 @@ function Choice(props) {
     },
     disabled: disabled || apiOptions.staticRender || apiOptions.readOnly,
     style: {
-      flex: 1
+      flex: 1,
+      color: Color__default["default"].offBlack
     },
     ref: forwardedRef,
     "aria-hidden": "true"
@@ -27654,7 +27655,7 @@ function BaseRadio(props) {
     style: {
       listStyle: "none"
     }
-  }, choices.map(function (choice, i) {
+  }, choices.map((choice, i) => {
     let Element = Choice$1;
     const ref = /*#__PURE__*/React__namespace.createRef();
     choiceRefs.current[i] = ref;
@@ -27673,6 +27674,11 @@ function BaseRadio(props) {
       showRationale: choice.hasRationale && (reviewMode || choice.showRationale),
       pos: i,
       onChange: newValues => {
+        // editMode selection is handled in clickHandler
+        if (editMode) {
+          return;
+        }
+
         updateChoice(i, newValues);
       },
       ref
@@ -27730,7 +27736,7 @@ function BaseRadio(props) {
           // radio icon, then we want to trigger the
           // check by flipping the choice of the icon.
           if (elem.getAttribute("data-is-radio-icon")) {
-            this.updateChoice(i, {
+            updateChoice(i, {
               checked: !choice.checked,
               crossedOut: choice.crossedOut
             });
@@ -27754,7 +27760,7 @@ function BaseRadio(props) {
       onClick: clickHandler,
       onTouchStart: !labelWrap ? null : captureScratchpadTouchStart$2
     }, /*#__PURE__*/React__namespace.createElement(Element, elementProps));
-  }, this))); // Allow for horizontal scrolling if content is too wide, which may be
+  }))); // Allow for horizontal scrolling if content is too wide, which may be
   // an issue especially on phones.
   // This is disabled in SAT, since it conflicts with their theming.
 
@@ -28101,7 +28107,7 @@ class Radio$1 extends React__namespace.Component {
         noneOfTheAboveSelected
       }; // Support legacy choiceState implementation
     }
-    /* istanbul ignore if - props.values is deprecated */
+    /* c8 ignore if - props.values is deprecated */
 
 
     const {
@@ -28196,7 +28202,7 @@ class Radio$1 extends React__namespace.Component {
     } else if (this.props.values) {
       // Support legacy choiceStates implementation
 
-      /* istanbul ignore next - props.values is deprecated */
+      /* c8 ignore next - props.values is deprecated */
       choiceStates = this.props.values.map(val => ({
         selected: val,
         crossedOut: false,
@@ -28579,7 +28585,7 @@ _defineProperty(Categorizer, "defaultProps", {
 
 const pageMargin = 16; // Stylesheets aren't directly testable
 
-/* istanbul ignore next */
+/* c8 ignore next */
 
 const styles$m = aphrodite.StyleSheet.create({
   mobileTable: {