Merge pull request #36 from concord-consortium/187321079-remove-seque…

…nce-max-length

update: Remove sequence length limit and change default delimiter [PT-187321079]

src/components/app.tsx

@@ -13,7 +13,6 @@ import "./app.scss";
 type GenerationMode = "ready" | "playing" | "paused" | "stepping";
 
 const AnyStartingState = "(any)";
-const MaxLengthLimit = 25;
 const AnimationDelay = 1000;
 
 type SequenceGroup = {
@@ -56,7 +55,7 @@ const SequenceOutputHeader = ({ group }: { group: SequenceGroup }) => {
 
 export const App = () => {
   const [lengthLimit, setLengthLimit] = useState<number | undefined>(5);
-  const [delimiter, setDelimiter] = useState("");
+  const [delimiter, setDelimiter] = useState(" ");
   const [startingState, setStartingState] = useState("");
   const [sequenceGroups, setSequenceGroups] = useState<SequenceGroup[]>([]);
   const [selectedNodeId, _setSelectedNodeId] = useState<string>();
@@ -216,7 +215,7 @@ export const App = () => {
 
   const handleChangeLengthLimit = (e: React.ChangeEvent<HTMLInputElement>) => {
     const numberValue = parseInt(e.target.value, 10);
-    setLengthLimit(isNaN(numberValue) ? undefined : Math.min(MaxLengthLimit, numberValue));
+    setLengthLimit(isNaN(numberValue) ? undefined : numberValue);
   };
 
   const handleChangeDelimiter = (e: React.ChangeEvent<HTMLInputElement>) => {
@@ -289,7 +288,6 @@ export const App = () => {
                 value={lengthLimit}
                 onChange={handleChangeLengthLimit}
                 min={1}
-                max={MaxLengthLimit}
                 disabled={disabled}
               />
             </div>