@bigbinary/neeto-form-frontend 4.2.4 → 4.2.6

package/dist/BuildForm.js

@@ -42,7 +42,6 @@ import { isEditorEmpty } from '@bigbinary/neeto-editor/utils';
 import * as yup from 'yup';
 import Radio from '@bigbinary/neetoui/formik/Radio';
 import Label from '@bigbinary/neetoui/Label';
-import HelpPopover from '@bigbinary/neeto-molecules/HelpPopover';
 import CreatableSelect from 'react-select/creatable';
 import _toArray from '@babel/runtime/helpers/toArray';
 import Close from '@bigbinary/neeto-icons/Close';
@@ -54,6 +53,7 @@ import Checkbox$1 from '@bigbinary/neetoui/formik/Checkbox';
 import Checkbox from '@bigbinary/neetoui/Checkbox';
 import OptionFields from '@bigbinary/neeto-molecules/OptionFields';
 import Slider from '@bigbinary/neetoui/Slider';
+import HelpPopover from '@bigbinary/neeto-molecules/HelpPopover';
 import '@bigbinary/neeto-commons-frontend/constants';
 import 'axios';
 import 'dompurify';
@@ -793,7 +793,7 @@ var Block = function Block(_ref) {
     children = _ref.children,
     dataTestid = _ref.dataTestid;
   return /*#__PURE__*/jsxs("div", {
-    className: "mt-4 space-y-4",
+    className: "mt-4 space-y-3",
     "data-testid": dataTestid,
     children: [/*#__PURE__*/jsx("div", {
       className: "flex items-center justify-between",
@@ -1022,7 +1022,8 @@ var Form$1 = function Form(_ref) {
   var shouldShowHideSwitch = isFieldCodeEnabled && !usesCustomSubmissionComponent && !includes(kind, NON_HIDEABLE_FIELDS);
   var canRestrictResponseVisibilityToHost = restrictResponseVisibilityToHost && !includes(kind, NON_HIDEABLE_FIELDS);
   var canMarkResponsesAsReadOnly = markResponsesAsReadOnly && !includes(kind, NON_READ_ONLY_FIELDS);
-  var isSettingsBlockVisible = !isRequired && (!isImmutable || shouldShowHideSwitch);
+  var isBasicSettingsVisible = !isRequired && (!isImmutable || shouldShowHideSwitch);
+  var isSettingsBlockVisible = isBasicSettingsVisible || canMarkResponsesAsReadOnly;
   var handleKindChange = function handleKindChange(option) {
     var data = buildQuestionData(option);
     updateFormState(data);
@@ -1087,29 +1088,31 @@ var Form$1 = function Form(_ref) {
         }), isSettingsBlockVisible && /*#__PURE__*/jsxs(Block, {
           dataTestid: "settings-card",
           title: t("neetoForm.common.settings"),
-          children: [!isImmutable && /*#__PURE__*/jsx(RequiredSwitch, {}), shouldShowHideSwitch && /*#__PURE__*/jsx(Switch, {
-            label: t("neetoForm.common.hideQuestion"),
-            labelProps: hideSwitchLabelProps(hideQuestionHelpDocUrl),
-            name: "isHidden"
-          }), canRestrictResponseVisibilityToHost && /*#__PURE__*/jsx(Switch, {
-            label: t("neetoForm.common.responseVisibleOnlyToHost"),
-            name: "isResponseVisibleOnlyToHost",
-            labelProps: responseVisibleOnlyToHostSwitchLabelProps(responseVisibleOnlyToHostHelpDocUrl)
+          children: [isBasicSettingsVisible && /*#__PURE__*/jsxs(Fragment, {
+            children: [!isImmutable && /*#__PURE__*/jsx(RequiredSwitch, {}), shouldShowHideSwitch && /*#__PURE__*/jsx(Switch, {
+              label: t("neetoForm.common.hideQuestion"),
+              name: "isHidden",
+              labelProps: hideSwitchLabelProps(hideQuestionHelpDocUrl)
+            }), canRestrictResponseVisibilityToHost && /*#__PURE__*/jsx(Switch, {
+              label: t("neetoForm.common.responseVisibleOnlyToHost"),
+              name: "isResponseVisibleOnlyToHost",
+              labelProps: responseVisibleOnlyToHostSwitchLabelProps(responseVisibleOnlyToHostHelpDocUrl)
+            })]
+          }), canMarkResponsesAsReadOnly && /*#__PURE__*/jsx(Switch, {
+            label: t("neetoForm.common.readOnly"),
+            labelProps: readOnlySwitchLabelProps(readOnlyHelpDocUrl),
+            name: "isReadOnly"
           })]
-        }), (shouldShowFieldCode || canMarkResponsesAsReadOnly) && /*#__PURE__*/jsx(Accordion, {
+        }), shouldShowFieldCode && /*#__PURE__*/jsx(Accordion, {
           className: "neeto-form-nano-advanced-properties-accordion",
           "data-testid": "advanced-properties-card",
           defaultActiveKey: isFieldCodeInvalid ? 0 : null // This is to open the accordion when field code is invalid
           ,
           children: /*#__PURE__*/jsx(Accordion.Item, {
             title: t("neetoForm.common.advancedProperties"),
-            children: /*#__PURE__*/jsxs("div", {
+            children: /*#__PURE__*/jsx("div", {
               className: "flex flex-col space-y-4",
-              children: [shouldShowFieldCode && /*#__PURE__*/jsx(FieldCode, {}), canMarkResponsesAsReadOnly && /*#__PURE__*/jsx(Switch, {
-                label: t("neetoForm.common.readOnly"),
-                name: "isReadOnly",
-                labelProps: readOnlySwitchLabelProps(readOnlyHelpDocUrl)
-              })]
+              children: shouldShowFieldCode && /*#__PURE__*/jsx(FieldCode, {})
             })
           })
         })]
@@ -1382,27 +1385,6 @@ var SwitchBlock = function SwitchBlock(_ref) {
   });
 };
 
-function ownKeys$5(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbols) { var o = Object.getOwnPropertySymbols(e); r && (o = o.filter(function (r) { return Object.getOwnPropertyDescriptor(e, r).enumerable; })), t.push.apply(t, o); } return t; }
-function _objectSpread$5(e) { for (var r = 1; r < arguments.length; r++) { var t = null != arguments[r] ? arguments[r] : {}; r % 2 ? ownKeys$5(Object(t), !0).forEach(function (r) { _defineProperty(e, r, t[r]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(e, Object.getOwnPropertyDescriptors(t)) : ownKeys$5(Object(t)).forEach(function (r) { Object.defineProperty(e, r, Object.getOwnPropertyDescriptor(t, r)); }); } return e; }
-var TitleWithHelpPopover = function TitleWithHelpPopover(_ref) {
-  var title = _ref.title,
-    description = _ref.description,
-    helpLink = _ref.helpLink;
-  return /*#__PURE__*/jsxs("div", {
-    className: "flex gap-2",
-    children: [title, /*#__PURE__*/jsx(HelpPopover, _objectSpread$5({
-      description: description,
-      popoverProps: {
-        position: "top"
-      }
-    }, helpLink && {
-      helpLinkProps: {
-        href: helpLink
-      }
-    }))]
-  });
-};
-
 var DOMAIN_RESTRICTION_TRANSLATIONS = {
   whitelistedDomains: {
     title: t("neetoForm.domainRestriction.whitelistedDomains.label"),
@@ -1421,8 +1403,8 @@ var EMAIL_VERIFICATION_TRANSLATIONS = {
 };
 
 var _excluded$2 = ["children"];
-function ownKeys$4(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbols) { var o = Object.getOwnPropertySymbols(e); r && (o = o.filter(function (r) { return Object.getOwnPropertyDescriptor(e, r).enumerable; })), t.push.apply(t, o); } return t; }
-function _objectSpread$4(e) { for (var r = 1; r < arguments.length; r++) { var t = null != arguments[r] ? arguments[r] : {}; r % 2 ? ownKeys$4(Object(t), !0).forEach(function (r) { _defineProperty(e, r, t[r]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(e, Object.getOwnPropertyDescriptors(t)) : ownKeys$4(Object(t)).forEach(function (r) { Object.defineProperty(e, r, Object.getOwnPropertyDescriptor(t, r)); }); } return e; }
+function ownKeys$5(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbols) { var o = Object.getOwnPropertySymbols(e); r && (o = o.filter(function (r) { return Object.getOwnPropertyDescriptor(e, r).enumerable; })), t.push.apply(t, o); } return t; }
+function _objectSpread$5(e) { for (var r = 1; r < arguments.length; r++) { var t = null != arguments[r] ? arguments[r] : {}; r % 2 ? ownKeys$5(Object(t), !0).forEach(function (r) { _defineProperty(e, r, t[r]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(e, Object.getOwnPropertyDescriptors(t)) : ownKeys$5(Object(t)).forEach(function (r) { Object.defineProperty(e, r, Object.getOwnPropertyDescriptor(t, r)); }); } return e; }
 var STYLES = {
   border: {
     "default": "1px solid rgb(var(--neeto-ui-gray-400))",
@@ -1434,8 +1416,8 @@ var STYLES = {
   }
 };
 var MultiValueRemove = function MultiValueRemove(props) {
-  return /*#__PURE__*/jsx(components.MultiValueRemove, _objectSpread$4(_objectSpread$4({}, props), {}, {
-    innerProps: _objectSpread$4(_objectSpread$4({}, props.innerProps), {}, _defineProperty({}, "data-testid", "".concat(hyphenize(props.data.label), "-remove-icon"))),
+  return /*#__PURE__*/jsx(components.MultiValueRemove, _objectSpread$5(_objectSpread$5({}, props), {}, {
+    innerProps: _objectSpread$5(_objectSpread$5({}, props.innerProps), {}, _defineProperty({}, "data-testid", "".concat(hyphenize(props.data.label), "-remove-icon"))),
     children: /*#__PURE__*/jsx(Close, {
       size: 16
     })
@@ -1455,8 +1437,8 @@ var ValueContainer = function ValueContainer(_ref) {
     firstChild = _children[0],
     rest = _children.slice(1);
   var shouldCollapse = !isFocused && value.length > visibleDomainsCount;
-  return /*#__PURE__*/jsxs(components.ValueContainer, _objectSpread$4(_objectSpread$4({}, props), {}, {
-    innerProps: _objectSpread$4(_objectSpread$4({}, props.innerProps), {}, _defineProperty({}, "data-testid", "multi-domain-input-container")),
+  return /*#__PURE__*/jsxs(components.ValueContainer, _objectSpread$5(_objectSpread$5({}, props), {}, {
+    innerProps: _objectSpread$5(_objectSpread$5({}, props.innerProps), {}, _defineProperty({}, "data-testid", "multi-domain-input-container")),
     children: [shouldCollapse ? firstChild.slice(0, visibleDomainsCount) : firstChild, shouldCollapse && /*#__PURE__*/jsx(Tag, {
       style: "secondary",
       label: t("neetoForm.common.remainingItemsCount", {
@@ -1466,8 +1448,8 @@ var ValueContainer = function ValueContainer(_ref) {
   }));
 };
 var ClearIndicator = function ClearIndicator(props) {
-  return /*#__PURE__*/jsx(components.ClearIndicator, _objectSpread$4(_objectSpread$4({}, props), {}, {
-    innerProps: _objectSpread$4(_objectSpread$4({}, props.innerProps), {}, _defineProperty({}, "data-testid", "clear-all-button")),
+  return /*#__PURE__*/jsx(components.ClearIndicator, _objectSpread$5(_objectSpread$5({}, props), {}, {
+    innerProps: _objectSpread$5(_objectSpread$5({}, props.innerProps), {}, _defineProperty({}, "data-testid", "clear-all-button")),
     children: /*#__PURE__*/jsx(Close, {
       className: "cursor-pointer",
       size: 16
@@ -1488,7 +1470,7 @@ var CUSTOM_STYLES = {
   input: assoc("overflow", "hidden"),
   multiValue: function multiValue(styles, _ref2) {
     var valid = _ref2.data.valid;
-    return _objectSpread$4(_objectSpread$4({}, styles), {}, {
+    return _objectSpread$5(_objectSpread$5({}, styles), {}, {
       border: valid ? STYLES.border["default"] : STYLES.border.error,
       color: valid ? STYLES.color["default"] : STYLES.color.error
     });
@@ -1531,8 +1513,8 @@ var formatErrorMessage = function formatErrorMessage(error) {
 };
 
 var _excluded$1 = ["label", "value", "onChange", "error", "onBlur"];
-function ownKeys$3(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbols) { var o = Object.getOwnPropertySymbols(e); r && (o = o.filter(function (r) { return Object.getOwnPropertyDescriptor(e, r).enumerable; })), t.push.apply(t, o); } return t; }
-function _objectSpread$3(e) { for (var r = 1; r < arguments.length; r++) { var t = null != arguments[r] ? arguments[r] : {}; r % 2 ? ownKeys$3(Object(t), !0).forEach(function (r) { _defineProperty(e, r, t[r]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(e, Object.getOwnPropertyDescriptors(t)) : ownKeys$3(Object(t)).forEach(function (r) { Object.defineProperty(e, r, Object.getOwnPropertyDescriptor(t, r)); }); } return e; }
+function ownKeys$4(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbols) { var o = Object.getOwnPropertySymbols(e); r && (o = o.filter(function (r) { return Object.getOwnPropertyDescriptor(e, r).enumerable; })), t.push.apply(t, o); } return t; }
+function _objectSpread$4(e) { for (var r = 1; r < arguments.length; r++) { var t = null != arguments[r] ? arguments[r] : {}; r % 2 ? ownKeys$4(Object(t), !0).forEach(function (r) { _defineProperty(e, r, t[r]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(e, Object.getOwnPropertyDescriptors(t)) : ownKeys$4(Object(t)).forEach(function (r) { Object.defineProperty(e, r, Object.getOwnPropertyDescriptor(t, r)); }); } return e; }
 var MultiDomainInput = function MultiDomainInput(_ref) {
   var _ref$label = _ref.label,
     label = _ref$label === void 0 ? "" : _ref$label,
@@ -1591,7 +1573,7 @@ var MultiDomainInput = function MultiDomainInput(_ref) {
         "data-testid": "".concat(hyphenize(label), "-input-label"),
         children: label
       })
-    }), /*#__PURE__*/jsx(CreatableSelect, _objectSpread$3({
+    }), /*#__PURE__*/jsx(CreatableSelect, _objectSpread$4({
       isMulti: true,
       required: true,
       classNamePrefix: "neeto-ui-react-select",
@@ -1601,7 +1583,7 @@ var MultiDomainInput = function MultiDomainInput(_ref) {
       className: classnames("neeto-ui-react-select__container neeto-ui-react-select__container--medium neeto-ui-email-input__select", {
         "neeto-ui-react-select__container--error": !!error
       }),
-      styles: _objectSpread$3(_objectSpread$3({}, CUSTOM_STYLES), {}, {
+      styles: _objectSpread$4(_objectSpread$4({}, CUSTOM_STYLES), {}, {
         control: mergeLeft({
           maxHeight: "200px",
           overflowY: "auto"
@@ -1613,7 +1595,7 @@ var MultiDomainInput = function MultiDomainInput(_ref) {
       },
       onInputChange: setInputValue,
       onKeyDown: handleKeyDown
-    }, _objectSpread$3({
+    }, _objectSpread$4({
       inputValue: inputValue,
       isFocused: isFocused,
       onChange: onChange,
@@ -1635,6 +1617,8 @@ var MultiDomainInput = function MultiDomainInput(_ref) {
   });
 };
 
+function ownKeys$3(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbols) { var o = Object.getOwnPropertySymbols(e); r && (o = o.filter(function (r) { return Object.getOwnPropertyDescriptor(e, r).enumerable; })), t.push.apply(t, o); } return t; }
+function _objectSpread$3(e) { for (var r = 1; r < arguments.length; r++) { var t = null != arguments[r] ? arguments[r] : {}; r % 2 ? ownKeys$3(Object(t), !0).forEach(function (r) { _defineProperty(e, r, t[r]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(e, Object.getOwnPropertyDescriptors(t)) : ownKeys$3(Object(t)).forEach(function (r) { Object.defineProperty(e, r, Object.getOwnPropertyDescriptor(t, r)); }); } return e; }
 var validateDomains = function validateDomains(value) {
   var schema = yup.array().of(yup.string().test("verify domains", t("neetoForm.error.invalidDomain"), function (domain) {
     return VALID_DOMAIN_REGEX.test(domain);
@@ -1670,6 +1654,20 @@ var validateDomains = function validateDomains(value) {
     };
   }());
 };
+var buildHelpPopoverLabelProps = function buildHelpPopoverLabelProps(popoverDescription, helpLink) {
+  return {
+    helpIconProps: {
+      popoverProps: _objectSpread$3({
+        description: popoverDescription
+      }, helpLink && {
+        helpLinkProps: {
+          href: helpLink,
+          label: t("neetoForm.common.viewHelpArticle")
+        }
+      })
+    }
+  };
+};
 
 var EmailDomains = function EmailDomains(_ref) {
   var name = _ref.name,
@@ -1722,12 +1720,9 @@ var RestrictDomainsBlock = function RestrictDomainsBlock(_ref) {
     className: "flex flex-col space-y-4",
     "data-testid": "".concat(hyphenate(switchName), "-container"),
     children: [/*#__PURE__*/jsx(SwitchBlock, {
-      name: switchName,
-      title: /*#__PURE__*/jsx(TitleWithHelpPopover, {
-        helpLink: helpLink,
-        title: title,
-        description: popoverDescription
-      })
+      label: title,
+      labelProps: buildHelpPopoverLabelProps(popoverDescription, helpLink),
+      name: switchName
     }), isDomainRestrictionEnabled && /*#__PURE__*/jsx(EmailDomains, {
       label: description,
       name: domainName
@@ -1746,16 +1741,9 @@ var VerifyEmails = function VerifyEmails() {
   return /*#__PURE__*/jsx("div", {
     "data-testid": "email-verification-switch-container",
     children: /*#__PURE__*/jsx(Switch, {
-      name: "isVerificationEnabled",
-      label: /*#__PURE__*/jsx(Typography, {
-        component: "span",
-        style: "body1",
-        children: /*#__PURE__*/jsx(TitleWithHelpPopover, {
-          helpLink: helpLink,
-          title: title,
-          description: popoverDescription
-        })
-      })
+      label: title,
+      labelProps: buildHelpPopoverLabelProps(popoverDescription, helpLink),
+      name: "isVerificationEnabled"
     })
   });
 };