From 4a7e359fba7d66c63f8f87d0dd528d7dd1b880d7 Mon Sep 17 00:00:00 2001
From: Nick Rowe <nixterrimus@dcxn.com>
Date: Thu, 5 Dec 2019 10:48:33 -0700
Subject: [PATCH] Hardware keyboard input (#36)

Adds the ability to use a hardware keyboard with the math input. This allows basic input of numbers, operators, and the delete key.

In the future we could choose to allow multi character input (like != or >=) and complex input (like sin, cos, tan), and more movement keys (like arrows, control+a, control+e). All of these things will greatly increase the complexity in the implementation and might be platform dependent so we want to wait until learners give a clear signal that they want these features.
---
 src/components/input/math-input.js | 51 ++++++++++++++++++++++++++++++
 1 file changed, 51 insertions(+)

diff --git a/src/components/input/math-input.js b/src/components/input/math-input.js
index f7e503c1..02b4dde6 100644
--- a/src/components/input/math-input.js
+++ b/src/components/input/math-input.js
@@ -15,6 +15,7 @@ const {
  } = require('../common-style');
 const {keypadElementPropType} = require('../prop-types');
 const {brightGreen, gray17} = require('../common-style');
+const Keys = require("../../data/keys");
 
 const i18n = window.i18n || {_: s => s};
 
@@ -697,6 +698,55 @@ class MathInput extends React.Component {
         this._updateCursorHandle(true);
     };
 
+    domKeyToMathQuillKey = (key) => {
+        const keyMap = {
+            "+": Keys.PLUS,
+            "-": Keys.MINUS,
+            "*": Keys.TIMES,
+            "/": Keys.DIVIDE,
+            ".": Keys.DECIMAL,
+            "%": Keys.PERCENT,
+            "=": Keys.EQUAL,
+            ">": Keys.GT,
+            "<": Keys.LT,
+            "^": Keys.EXP
+        };
+
+        // Numbers
+        if (['0', '1', '2', '3', '4', '5', '6', '7', '8', '9'].includes(key)){
+            return `NUM_${key}`;
+        }
+
+        // Movement keys
+        else if (key == "Backspace"){
+            return Keys.BACKSPACE
+        }
+
+        // Operators
+        else if (key in keyMap){
+             return keyMap[key];
+
+        }
+
+        // The key pressed doesn't map to any of the math input operators
+        return null;
+    }
+
+    handleKeyUp = (event) => {
+        const mathQuillKey = this.domKeyToMathQuillKey(event.key);
+
+        if (mathQuillKey){
+            this.mathField.pressKey(mathQuillKey);
+
+            const value = this.mathField.getContent();
+            if (this.props.value !== value) {
+                this.mathField.setContent(this.props.value);
+                this.props.onChange(value, false);
+                this._hideCursorHandle();
+            }
+        }
+    };      
+
     render() {
         const {focused, handle} = this.state;
         const {style} = this.props;
@@ -753,6 +803,7 @@ class MathInput extends React.Component {
                 ref={node => {
                     this.inputRef = node;
                 }}
+                onKeyUp={this.handleKeyUp}
             >
                 {/* NOTE(charlie): This element must be styled with inline
                     styles rather than with Aphrodite classes, as MathQuill