Merge pull request #1250 from complexdatacollective/fix/max-nodes

jthrilly · web-flow · commit 717c15d710a9 · 2023-10-04T15:32:42.000+02:00
fix/ action button still clickable after max reached
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -55,7 +55,7 @@
     "@babel/preset-react": "7.10.1",
     "@codaco/eslint-plugin-spellcheck": "0.0.14",
     "@codaco/shared-consts": "~0.0.1-alpha.3",
-    "@codaco/ui": "~5.8.3",
+    "@codaco/ui": "~5.8.5",
     "@electron/notarize": "~1.2.3",
     "@faker-js/faker": "~6.0.0-alpha.5",
     "@zippytech/sorty": "^2.0.0",
@@ -315,4 +315,4 @@
       "cordova-plugin-ionic-keyboard": {}
     }
   }
-}
+}
diff --git a/src/containers/QuickNodeForm.js b/src/containers/QuickNodeForm.js
@@ -185,7 +185,7 @@ const QuickAddForm = ({
           >
             <ActionButton
               disabled={disabled}
-              onClick={() => !disabled && setShowForm(true)}
+              onClick={() => setShowForm(true)}
               icon={icon}
               title={`Add ${nodeType}...`}
             />

Original file line number	Diff line number	Diff line change
`@@ -55,7 +55,7 @@`
`55`	`55`	`"@babel/preset-react": "7.10.1",`
`56`	`56`	`"@codaco/eslint-plugin-spellcheck": "0.0.14",`
`57`	`57`	`"@codaco/shared-consts": "~0.0.1-alpha.3",`
`58`		`- "@codaco/ui": "~5.8.3",`
	`58`	`+ "@codaco/ui": "~5.8.5",`
`59`	`59`	`"@electron/notarize": "~1.2.3",`
`60`	`60`	`"@faker-js/faker": "~6.0.0-alpha.5",`
`61`	`61`	`"@zippytech/sorty": "^2.0.0",`
`@@ -315,4 +315,4 @@`
`315`	`315`	`"cordova-plugin-ionic-keyboard": {}`
`316`	`316`	`}`
`317`	`317`	`}`
`318`		`-}`
	`318`	`+}`
Original file line number	Diff line number	Diff line change
`@@ -185,7 +185,7 @@ const QuickAddForm = ({`
`185`	`185`	`>`
`186`	`186`	`<ActionButton`
`187`	`187`	`disabled={disabled}`
`188`		`- onClick={() => !disabled && setShowForm(true)}`
	`188`	`+ onClick={() => setShowForm(true)}`
`189`	`189`	`icon={icon}`
`190`	`190`	title={`Add ${nodeType}...`}
`191`	`191`	`/>`