Merge pull request #960 from ProvableHQ/fix/algebra-ui-fixes

[Fix] Fix minor UI issues with scalar multiplication and multiplicative inverse of fields.

Author: iamalwaysuncomfortable

website/src/tabs/algebra/FieldArithmetic.jsx

@@ -47,8 +47,8 @@ export const FieldArithmetic = () => {
     };
 
     const calculateResult = (num1, num2, op) => {
-        if ((op === "inv" && num1 === "") || 
-            (op !== "inv" && (num1 === "" || num2 === ""))) {
+        if (((op === "inv" || op === "mulinv") && num1 === "") ||
+            ((op !== "inv" && op !== "mulinv") && (num1 === "" || num2 === ""))) {
             setResult("");
             return;
         }

website/src/tabs/algebra/GroupArithmetic.jsx

@@ -152,6 +152,12 @@ export const GroupArithmetic = () => {
         calculateResult(groupValueOne, newValue, operation);
     };
 
+    const onScalarRandom = () => {
+        const newValue = generateRandomScalar();
+        setScalarValue(newValue);
+        calculateResult(groupValueOne, groupValueTwo, operation, newValue);
+    }
+
     const layout = { 
         labelCol: { span: 6 }, 
         wrapperCol: { span: 18 },
@@ -264,7 +270,7 @@ export const GroupArithmetic = () => {
                             />
                             <Button 
                                 size="large"
-                                onClick={() => setScalarValue(generateRandomScalar())}
+                                onClick={onScalarRandom}
                                 style={{ width: '110px' }}
                             >
                                 Random