Cleaning up PR

Khan · Oct 10, 2023 · 1c49d4b · 1c49d4b
1 parent ddf9f55
commit 1c49d4b
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 19 deletions.
diff --git a/packages/perseus/src/__stories__/article-renderer.stories.tsx b/packages/perseus/src/__stories__/article-renderer.stories.tsx
@@ -67,14 +67,6 @@ export const ExpressionArticle = ({useNewStyles}): any => (
                     apiOptions={{
                         isMobile: true,
                         customKeypad: true,
-                        onFocusChange(
-                            newFocusPath,
-                            oldFocusPath,
-                            keypadElement,
-                            focusedElement,
-                        ) {
-                            // console.log(focusedElement);
-                        },
                     }}
                     keypadElement={keypadElement}
                 />
@@ -97,14 +89,6 @@ export const MultiSectionedExpressionArticle = ({useNewStyles}): any => (
                     apiOptions={{
                         isMobile: true,
                         customKeypad: true,
-                        onFocusChange(
-                            newFocusPath,
-                            oldFocusPath,
-                            keypadElement,
-                            focusedElement,
-                        ) {
-                            // console.log(focusedElement);
-                        },
                     }}
                     keypadElement={keypadElement}
                 />

diff --git a/packages/perseus/src/article-renderer.tsx b/packages/perseus/src/article-renderer.tsx
@@ -92,7 +92,6 @@ class ArticleRenderer
 
         if (this._currentFocus) {
             const [sectionRef, ...focusPath] = this._currentFocus;
-            // eslint-disable-next-line react/no-string-refs
 
             const inputPaths =
                 this.sectionRenderers[sectionRef].getInputPaths();
@@ -146,7 +145,6 @@ class ArticleRenderer
     blur: () => void = () => {
         if (this._currentFocus) {
             const [sectionRef, ...inputPath] = this._currentFocus;
-            // eslint-disable-next-line react/no-string-refs
             this.sectionRenderers[sectionRef].blurPath(inputPath);
         }
     };
@@ -176,7 +174,10 @@ class ArticleRenderer
         const sections = this._sections().map((section, i) => {
             const refForSection: any = i;
             return (
-                <DependenciesContext.Provider value={this.props.dependencies}>
+                <DependenciesContext.Provider
+                    key={i}
+                    value={this.props.dependencies}
+                >
                     <div key={i} className="clearfix">
                         <Renderer
                             {...section}