fix: add error prop to FormGroup (#4437)

Semantic-Org · Sep 4, 2023 · ba4b720 · ba4b720
1 parent 79ffb36
commit ba4b720
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 1 deletion.
diff --git a/src/collections/Form/FormGroup.d.ts b/src/collections/Form/FormGroup.d.ts
@@ -18,6 +18,9 @@ export interface StrictFormGroupProps {
   /** A Form Group can be disabled. */
   disabled?: boolean
 
+  /** A Form Group can have error. */
+  error?: boolean
+
   /** Fields can show related choices. */
   grouped?: boolean
 

diff --git a/src/collections/Form/FormGroup.js b/src/collections/Form/FormGroup.js
@@ -16,9 +16,10 @@ import {
  * @see Form
  */
 const FormGroup = React.forwardRef((props, ref) => {
-  const { children, className, disabled, grouped, inline, unstackable, widths } = props
+  const { children, className, disabled, error, grouped, inline, unstackable, widths } = props
 
   const classes = cx(
+    useKeyOnly(error, 'error'),
     useKeyOnly(disabled, 'disabled'),
     useKeyOnly(grouped, 'grouped'),
     useKeyOnly(inline, 'inline'),
@@ -52,6 +53,9 @@ FormGroup.propTypes = {
   /** A Form Group can be disabled. */
   disabled: PropTypes.bool,
 
+  /** A Form Group can have error. */
+  error: PropTypes.bool,
+
   /** Fields can show related choices. */
   grouped: customPropTypes.every([customPropTypes.disallow(['inline']), PropTypes.bool]),
 

diff --git a/test/specs/collections/Form/FormGroup-test.js b/test/specs/collections/Form/FormGroup-test.js
@@ -16,6 +16,7 @@ describe('FormGroup', () => {
   common.propKeyOnlyToClassName(FormGroup, 'grouped')
   common.propKeyOnlyToClassName(FormGroup, 'disabled')
   common.propKeyOnlyToClassName(FormGroup, 'inline')
+  common.propKeyOnlyToClassName(FormGroup, 'error')
   common.propKeyOnlyToClassName(FormGroup, 'unstackable')
 
   common.forwardsRef(FormGroup)