@bpmn-io/form-js-viewer 1.7.0 → 1.7.1

package/dist/index.es.js

@@ -1569,7 +1569,8 @@ function useCleanupMultiSelectValue(props) {
 
 function Description(props) {
   const {
-    description
+    description,
+    id
   } = props;
   const evaluatedDescription = useSingleLineTemplateEvaluation(description || '', {
     debug: true
@@ -1578,6 +1579,7 @@ function Description(props) {
     return null;
   }
   return jsx("div", {
+    id: id,
     class: "fjs-form-field-description",
     children: evaluatedDescription
   });
@@ -1608,6 +1610,7 @@ function Errors(props) {
 function Label(props) {
   const {
     id,
+    htmlFor,
     label,
     collapseOnEmpty = true,
     required = false
@@ -1616,12 +1619,14 @@ function Label(props) {
     debug: true
   });
   return jsxs("label", {
-    for: id,
+    id: id,
+    for: htmlFor,
     class: classNames('fjs-form-field-label', {
       'fjs-incollapsible-label': !collapseOnEmpty
     }, props['class']),
     children: [props.children, evaluatedLabel, required && jsx("span", {
       class: "fjs-asterix",
+      "aria-hidden": true,
       children: "*"
     })]
   });
@@ -1632,7 +1637,6 @@ function Checkbox(props) {
   const {
     disabled,
     errors = [],
-    errorMessageId,
     domId,
     onBlur,
     onFocus,
@@ -1656,6 +1660,8 @@ function Checkbox(props) {
       value: target.checked
     });
   };
+  const descriptionId = `${domId}-description`;
+  const errorMessageId = `${domId}-error-message`;
   return jsxs("div", {
     class: classNames(formFieldClasses(type$f, {
       errors,
@@ -1665,7 +1671,7 @@ function Checkbox(props) {
       'fjs-checked': value
     }),
     children: [jsx(Label, {
-      id: domId,
+      htmlFor: domId,
       label: label,
       required: required,
       children: jsx("input", {
@@ -1678,13 +1684,16 @@ function Checkbox(props) {
         onChange: onChange,
         onBlur: () => onBlur && onBlur(),
         onFocus: () => onFocus && onFocus(),
-        "aria-describedby": errorMessageId
+        required: required,
+        "aria-invalid": errors.length > 0,
+        "aria-describedby": [descriptionId, errorMessageId].join(' ')
       })
     }), jsx(Description, {
+      id: descriptionId,
       description: description
     }), jsx(Errors, {
-      errors: errors,
-      id: errorMessageId
+      id: errorMessageId,
+      errors: errors
     })]
   });
 }
@@ -1707,7 +1716,6 @@ function Checklist(props) {
   const {
     disabled,
     errors = [],
-    errorMessageId,
     domId,
     onBlur,
     onFocus,
@@ -1754,6 +1762,8 @@ function Checklist(props) {
     values,
     onChange: props.onChange
   });
+  const descriptionId = `${domId}-description`;
+  const errorMessageId = `${domId}-error-message`;
   return jsxs("div", {
     class: classNames(formFieldClasses(type$e, {
       errors,
@@ -1768,7 +1778,7 @@ function Checklist(props) {
       const itemDomId = `${domId}-${index}`;
       const isChecked = hasEqualValue(o.value, values);
       return jsx(Label, {
-        id: itemDomId,
+        htmlFor: itemDomId,
         label: o.label,
         class: classNames({
           'fjs-checked': isChecked
@@ -1784,14 +1794,17 @@ function Checklist(props) {
           onClick: () => toggleCheckbox(o.value),
           onBlur: onCheckboxBlur,
           onFocus: onCheckboxFocus,
-          "aria-describedby": errorMessageId
+          required: required,
+          "aria-invalid": errors.length > 0,
+          "aria-describedby": [descriptionId, errorMessageId].join(' ')
         })
       });
     }), jsx(Description, {
+      id: descriptionId,
       description: description
     }), jsx(Errors, {
-      errors: errors,
-      id: errorMessageId
+      id: errorMessageId,
+      errors: errors
     })]
   });
 }
@@ -1871,6 +1884,7 @@ function FormField(props) {
     }
   }, [viewerCommands, field, initialValue, initialValidationTrigger, indexes]);
   const onBlur = useCallback(() => {
+    const value = get(data, valuePath);
     if (initialValidationTrigger) {
       setInitialValidationTrigger(false);
       viewerCommands.updateFieldValidation(field, value, indexes);
@@ -1878,7 +1892,7 @@ function FormField(props) {
     eventBus.fire('formField.blur', {
       formField: field
     });
-  }, [eventBus, field, indexes, value, viewerCommands, initialValidationTrigger]);
+  }, [eventBus, field, indexes, viewerCommands, initialValidationTrigger, data, valuePath]);
   const onFocus = useCallback(() => {
     eventBus.fire('formField.focus', {
       formField: field
@@ -1902,7 +1916,6 @@ function FormField(props) {
   }
   const domId = `${prefixId(field.id, formId, indexes)}`;
   const fieldErrors = get(errors, [field.id, ...Object.values(indexes || {})]) || [];
-  const errorMessageId = errors.length === 0 ? undefined : `${domId}-error-message`;
   return jsx(Column, {
     field: field,
     class: gridColumnClasses(field),
@@ -1913,7 +1926,6 @@ function FormField(props) {
         ...props,
         disabled: disabled,
         errors: fieldErrors,
-        errorMessageId: errorMessageId,
         domId: domId,
         onChange: disabled || readonly ? noop$1 : onChangeIndexed,
         onBlur: disabled || readonly ? noop$1 : onBlur,
@@ -2316,7 +2328,7 @@ function Datepicker(props) {
   return jsxs("div", {
     class: "fjs-datetime-subsection",
     children: [jsx(Label, {
-      id: domId,
+      htmlFor: domId,
       label: label,
       collapseOnEmpty: collapseLabelOnEmpty,
       required: required
@@ -2588,7 +2600,7 @@ function Timepicker(props) {
   return jsxs("div", {
     class: "fjs-datetime-subsection",
     children: [jsx(Label, {
-      id: domId,
+      htmlFor: domId,
       label: label,
       collapseOnEmpty: collapseLabelOnEmpty,
       required: required
@@ -2775,6 +2787,7 @@ function Datetime(props) {
     });
   }, []);
   const errorMessageId = allErrors.length === 0 ? undefined : `${prefixId(id, formId)}-error-message`;
+  const descriptionId = `${prefixId(id, formId)}-description`;
   const datePickerProps = {
     label: dateLabel,
     collapseLabelOnEmpty: !timeLabel,
@@ -2787,7 +2800,7 @@ function Datetime(props) {
     date: dateTime.date,
     readonly,
     setDate,
-    'aria-describedby': errorMessageId
+    'aria-describedby': [descriptionId, errorMessageId].join(' ')
   };
   const timePickerProps = {
     label: timeLabel,
@@ -2802,7 +2815,7 @@ function Datetime(props) {
     timeInterval,
     time: dateTime.time,
     setTime,
-    'aria-describedby': errorMessageId
+    'aria-describedby': [descriptionId, errorMessageId].join(' ')
   };
   return jsxs("div", {
     class: formFieldClasses(type$d, {
@@ -2821,6 +2834,7 @@ function Datetime(props) {
         ...timePickerProps
       })]
     }), jsx(Description, {
+      id: descriptionId,
       description: description
     }), jsx(Errors, {
       errors: allErrors,
@@ -2924,7 +2938,7 @@ function IFrame(props) {
       readonly
     }),
     children: [jsx(Label, {
-      id: domId,
+      htmlFor: domId,
       label: evaluatedLabel
     }), !evaluatedUrl && jsx(IFramePlaceholder, {
       text: "No content to show."
@@ -3470,7 +3484,6 @@ function Numberfield(props) {
   const {
     disabled,
     errors = [],
-    errorMessageId,
     domId,
     onBlur,
     onFocus,
@@ -3608,6 +3621,8 @@ function Numberfield(props) {
       e.preventDefault();
     }
   };
+  const descriptionId = `${domId}-description`;
+  const errorMessageId = `${domId}-error-message`;
   return jsxs("div", {
     class: formFieldClasses(type$a, {
       errors,
@@ -3615,7 +3630,7 @@ function Numberfield(props) {
       readonly
     }),
     children: [jsx(Label, {
-      id: domId,
+      htmlFor: domId,
       label: label,
       required: required
     }), jsx(TemplatedInputAdorner, {
@@ -3649,7 +3664,9 @@ function Numberfield(props) {
           autoComplete: "off",
           step: incrementAmount,
           value: displayValue,
-          "aria-describedby": errorMessageId
+          "aria-describedby": [descriptionId, errorMessageId].join(' '),
+          required: required,
+          "aria-invalid": errors.length > 0
         }), jsxs("div", {
           class: classNames('fjs-number-arrow-container', {
             'fjs-disabled': disabled,
@@ -3675,10 +3692,11 @@ function Numberfield(props) {
         })]
       })
     }), jsx(Description, {
+      id: descriptionId,
       description: description
     }), jsx(Errors, {
-      errors: errors,
-      id: errorMessageId
+      id: errorMessageId,
+      errors: errors
     })]
   });
 }
@@ -3708,7 +3726,6 @@ function Radio(props) {
   const {
     disabled,
     errors = [],
-    errorMessageId,
     domId,
     onBlur,
     onFocus,
@@ -3754,6 +3771,8 @@ function Radio(props) {
     value,
     onChange: props.onChange
   });
+  const descriptionId = `${domId}-description`;
+  const errorMessageId = `${domId}-error-message`;
   return jsxs("div", {
     class: formFieldClasses(type$9, {
       errors,
@@ -3768,7 +3787,7 @@ function Radio(props) {
       const itemDomId = `${domId}-${index}`;
       const isChecked = isEqual(option.value, value);
       return jsx(Label, {
-        id: itemDomId,
+        htmlFor: itemDomId,
         label: option.label,
         class: classNames({
           'fjs-checked': isChecked
@@ -3784,14 +3803,17 @@ function Radio(props) {
           onClick: () => onChange(option.value),
           onBlur: onRadioBlur,
           onFocus: onRadioFocus,
-          "aria-describedby": errorMessageId
+          "aria-describedby": [descriptionId, errorMessageId].join(' '),
+          required: required,
+          "aria-invalid": errors.length > 0
         })
       }, index);
     }), jsx(Description, {
+      id: descriptionId,
       description: description
     }), jsx(Errors, {
-      errors: errors,
-      id: errorMessageId
+      id: errorMessageId,
+      errors: errors
     })]
   });
 }
@@ -3856,7 +3878,7 @@ function SearchableSelect(props) {
 
   // whenever we change the underlying value, set the label to it
   useEffect(() => {
-    setFilter(label);
+    setFilter(label || '');
   }, [label]);
   const filteredOptions = useMemo(() => {
     if (loadState !== LOAD_STATES.LOADED) {
@@ -3936,7 +3958,7 @@ function SearchableSelect(props) {
   }, [onFocus]);
   const onInputBlur = useCallback(() => {
     setIsDropdownExpanded(false);
-    setFilter(label);
+    setFilter(label || '');
     onBlur && onBlur();
   }, [onBlur, label]);
   return jsxs(Fragment, {
@@ -4032,6 +4054,9 @@ function SimpleSelect(props) {
   }, [disabled, isDropdownExpanded, loadState, readonly, value]);
   const onMouseDown = useCallback(e => {
     const input = inputRef.current;
+    if (disabled || !input) {
+      return;
+    }
     setIsDropdownExpanded(!isDropdownExpanded);
     if (isDropdownExpanded) {
       input.blur();
@@ -4039,7 +4064,7 @@ function SimpleSelect(props) {
       input.focus();
     }
     e.preventDefault();
-  }, [isDropdownExpanded]);
+  }, [disabled, isDropdownExpanded]);
   const initialFocusIndex = useMemo(() => value && findIndex(options, o => o.value === value) || 0, [options, value]);
   const onInputFocus = useCallback(() => {
     if (!readonly) {
@@ -4111,7 +4136,6 @@ function Select(props) {
   const {
     disabled,
     errors = [],
-    errorMessageId,
     domId,
     onBlur,
     onFocus,
@@ -4129,6 +4153,8 @@ function Select(props) {
   const {
     required
   } = validate;
+  const descriptionId = `${domId}-description`;
+  const errorMessageId = `${domId}-error-message`;
   const selectProps = {
     domId,
     disabled,
@@ -4139,7 +4165,9 @@ function Select(props) {
     value,
     onChange,
     readonly,
-    'aria-describedby': errorMessageId
+    required,
+    'aria-invalid': errors.length > 0,
+    'aria-describedby': [descriptionId, errorMessageId].join(' ')
   };
   return jsxs("div", {
     class: formFieldClasses(type$8, {
@@ -4154,7 +4182,7 @@ function Select(props) {
       }
     },
     children: [jsx(Label, {
-      id: domId,
+      htmlFor: domId,
       label: label,
       required: required
     }), searchable ? jsx(SearchableSelect, {
@@ -4162,10 +4190,11 @@ function Select(props) {
     }) : jsx(SimpleSelect, {
       ...selectProps
     }), jsx(Description, {
+      id: descriptionId,
       description: description
     }), jsx(Errors, {
-      errors: errors,
-      id: errorMessageId
+      id: errorMessageId,
+      errors: errors
     })]
   });
 }
@@ -4298,7 +4327,6 @@ function Taglist(props) {
   const {
     disabled,
     errors = [],
-    errorMessageId,
     onFocus,
     domId,
     onBlur,
@@ -4434,6 +4462,8 @@ function Taglist(props) {
     inputRef.current.focus();
   };
   const shouldDisplayDropdown = useMemo(() => !disabled && loadState === LOAD_STATES.LOADED && isDropdownExpanded && !isEscapeClosed, [disabled, isDropdownExpanded, isEscapeClosed, loadState]);
+  const descriptionId = `${domId}-description`;
+  const errorMessageId = `${domId}-error-message`;
   return jsxs("div", {
     ref: focusScopeRef,
     class: formFieldClasses(type$5, {
@@ -4450,7 +4480,7 @@ function Taglist(props) {
     children: [jsx(Label, {
       label: label,
       required: required,
-      id: domId
+      htmlFor: domId
     }), !disabled && !readonly && !!values.length && jsx(SkipLink, {
       className: "fjs-taglist-skip-link",
       label: "Skip to search",
@@ -4498,7 +4528,9 @@ function Taglist(props) {
         onMouseDown: () => setIsEscapeClose(false),
         onFocus: onInputFocus,
         onBlur: onInputBlur,
-        "aria-describedby": errorMessageId
+        "aria-describedby": [descriptionId, errorMessageId].join(' '),
+        required: required,
+        "aria-invalid": errors.length > 0
       })]
     }), jsx("div", {
       class: "fjs-taglist-anchor",
@@ -4510,10 +4542,11 @@ function Taglist(props) {
         listenerElement: inputRef.current
       })
     }), jsx(Description, {
+      id: descriptionId,
       description: description
     }), jsx(Errors, {
-      errors: errors,
-      id: errorMessageId
+      id: errorMessageId,
+      errors: errors
     })]
   });
 }
@@ -4830,7 +4863,6 @@ function Textfield(props) {
   const {
     disabled,
     errors = [],
-    errorMessageId,
     domId,
     onBlur,
     onFocus,
@@ -4866,6 +4898,8 @@ function Textfield(props) {
   const onInputFocus = () => {
     onFocus && onFocus();
   };
+  const descriptionId = `${domId}-description`;
+  const errorMessageId = `${domId}-error-message`;
   return jsxs("div", {
     class: formFieldClasses(type$2, {
       errors,
@@ -4873,7 +4907,7 @@ function Textfield(props) {
       readonly
     }),
     children: [jsx(Label, {
-      id: domId,
+      htmlFor: domId,
       label: label,
       required: required
     }), jsx(TemplatedInputAdorner, {
@@ -4891,13 +4925,16 @@ function Textfield(props) {
         onFocus: onInputFocus,
         type: "text",
         value: value,
-        "aria-describedby": errorMessageId
+        "aria-describedby": [descriptionId, errorMessageId].join(' '),
+        required: required,
+        "aria-invalid": errors.length > 0
       })
     }), jsx(Description, {
+      id: descriptionId,
       description: description
     }), jsx(Errors, {
-      errors: errors,
-      id: errorMessageId
+      id: errorMessageId,
+      errors: errors
     })]
   });
 }
@@ -4930,7 +4967,6 @@ function Textarea(props) {
   const {
     disabled,
     errors = [],
-    errorMessageId,
     domId,
     onBlur,
     onFocus,
@@ -4974,6 +5010,8 @@ function Textarea(props) {
   useEffect(() => {
     autoSizeTextarea(textareaRef.current);
   }, []);
+  const descriptionId = `${domId}-description`;
+  const errorMessageId = `${domId}-error-message`;
   return jsxs("div", {
     class: formFieldClasses(type$1, {
       errors,
@@ -4981,7 +5019,7 @@ function Textarea(props) {
       readonly
     }),
     children: [jsx(Label, {
-      id: domId,
+      htmlFor: domId,
       label: label,
       required: required
     }), jsx("textarea", {
@@ -4994,12 +5032,15 @@ function Textarea(props) {
       onFocus: onInputFocus,
       value: value,
       ref: textareaRef,
-      "aria-describedby": errorMessageId
+      "aria-describedby": [descriptionId, errorMessageId].join(' '),
+      required: required,
+      "aria-invalid": errors.length > 0
     }), jsx(Description, {
+      id: descriptionId,
       description: description
     }), jsx(Errors, {
-      errors: errors,
-      id: errorMessageId
+      id: errorMessageId,
+      errors: errors
     })]
   });
 }
@@ -5168,7 +5209,7 @@ function Table(props) {
   return jsxs("div", {
     class: formFieldClasses(type),
     children: [jsx(Label, {
-      id: prefixId(id),
+      htmlFor: prefixId(id),
       label: label
     }), jsxs("div", {
       class: classNames('fjs-table-middle-container', {
@@ -5568,7 +5609,7 @@ class FormFields {
 }
 
 const EXPRESSION_PROPERTIES = ['alt', 'appearance.prefixAdorner', 'appearance.suffixAdorner', 'conditional.hide', 'description', 'label', 'source', 'readonly', 'text', 'validate.min', 'validate.max', 'validate.minLength', 'validate.maxLength', 'valuesExpression', 'url', 'dataSource', 'columnsExpression'];
-const TEMPLATE_PROPERTIES = ['alt', 'appearance.prefixAdorner', 'appearance.suffixAdorner', 'description', 'label', 'source', 'text', 'url'];
+const TEMPLATE_PROPERTIES = ['alt', 'appearance.prefixAdorner', 'appearance.suffixAdorner', 'description', 'label', 'source', 'text', 'content', 'url'];
 
 /**
  * @typedef { import('../types').Schema } Schema
@@ -6554,47 +6595,17 @@ class RepeatRenderManager {
     };
     const parentExpressionContextInfo = useContext(LocalExpressionContext);
     return jsx(Fragment, {
-      children: displayValues.map((value, index) => {
-        const elementProps = useMemo(() => ({
-          ...restProps,
-          indexes: {
-            ...(indexes || {}),
-            [repeaterField.id]: index
-          }
-        }), [index]);
-        const localExpressionContextInfo = useMemo(() => ({
-          data: parentExpressionContextInfo.data,
-          this: value,
-          parent: buildExpressionContext(parentExpressionContextInfo),
-          i: [...parentExpressionContextInfo.i, index + 1]
-        }), [index, value]);
-        return !showRemove ? jsx(LocalExpressionContext.Provider, {
-          value: localExpressionContextInfo,
-          children: jsx(RowsRenderer, {
-            ...elementProps
-          })
-        }) : jsxs("div", {
-          class: "fjs-repeat-row-container",
-          children: [jsx("div", {
-            class: "fjs-repeat-row-rows",
-            children: jsx(LocalExpressionContext.Provider, {
-              value: localExpressionContextInfo,
-              children: jsx(RowsRenderer, {
-                ...elementProps
-              })
-            })
-          }), jsx("button", {
-            class: "fjs-repeat-row-remove",
-            type: "button",
-            "aria-label": `Remove list item ${index + 1}`,
-            onClick: () => onDeleteItem(index),
-            children: jsx("div", {
-              class: "fjs-repeat-row-remove-icon-container",
-              children: jsx(DeleteSvg, {})
-            })
-          })]
-        });
-      })
+      children: displayValues.map((value, index) => jsx(RepetitionScaffold, {
+        index: index,
+        value: value,
+        parentExpressionContextInfo: parentExpressionContextInfo,
+        repeaterField: repeaterField,
+        RowsRenderer: RowsRenderer,
+        indexes: indexes,
+        onDeleteItem: onDeleteItem,
+        showRemove: showRemove,
+        ...restProps
+      }, index))
     });
   }
   RepeatFooter(props) {
@@ -6686,6 +6697,73 @@ class RepeatRenderManager {
     return nonCollapsedItems ? nonCollapsedItems : DEFAULT_NON_COLLAPSED_ITEMS;
   }
 }
+
+/**
+ * Individual repetition of a repeated field and context scaffolding.
+ *
+ * @param {Object} props
+ * @param {number} props.index
+ * @param {Object} props.value
+ * @param {Object} props.parentExpressionContextInfo
+ * @param {Object} props.repeaterField
+ * @param {Function} props.RowsRenderer
+ * @param {Object} props.indexes
+ * @param {Function} props.onDeleteItem
+ * @param {boolean} props.showRemove
+ */
+
+const RepetitionScaffold = props => {
+  const {
+    index,
+    value,
+    parentExpressionContextInfo,
+    repeaterField,
+    RowsRenderer,
+    indexes,
+    onDeleteItem,
+    showRemove,
+    ...restProps
+  } = props;
+  const elementProps = useMemo(() => ({
+    ...restProps,
+    indexes: {
+      ...(indexes || {}),
+      [repeaterField.id]: index
+    }
+  }), [index, indexes, repeaterField.id, restProps]);
+  const localExpressionContextInfo = useMemo(() => ({
+    data: parentExpressionContextInfo.data,
+    this: value,
+    parent: buildExpressionContext(parentExpressionContextInfo),
+    i: [...parentExpressionContextInfo.i, index + 1]
+  }), [index, parentExpressionContextInfo, value]);
+  return !showRemove ? jsx(LocalExpressionContext.Provider, {
+    value: localExpressionContextInfo,
+    children: jsx(RowsRenderer, {
+      ...elementProps
+    })
+  }) : jsxs("div", {
+    class: "fjs-repeat-row-container",
+    children: [jsx("div", {
+      class: "fjs-repeat-row-rows",
+      children: jsx(LocalExpressionContext.Provider, {
+        value: localExpressionContextInfo,
+        children: jsx(RowsRenderer, {
+          ...elementProps
+        })
+      })
+    }), jsx("button", {
+      class: "fjs-repeat-row-remove",
+      type: "button",
+      "aria-label": `Remove list item ${index + 1}`,
+      onClick: () => onDeleteItem(index),
+      children: jsx("div", {
+        class: "fjs-repeat-row-remove-icon-container",
+        children: jsx(DeleteSvg, {})
+      })
+    })]
+  });
+};
 RepeatRenderManager.$inject = ['form', 'formFields', 'formFieldRegistry', 'pathRegistry'];
 
 const RepeatRenderModule = {