feat(app-rfi): only include space if icon present

Author: scott-williams-az

packages/app-rfi/src/components/controls/RfiRadioGroup.js

@@ -5,7 +5,7 @@ import { Field } from "formik";
 import PropTypes from "prop-types";
 import React from "react";
 
-import {RfiError, RfiLabel, RfiLegend} from "./controls-helpers";
+import { RfiError, RfiLegend } from "./controls-helpers";
 
 const RfiRadioGroup = ({ name, id, options, label, onBlur }) => {
   return (

packages/app-rfi/src/components/controls/controls-helpers.js

@@ -17,37 +17,41 @@ const GaEventPropTypes = {
 
 const RfiLabel = ({ label, name, id, requiredIcon }) => (
   <label htmlFor={id || name}>
-    {requiredIcon ? (
-      <span title="Required">
-        <i className="fas fa-circle uds-field-required" aria-hidden="true" />
-      </span>
-    ) : null}
-    &nbsp;
+    {requiredIcon && (
+      <>
+        <span title="Required">
+          <i className="fas fa-circle uds-field-required" aria-hidden="true" />
+        </span>
+        &nbsp;
+      </>
+    )}
     {label}
   </label>
 );
 
 const RfiLegend = ({ label, requiredIcon }) => (
   <legend className="fw-bold">
-    {requiredIcon ? (
-      <span title="Required">
-        <i className="fas fa-circle uds-field-required" aria-hidden="true" />
-      </span>
-    ) : null}
-    &nbsp;
+    {requiredIcon && (
+      <>
+        <span title="Required">
+          <i className="fas fa-circle uds-field-required" aria-hidden="true" />
+        </span>
+        &nbsp;
+      </>
+    )}
     {label}
   </legend>
 );
 
 const RfiError = ({ isError, metaError }) => (
   <div role="alert">
-    {isError ? (
+    {isError && (
       <small className="form-text invalid-feedback">
         <i className="fas fa-exclamation-triangle" aria-hidden="true" />
         &nbsp;
         {metaError}
       </small>
-    ) : null}
+    )}
   </div>
 );
 
@@ -67,6 +71,16 @@ RfiLabel.propTypes = {
   requiredIcon: PropTypes.bool,
 };
 
+RfiLegend.defaultProps = {
+  label: undefined,
+  requiredIcon: undefined,
+};
+
+RfiLegend.propTypes = {
+  label: PropTypes.string.isRequired,
+  requiredIcon: PropTypes.bool,
+};
+
 RfiError.defaultProps = {
   isError: undefined,
   metaError: undefined,