@strictly/react-form 0.0.26 → 0.0.27

package/.out/core/mobx/form_model.d.ts

@@ -59,6 +59,7 @@ export declare abstract class FormModel<T extends Type, ValueToTypePaths extends
     setFieldValue<K extends keyof ValuePathsToAdapters>(valuePath: K, value: ToOfFieldAdapter<ValuePathsToAdapters[K]>, validation?: Validation): boolean;
     addListItem<K extends keyof FlattenedListTypesOfType<T>>(valuePath: K, elementValue?: Maybe<ElementOfArray<FlattenedValuesOfType<T>[K]>>, index?: number): void;
     removeListItem<K extends keyof FlattenedListTypesOfType<T>>(...elementValuePaths: readonly `${K}.${number}`[]): void;
+    protected moveListItem<K extends keyof FlattenedListTypesOfType<T>>(fromValuePath: K, toValuePath: K): void;
     private internalSetFieldValue;
     /**
      * Forces an error onto a field. Error will be removed if the field value changes

package/.out/core/mobx/form_model.js

@@ -432,6 +432,8 @@ let FormModel = (() => {
                             const validation = this.validation[fromJsonPath];
                             delete this.validation[fromJsonPath];
                             this.validation[toJsonPath] = validation;
+                            // eslint-disable-next-line @typescript-eslint/consistent-type-assertions, @typescript-eslint/no-explicit-any
+                            this.moveListItem(fromJsonPath, toJsonPath);
                         });
                         accessor.set(newList);
                         // delete any value overrides so the new list isn't shadowed
@@ -440,6 +442,12 @@ let FormModel = (() => {
                     });
                 });
             }
+            moveListItem(fromValuePath, toValuePath) {
+                // do nothing, this is for subclasses to override
+                // put in some nonsense so TS doesn't complain about the parameters not being used
+                fromValuePath;
+                toValuePath;
+            }
             internalSetFieldValue(valuePath, value, validation) {
                 const { revert } = this.getAdapterForValuePath(valuePath);
                 assertExists(revert, 'setting value not supported {}', valuePath);

package/.out/core/mobx/hooks.js

@@ -19,8 +19,8 @@ export function useDefaultMobxFormHooks(model, { onValidFieldSubmit, onValidForm
         // (e.g. changing a discriminator)
         // TODO debounce?
         setTimeout(function () {
-            // only start validation if the user has changed the field
-            if (model.isValuePathActive(path) && model.isFieldDirty(path)) {
+            // only start validation if the user has changed the field and there isn't already an error visible
+            if (model.isValuePathActive(path) && model.isFieldDirty(path) && model.fields[path].error == null) {
                 // further workaround to make sure we don't downgrade the existing validation
                 model.validateField(path, Math.max(Validation.Changed, model.getValidation(path)));
             }